project-anacapa / course-github-org-tool

PLEASE USE: https://github.com/project-anacapa/github-roster-tool instead!
https://github.com/project-anacapa/github-roster-tool
MIT License
1 stars 4 forks source link

Refactor `instructorize` method in User model #49

Open jaeaster opened 6 years ago

jaeaster commented 6 years ago

Instructorize is messy, unreadable and has unwanted side effects.

We should be using the machine user for all admin-specific organization reads/writes and not the instructors github privileges. Furthermore, we should only ask for the standard user:email scope from our users instead of user:email, admin:org.