Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Switch group set reads to the GroupSetService #6844

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Conversation

marcospri
Copy link
Member

@marcospri marcospri commented Nov 7, 2024

Testing

Reading group sets is used during course copy.

  • First we need to fetch the groups on the original course to simulate it's initial configuration (when we store the groups).

Open https://aunltd.brightspacedemo.com/d2l/le/content/6782/viewContent/2132/View, edit it and force loading the group sets. No need to save the edit.

  • Check the groups set are in the DB
select * from lms_group_set;
 id | lms_id |          name          | lms_course_id |          created           |          updated           
----+--------+------------------------+---------------+----------------------------+----------------------------
 38 | 22     | Group Category testing |           339 | 2024-11-07 14:23:43.794203 | 2024-11-07 14:23:43.794203
 39 | 23     | Empty group category   |           339 | 2024-11-07 14:23:43.794203 | 2024-11-07 14:23:43.794203
 40 | 26     | Support test Group Set |           339 | 2024-11-07 14:23:43.794203 | 2024-11-07 14:23:43.794203
 41 | 65     | MMC Category Settings  |           339 | 2024-11-07 14:23:43.794203 | 2024-11-07 14:23:43.794203
(4 rows)
  • Launch the equivalent assigment in a copied course:

https://aunltd.brightspacedemo.com/d2l/le/content/6888/viewContent/2656/View?ou=6888 (you'll get an error about the group set being empty, that's fine)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant