-
Notifications
You must be signed in to change notification settings - Fork 58
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
Add GitHub Oauth registration and login #101
Comments
Can I work on it? @isabelcosta |
@jalajcodes You are assigned to #63. @isabelcosta I want to give it a try. |
assigned to @codesankalp because you're not assigned to anything |
@isabelcosta Can I create an issue on the frontend also for this and start working on both as I will need the frontend to test the implementation from the backend? |
Yes, you can do that and build the feature full stack. Do create the issue and link to this comment of mine and to this issue. @codesankalp |
Update: Implemented the Github OAuth on both frontend and backend but facing problem in making jwt authorization. Will solve this problem and make a PR asap. |
Awesome @codesankalp I love me some updates 💯 thank you! |
Added GitHub Oauth which uses JWT REST API for authenticating users. Fixes anitab-org#101
Added GitHub Oauth which uses JWT REST API for authenticating users. Fixes anitab-org#101
Added GitHub Oauth which uses JWT REST API for authenticating users. Fixes anitab-org#101
Added GitHub Oauth which uses JWT REST API for authenticating users. Fixes anitab-org#101
Added GitHub Oauth which uses JWT REST API for authenticating users. Fixes anitab-org#101
Description
As a developer,
I need to implement GitHub sign on,
so that I can allow GitHub users to login using their GitHub account.
For being able to use the application, the user has to create a new account and confirm the email just to login. It's a long process. So by having GitHub OAuth we can make the process shorter for users who have a GitHub account.
Acceptance Criteria
Update [Required]
Definition of Done
Estimation
8 hours
The text was updated successfully, but these errors were encountered: