Closed Bug 955945 Opened 10 years ago Closed 10 years ago

Implement age verification screens for FxA Signup


(Firefox OS Graveyard :: FxA, defect)

Gonk (Firefox OS)
Not set



feature-b2g 2.0


(Reporter: spenrose, Assigned: ferjm)



(Whiteboard: [ucid:FxA48, 1.4:p1, ft:FirefoxAccounts])


(2 files)

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.
bug 949609 is Bengali, I suspect you wanted to block another bug?
No longer blocks: 949609
(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.
Blocks: 949069
Whiteboard: [ucid:FxA48, 1.4:p1, ft:FirefoxAccounts]
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.
Flags: needinfo?(spenrose)
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.
Flags: needinfo?(spenrose)
Blocks: 965492
No longer blocks: 965492
Blocks: 955952
removing from blocker list. COPPA is not committed for 1.4
whoops, the spreadsheet is a lie. reblocking
Assignee: nobody → ferjmoreno
Attached patch WIPSplinter Review
Comment on attachment 8382385 [details] [diff] [review]

I still need to add some tests for the new screen but I can use some feedback in the meantime.
Attachment #8382385 - Flags: feedback?(borja.bugzilla)
The PR contains tests now
Comment on attachment 8382385 [details] [diff] [review]

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! :)
Attachment #8382385 - Flags: feedback?(borja.bugzilla) → feedback+
Attached file UX
Comment on attachment 8382385 [details] [diff] [review]

Thanks Borja! I've just updated the PR.
Attachment #8382385 - Attachment is patch: true
Attachment #8382385 - Attachment mime type: text/x-github-pull-request → text/plain
Attachment #8382385 - Flags: review?(alive)
Comment on attachment 8382385 [details] [diff] [review]

Please make any new file jshint friendly! Thanks.
Attachment #8382385 - Flags: review?(alive)
Comment on attachment 8382385 [details] [diff] [review]

Thanks Alive. I've just update the PR.
Attachment #8382385 - Flags: review?(alive)
Attachment #8382385 - Flags: review?(alive) → review+
Thanks Alive!
Closed: 10 years ago
Resolution: --- → FIXED
Flags: in-moztrap+
Component: Identity → FxA
Product: Core → Firefox OS
feature-b2g: --- → 2.0
You need to log in before you can comment on or make changes to this bug.