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

Data 100 Summer Staff Privileges #5802

Closed
Kevin-Miao opened this issue Jun 14, 2024 · 13 comments · Fixed by #5803 or #5804
Closed

Data 100 Summer Staff Privileges #5802

Kevin-Miao opened this issue Jun 14, 2024 · 13 comments · Fixed by #5803 or #5804
Assignees
Labels

Comments

@Kevin-Miao
Copy link

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.

@Kevin-Miao
Copy link
Author

CC: @jcbym

@balajialg
Copy link
Contributor

balajialg commented Jun 15, 2024

@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.

@balajialg balajialg reopened this Jun 15, 2024
@Kevin-Miao
Copy link
Author

Thank you Balaji! 🙌

@shaneknapp
Copy link
Contributor

it's actually not deployed yet... there's something up w/the config. i'll try and get this debugged ASAP.

@shaneknapp
Copy link
Contributor

ok, this is now live on data100-staging.datahub.berkeley.edu ... let me know if you can see your elevated privs!

@Kevin-Miao
Copy link
Author

I can log into staging, but do not have access to the elevated permissions.

@balajialg
Copy link
Contributor

balajialg commented Jun 17, 2024

@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!

@Kevin-Miao
Copy link
Author

@balajialg Hmm it doesn't let me add staff. Since Canvas only allows for students to be added to the group.

@balajialg
Copy link
Contributor

@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)

@ryanlovett
Copy link
Collaborator

@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.

@balajialg
Copy link
Contributor

balajialg commented Jun 21, 2024

@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.

@Kevin-Miao
Copy link
Author

I can confirm @ryanlovett 's solution works! Thank you for helping us debug this. :)

@balajialg
Copy link
Contributor

Awesome

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