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

Jd/big blue button #4

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Jd/big blue button #4

wants to merge 3 commits into from

Conversation

dittonjs
Copy link

No description provided.

…roup or section to a conference.

This change adds checkboxes for groups and sections to the members list section of the dialog for creating / editing a conference. These checkboxes are "shortcuts" that allow the user to quickly add all members of a specific section or group to a conference. These shortcuts are implemented client side and the  model for conferences is unchanged.

Test Plan:
	- Make sure the conferences feature is enabled
	- In a course open the dialog to create a new conference
	- Uncheck the "Invite All" box
	- Verify the following in the member list:
		- A checkbox for each section should appear in the "Sections" partition (only if there is more than one section in the course)
		- A checkbox for each group should appear in the "Groups" partition (only if there is at least one section in the course)
		- The headers for "Sections" and "Groups" should only appear if there is atleast item in the respective partition
		- The header for "Users" should only appear if one or more of the other partitions is present
		- Checking the box for section / group also checks the boxes for each of the users in that section / group
		- Unchecking the box for section / group also unchecks the boxes for each of the users in that section / group provided that another section / group that contains that user isn't also checked
		- If a user is added to the conference via a section or group you should not be able to uncheck the checkbox for that user.
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.

2 participants