DevProgress / HillaryBNB

Home sharing app for Hillary Clinton supporters
Other
10 stars 13 forks source link

Reinstate /stats page, restrict it to signed-in admin users #1

Closed dsjoerg closed 8 years ago

dsjoerg commented 8 years ago

Used to be here: https://github.com/dsjoerg/BernieBNB/blob/45aca31177807006e347ccb502e17af2f0dfadf0/app/views/pages/stats.html.haml

It was removed in https://github.com/dsjoerg/BernieBNB/commit/22b838a564641275173186e83f0378d0f1f9bb31 and can be reinstated once this issue is resolved.

wbronitsky commented 8 years ago

Currently, there is no concept of different user types. To solve this issue, should we add that concept? We could whitelist emails or something along those lines to decide who is an admin user and who isn't.

dsjoerg commented 8 years ago

@wbronitsky yes i'd say add a boolean flag to the User model for admin

dsjoerg commented 8 years ago

@mattjacobus has a PR in progress for this: https://github.com/DevProgress/HillaryBNB/pull/39

dsjoerg commented 8 years ago

Done by @mattjacobus