sumeetjain / outcomes-tracker

0 stars 0 forks source link

Admin/Staff Users should add a list of Github usernames that can authenticate to the application #37

Closed halfghaninne closed 8 years ago

halfghaninne commented 8 years ago

No others should be allowed to automatically authenticate.

ORIGINAL PLAN HIDDEN

Maybe this happens like so: - Staff Users should have the ability to create Student Users - In the User model there should be some utility method `check_user_acess` - `check_user_access` should be called in a conditional in the `self.from_omniauth(auth_hash)` method in the User model.


[REVISED 9/2/16]

halfghaninne commented 8 years ago

Notes to self:

- Admin User creates a cohort - Admin User creates Student Users that belong to cohort - Students belong to cohorts through profiles. Therefore, Admin User must also create these. > A note for another time, maybe if a User is a Student they MUST have a cohort? Probably no way to do conditional validations on a single model like that - Student Users can only log into the application if they already exist in the application --- > A consideration after this amount of work is finished -- what happens if a student changes their email or github username?