AmericanRedCross / damage-assessment-bot

Chatbot for immediate damage and needs assessments
BSD 3-Clause "New" or "Revised" License
7 stars 3 forks source link

As an admin, I want to be able to grant other users access to the admin UI #78

Open m4thfr34k opened 6 years ago

m4thfr34k commented 6 years ago

@dalekunce / @danbjoseph , 1) Is it sufficient to require the user being promoted to admin to have already logged in as a dashboard user or should this query the GO authentication platform to validate the user ID? 2) Does this need to assist the user, with a lookup, or will the user utilize a text field?

danbjoseph commented 6 years ago

1 - i'm not sure i understand the options? 2 -imo, an assist would help, but at least a warning if you submit a name and it's not someone that exists in the system?

maxnorth commented 6 years ago

@danbjoseph question 1 is asking if it's ok that it may not be possible to set up a user as an admin until they have already logged into the app, because we won't have a record of them in our DB until they do. The only way around this would be if we could query the GO api to verify a username. From what I'm seeing this doesn't look possible, but I've emailed them to check. We just want to make sure this potential limitation is understood and considered acceptable.

danbjoseph commented 6 years ago

only being able to promote a user that has created an account with the app and not being able to generate an admin seems like an acceptable limitation to me.

dalekunce commented 6 years ago

Agree with @danbjoseph no need to proactively make people admins before they have ever logged in.