Skip to content

Option for Clearing All Rotating Section Assignments#3361

Merged
bmcutler merged 1 commit intoSubmitty:masterfrom
zjxiaohan:drop_all
Mar 24, 2019
Merged

Option for Clearing All Rotating Section Assignments#3361
bmcutler merged 1 commit intoSubmitty:masterfrom
zjxiaohan:drop_all

Conversation

@zjxiaohan
Copy link
Copy Markdown
Contributor

Closes #2366

This pull request adds another option for clearing all rotating section assignments, in other words, this option makes all the assigned rotating sections of students/teams NULL.

remove_all

@bmcutler bmcutler requested a review from emaicus March 19, 2019 17:17
@bmcutler bmcutler added this to the Early Spring 2019 Tasks milestone Mar 19, 2019
Copy link
Copy Markdown
Contributor

@emaicus emaicus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR works as intended for both team and individual assignments and should be ready to go. As we continue to iterate on the rotating section portion of the manage sections page, we might want to take some time to consider its layout. It strikes me as odd that of the four options on the manage rotating sections form, only the final radio button, Redo rotating sections completely makes use of the interface for placing students into rotating sections, which dominates the form. Perhaps we should consider hiding this section of the GUI until the redo rotating sections completely radio button is selected.

@zjxiaohan
Copy link
Copy Markdown
Contributor Author

@emaicus Thank you for your review! I agree with you. Probably it is better to move that form below the redo option, and only show it when redo is checked. Another issue has been set up for further discussion on this. #3448

@bmcutler bmcutler merged commit f72822a into Submitty:master Mar 24, 2019
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.

3 participants