-
Notifications
You must be signed in to change notification settings - Fork 174
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
Rethink User management #111
Comments
I've got some ideas as to how we could do this: Option 1This could look confusing, but the way this is done is pretty standard for user management..
Option 2
thoughts, @nshntarora ? |
I'd vote for option 2. Other than the workflow being simpler, my reason is: Having said that, I'm not sure how it will mess up the permissions. Can you please elaborate some scenarios we might face issues in? |
Alrighty that makes sense! We could start working on sign ins with Slack, GitHub and GApps for now. My problems with this is that we might not be able to have granular control on roles, but now that I think about it, I might be totally wrong 😀 |
I have experience with Google OAuth. Should I get started with that? |
Of course! We could then deal with Slack, GitHub, Atlassian etc. |
@rja907 Thanks a lot of showing interest. Would be great if you could pick this up. Just start working on it, and create a PR, link it to this issue so that everyone else could keep track and maybe help you out along the way. If you need any kind of help in understanding the codebase, or anything else, just send me an email on |
+1 for option 2. A lot of wiki have also a LDAP support, I personally use this method for managing users |
At the moment the app has some problems with its user management.
The text was updated successfully, but these errors were encountered: