955945 - Implement age verification screens for FxA Signup

Status: RESOLVED FIXED

(Firefox OS Graveyard :: FxA, defect)

Description

[Sam Penrose]

Per UX specs, when a user enters the signin/signup flow and is sent down the signUP branch (because the FxA server believes their email to represent a new account), they should be sent through Age Verification flow UNLESS the signin/signup flow was triggered via FTU.

Comment 1

[Axel Hecht [:Pike]]

bug 949609 is Bengali, I suspect you wanted to block another bug?

Comment 2

[Sam Penrose]

(In reply to Axel Hecht [:Pike] from comment #1)
> bug 949609 is Bengali, I suspect you wanted to block another bug?

Yes, I transopsed a pair of digits ... apologies for the noise.

Comment 3

[Jared Hirsch [:jhirsch] (he/him) (Needinfo please)]

Sam, I imagine this should actually be filed under a bug for the system app? Settings really just launches the signup flow by opening the system app.

Comment 4

[Sam Penrose]

Jared: sure. I confess the Product/Component rules remain opaque to me. Referring to "Settings" in the description b/c that's how Product/UX experience it, of course.

Comment 5

[Jared Hirsch [:jhirsch] (he/him) (Needinfo please)]

removing from blocker list. COPPA is not committed for 1.4

Comment 6

[Jared Hirsch [:jhirsch] (he/him) (Needinfo please)]

whoops, the spreadsheet is a lie. reblocking

Comment 7

[Fernando Jiménez Moreno [:ferjm]]

Attachment: WIP

Comment 8

[Fernando Jiménez Moreno [:ferjm]]

Comment on attachment 8382385 [details] [diff] [review]
WIP

I still need to add some tests for the new screen but I can use some feedback in the meantime.

Comment 9

[Fernando Jiménez Moreno [:ferjm]]

The PR contains tests now

Comment 10

[Borja Salguero [:borjasalguero]]

Comment on attachment 8382385 [details] [diff] [review]
WIP

Tests coverage is great! This code is working as expected. Some comments to fix before the final review to this code. Let me know when ready! Gracias! :)

Comment 11

[Fernando Jiménez Moreno [:ferjm]]

Attachment: UX

Comment 12

[Fernando Jiménez Moreno [:ferjm]]

Comment on attachment 8382385 [details] [diff] [review]
WIP

Thanks Borja! I've just updated the PR.

Comment 13

[(inactive after 6/18) Alive Kuo [:alive]]

Comment on attachment 8382385 [details] [diff] [review]
WIP

Please make any new file jshint friendly! Thanks.

Comment 14

[Fernando Jiménez Moreno [:ferjm]]

Comment on attachment 8382385 [details] [diff] [review]
WIP

Thanks Alive. I've just update the PR.

Comment 15

[Fernando Jiménez Moreno [:ferjm]]

Thanks Alive!

https://github.com/mozilla-b2g/gaia/commit/b3758a90b8888e9d95128846b2833b4d9444ef7f