Skip to content
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

Enable github login #419

Closed
wants to merge 1 commit into from
Closed

Conversation

exussum12
Copy link
Contributor

Adds a button and the call to the API to enable github users to register
or login without an account here

Adds a button and the call to the API to enable github users to register
or login without an account here
@joindin-jenkins
Copy link

Can one of the admins verify this patch?

@liam-wiltshire
Copy link
Member

ok to test

@liam-wiltshire
Copy link
Member

Thanks for this @exussum12 - I'm waiting for joindin/joindin-api#424 to be sorted before I merge anything else that relies on the API as that PR touches lots of files, but as soon as that's through I'll look at this

@exussum12
Copy link
Contributor Author

I am going to update this to be id based instead of email so will need a little extra time either way

@mwillbanks mwillbanks added the WIP Work in Progress (DO NOT MERGE) label Feb 6, 2019
@svpernova09
Copy link
Contributor

svpernova09 commented Aug 21, 2019

Closing as we've also put on hold adding providers until we clean up the oauth code we're using

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WIP Work in Progress (DO NOT MERGE)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants