Closed Bug 991351 (mdn-gh-login) Opened 10 years ago Closed 10 years ago

Users should be able to log in with their GitHub account

Categories

(developer.mozilla.org Graveyard :: Sign-in, enhancement)

All
Other
enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mars, Assigned: mars)

References

Details

(Whiteboard: [specification][type:feature])

What problems would this solve?
===============================
Evidence suggests that a fair number of MDN users have GitHub accounts.  It may be easier for these users to get involved with MDN if they can sign in or sign up with their GitHub account.

Who would use this?
===================
Anyone visiting MDN who has an active GitHub account.

What would users see?
=====================
Logging in to MDN with authentication delegated to GitHub.  Possibly a streamlined signup process as well.

What would users do? What would happen as a result?
===================================================
Standard OAuth sign-in/sign-up delegation, presumably a 'Log in with GitHub' tile like what you find on Stack Overflow's login page.

Is there anything else we should know?
======================================
Severity: normal → enhancement
Component: General → Login
Depends on: 992989
Assignee: nobody → mars
:lorchard and :jezdez - Let's make the call on django-allauth vs. roll-our-own. (https://github.com/mozilla/kuma/issues/2135)
Flags: needinfo?(lorchard)
Flags: needinfo?(jezdez)
(In reply to Luke Crouch [:groovecoder] from comment #1)
> :lorchard and :jezdez - Let's make the call on django-allauth vs.
> roll-our-own. (https://github.com/mozilla/kuma/issues/2135)

My main concerns orbit around the weirdness of MDN's sign-in legacy.

I think the weirdness around the old single-sign-in for Django & Mindtouch & phpBB is so far in the past, that we can ignore it. Though, of course, I always wait for it to be a problem again, lurking somewhere

That just leaves needing to have OAuth/Github and Persona co-exist as sign-in methods, at least for now.  As long as django-allauth can bend into the right shape to deal with that, I'm fine with it.
Flags: needinfo?(lorchard)
Holly's first mock-up/wire-frame for the sign-in button: http://cl.ly/image/2w0K3P1n3M0m
Interesting! I prefer just Step 1, and to show a log-in solely for the option the user chose (whether that's Persona, GitHub etc). If Step 2 is deemed needed (is it?) with showing the choices a second time, I prefer version A.
IIRC, we want Step 2 because users may accidentally click the wrong provider icon (because they are so small) to trigger such a large experience - i.e., redirecting off-site.
I see. I envisioned it to be a pop-up or similar with some info, potential authorization etc in context for the choice they made, and not send them off directly to another web site, e.g. GitHub.
:habber - do you have a suggestion for pop-up vs. window redirect?
Flags: needinfo?(hhabstritt.bugzilla)
Making progress wiring up the server backend for GitHub sign-in. 

Since this will probably be easier to revise than a flood of bugzilla comments, I've started assembling a checklist of work I'm discovering along the way along with which bits I've completed:

https://github.com/mozilla/kuma/issues/2135#issuecomment-42215910
Depends on: 1011004
Depends on: 775485
Depends on: 1017085
Depends on: 1021712
Depends on: 1024086
Depends on: 1024507
Depends on: 1025348
Status: NEW → ASSIGNED
Depends on: 1029086
Depends on: 1036113
No longer depends on: 775485
Alias: mdn-gh-login
Depends on: 775485
Depends on: 1031547
Flags: needinfo?(jezdez)
Depends on: 866533
Depends on: 1047018
Depends on: 1047448
Depends on: 1045187
Blocks: 795739
Depends on: 1051001
No longer depends on: 775485
Depends on: 1052453
Depends on: 1052463
No longer depends on: 1045187
Depends on: 1083161
We've launched!  https://groups.google.com/d/topic/mozilla.dev.mdn/BRLSaBAayss/discussion
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Flags: needinfo?(hhabstritt.bugzilla)
Resolution: --- → FIXED
Product: developer.mozilla.org → developer.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.