767220 - Use Identity Module for WebApps ID operations

Status: RESOLVED WONTFIX

(Web Apps Graveyard :: AppsInTheCloud, defect)

Description

[Anant Narayanan [:anant]]

Replace use of services/aitc/modules/browserid.js with Identity.jsm from bug 753238 when it is ready!

Comment 1

[Jason Smith [:jsmith]]

Could you explain the importance of this? Just wondering if this is a k9o nom or not, given that the dependency got a +.

Comment 2

[Anant Narayanan [:anant]]

Yes, please nom for k9o. This significantly improves the login UX for AITC.

Comment 3

[Justin Dolske [:Dolske]]

Is this purely backend? Or is it about making AITC's UX consistent with the native Persona support? (implies - for the former, + for the latter).

Comment 4

[Jason Smith [:jsmith]]

jsmith_	anant: Could you give some more background behind why bug 767220 is important?
	anant	jsmith_: in bug 745345, we promised gavin that we'd move over to the native ID implementation as soon as possible, and that's what 767220. Using the native ID module instead of the in-built BrowserID module will result in a better user experience for AITC logins
	anant	err, *that's what 767220 tracks
	jsmith_	anant: Okay. Could you explain more on the UX side? Like how does the end-user benefit from this?
	anant	jsmith_: so right now you get the ugly popup when visiting the dashboard. with native ID integration, we'll get a nice doorhanger login with explicit login/logout buttons etc.
	jsmith_	anant: Oh, that sounds far more plesant

Comment 5

[Jason Smith [:jsmith]]

jsmith_	anant: So this falls in line with making the UX consistent with native persona support, right (as Dolske put it)?
	anant	that's exactly what the bug is about, integration with native persona support
	jsmith_	anant: Gotcha

Comment 6

[Jason Smith [:jsmith]]

(In reply to Justin Dolske [:Dolske] from comment #3)
> Is this purely backend? Or is it about making AITC's UX consistent with the
> native Persona support? (implies - for the former, + for the latter).

AITC's UX consistent with native Persona support

Comment 7

[Sylvestre Ledru [:Sylvestre]]

Dead component, closing bugs in it.