-
Notifications
You must be signed in to change notification settings - Fork 39
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
Data 100 Summer Staff Privileges #5802
Comments
CC: @jcbym |
@Kevin-Miao I have merged the config changes to https://data100-staging.datahub.berkeley.edu/. You can test whether you have enhanced privileges in another 20 minutes (and let me know if you have any issues). I will let you know once the changes gets merged to prod. |
Thank you Balaji! 🙌 |
it's actually not deployed yet... there's something up w/the config. i'll try and get this debugged ASAP. |
ok, this is now live on data100-staging.datahub.berkeley.edu ... let me know if you can see your elevated privs! |
I can log into staging, but do not have access to the elevated permissions. |
@Kevin-Miao I created a new group named "Admins" in https://bcourses.berkeley.edu/courses/1535115. Can you please add the folks who require elevated permissions in this group? Once you add them to the group, ask them to log out and log back in so that the changes from bcourses gets applied. Let us know if you face any issues! |
@balajialg Hmm it doesn't let me add staff. Since Canvas only allows for students to be added to the group. |
@Kevin-Miao I have changed the allocation based on enrollment type and created a new PR. Once it gets merged to staging, will give an update @shaneknapp I haven't merged the PR to staging yet as I want your fresh pair of eyes (to avoid a deploy failure; I have tested it in yaml linter already) |
@Kevin-Miao If you need non-students to appear in manually created groups, you can also add them to the bCourse as students. One person can be both a student and an instructor, observer, ta, etc. It is odd that bCourses doesn't let non-student enrollment types into groups. @balajialg's patch uses the enrollment type to identify course staff, so if that finds the same people that are specified in your manual Admin group, it is easier to use the former. |
@Kevin-Miao I can close this PR if you go with @ryanlovett's suggestion to add users to admin group. If not, I can merge the above PR to staging. |
I can confirm @ryanlovett 's solution works! Thank you for helping us debug this. :) |
Awesome |
Your Name
Kevin Miao
Affiliated Course Name
Data 100
bCourses ID
1535115
Role
Instructor
Hub URL
data100.datahub.berkeley.edu
End Date
08/15/2024
Request Deadline
Sorry for the super late heads-up. It would be nice to get this set up by Wednesday 6/19.
The text was updated successfully, but these errors were encountered: