Closed Bug 971832 Opened 10 years ago Closed 10 years ago

Create frontend JS module to provide auth for Webmaker

Categories

(Webmaker Graveyard :: Login, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jon, Assigned: kate)

References

Details

(Whiteboard: login2-sprint)

Attachments

(1 file)

We'll need a frontend JS module to provide auth for Webmaker. It should:

1) Use navigator.id.get() to avoid Persona's state management
2) Doesn't log in on every page load
3) Has working SSO for webmaker.org
4) A better user creation experience
In testing, I noticed that webmaker-auth-client isn't emitting the user object to it's event listener callbacks for the login event.

for example: https://github.com/cadecairos/webmaker.org/compare/new-login#diff-08f8485df196e866f46aa7f87a4c4b62R27 - in this callback, user is undefined.
Blocks: 972502
No longer blocks: 972502
Whiteboard: login2
Attachment #8375937 - Flags: review? → review?(jon)
Whiteboard: login2 → login2-sprint
Comment on attachment 8375937 [details]
https://github.com/mozilla/webmaker-auth-client

https://github.com/mozilla/webmaker-auth-client
Attachment #8375937 - Attachment description: https://github.com/mozilla/login.webmaker.org/pull/236/files → https://github.com/mozilla/webmaker-auth-client
Attachment #8375937 - Flags: review?(jon) → review?(cade)
Comment on attachment 8375937 [details]
https://github.com/mozilla/webmaker-auth-client

There's some issues when you run `grunt validate` and `grunt jshint:all`
Comment on attachment 8375937 [details]
https://github.com/mozilla/webmaker-auth-client

Needs some lint fixes, also jsbeautifier says it needs to be run. Otherwise this looks great.
Attachment #8375937 - Flags: review?(cade) → review-
Comment on attachment 8375937 [details]
https://github.com/mozilla/webmaker-auth-client

Ok it's all beautified
Attachment #8375937 - Flags: review- → review?
Attachment #8375937 - Flags: review? → review?(cade)
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: