Closed damianhxy closed 4 months ago
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Description
Re-allow instructors to be CAs.
Motivation and Context
2065 prevented instructors from also being CAs. However, this caused problems in parts of the code that set users to be both. Furthermore, it might be desirable to allow instructors to also be CAs since only CAs have sections.
This PR re-allows instructors to be CAs, but improves upon #1420 by disabling the instructor & ca fields when the dropped checkbox is clicked.
How Has This Been Tested?
Edit a user, and check that dropped checkbox disables instructor / ca checkboxes and vice versa.
dropped selected
instructor / ca selected
none selected
Types of changes
Checklist:
overcommit --install && overcommit --sign
to use pre-commit hook for linting