Skip to content
This repository has been archived by the owner on Jan 26, 2021. It is now read-only.

Allow user signup using GIthub #1027

Open
5 tasks
Kajol-Kumari opened this issue Aug 23, 2020 · 18 comments · May be fixed by #1082
Open
5 tasks

Allow user signup using GIthub #1027

Kajol-Kumari opened this issue Aug 23, 2020 · 18 comments · May be fixed by #1082
Assignees
Labels
Category: Coding Changes to code base or refactored code that doesn't fix a bug. Open Source Hack

Comments

@Kajol-Kumari
Copy link

Description

Currently user can only create a new account using his/her emailID and have to fill all the details manually.

Acceptance Criteria

Update

  • Allow users to signup using gitHub.

  • Collect all the basic user details from github only.

  • Show user a dialog box after signup through Github having a message telling them to complete their profile in order to proceed.

Definition of Done

  • All of the required items are completed.
  • Approval by 1 mentor.

Estimation

48 hours

@Kajol-Kumari Kajol-Kumari added Category: Coding Changes to code base or refactored code that doesn't fix a bug. Open Source Hack labels Aug 23, 2020
@Devanshi-123
Copy link

Can i work on this?

@Kajol-Kumari
Copy link
Author

Kajol-Kumari commented Aug 25, 2020

@Devanshi-123 It's good to see that you are interested in taking up this issue. But all the issues with the label of open source hack will be assigned to the OSH participants starting from 1st of september so please wait patiently till 1st of september.

@Devanshi-123
Copy link

Devanshi-123 commented Aug 25, 2020 via email

@SanketDG
Copy link

Do you mean to integrate oAuth here? @Kajol-Kumari

@Kajol-Kumari
Copy link
Author

@SanketDG yes, but i didn't mentioned it explicitly as i want assignee to do a bit of research on finding out the best way to implement it :)

@Kajol-Kumari
Copy link
Author

Hey Everybody, all who are interested in this issue. Please accept my apologies but according to the new guidelines we received, we will count First come first serve on the basis of comments/claims done 1st september onwards. So, everyone who are interested in this issue, do come again on 1st and comment first to get assigned.

@Nibba2018
Copy link

Hello @Kajol-Kumari , Can I work on this issue?

@Kajol-Kumari
Copy link
Author

@Nibba2018 please go ahead with this issue.

@Nibba2018
Copy link

Thank you @Kajol-Kumari

@Nibba2018 Nibba2018 linked a pull request Sep 2, 2020 that will close this issue
8 tasks
@satya7289
Copy link
Contributor

@Kajol-Kumari, It'll be great if you update the issue with allowing a user to sign in/signup using google, and Facebook as well.

@Kajol-Kumari
Copy link
Author

@satya7289 I will be opening single issue for each of them as keeping everything in single issue will led to the huge PR.

@sanjibansg
Copy link

Do assign the issue to me if available.

@Kajol-Kumari
Copy link
Author

@Nibba2018 Can you please provide the update here?

@Nibba2018
Copy link

Nibba2018 commented Sep 28, 2020

@Kajol-Kumari Everything else is implemented except for the auto-fill part. #1082

@Kajol-Kumari
Copy link
Author

@Kajol-Kumari Everything else is implemented except for the auto-fill part. #1082

Can you please implement it ASAP and change the PR from Draft to Ready to Review?

@Nibba2018
Copy link

@Kajol-Kumari we discussed this issue on Zulip and you decided that you will look into it as I was not able to implement it correctly.

@Kajol-Kumari
Copy link
Author

@Kajol-Kumari we discussed this issue on Zulip and you decided that you will look into it as I was not able to implement it correctly.

Ah, probably I missed it, btw If I am not wrong I had tagged @satya7289 to help you out with this. Btw I would like to see what you have implemented till now. Can you please share a GIF in the PR description?

@Nibba2018
Copy link

@Kajol-Kumari Done

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Category: Coding Changes to code base or refactored code that doesn't fix a bug. Open Source Hack
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants