-
Notifications
You must be signed in to change notification settings - Fork 13
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
Get rid of usernames #27
Comments
That's more or less correct, but, as far as OpenSpending is concerned, we still have an unresolved issue here: We have assumed usernames would be part of the system, and part of the namespacing of packages. Though, @tryggvib raised the issue as part of the migration that current data has multiple users, which breaks the pattern of So, I suggest to hold on this for the time being: I'm not particularly keen on namespacing packages by uuid, nor on enforcing unique names for packages system wide. |
Understood! So far, my sense had been that Google may be the best OAuth to go with, simply because it is so universal, and offers the option to hook into Google Docs later. I guess the usernames req puts the alternatives at using twitter or github, or running something separate (which makes a whole new project...). Or having this weird second-level dialogue, but I'd kind of like to avoid that. |
I'm pretty partial to Google as OAuth provider, to get running at least. But, it has been quite interesting that each year, people pull us up on this for the Open Data Census (require evil corp. to access functionality). |
Looks like the future might hold OAuth, according to discussions with @pwalsh. A sensible first step towards making SpenDB into an OAuth consumer would be to rip user names out of the app, and use IDs instead for profile pages and the account API. People can still log in using their email, and we can later switch that to OAuth which doesn't reliably provide user names.
The text was updated successfully, but these errors were encountered: