Closed Bug 955945 Opened 11 years ago Closed 11 years ago

Implement age verification screens for FxA Signup

Categories

(Firefox OS Graveyard :: FxA, defect)

Product:
Firefox OS Graveyard
Component:
FxA
Platform:
All
Gonk (Firefox OS)
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(feature-b2g:2.0)

Status:
RESOLVED FIXED
Project Flags:
feature-b2g 2.0

People

(Reporter: spenrose, Assigned: ferjm)

References

Details

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

Attachments

(2 files)

WIP
46 bytes, patch
alive
: review+
borjasalguero
: feedback+
Details | Diff | Splinter Review
UX
55.82 KB, application/pdf
Details
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 WIP — — Splinter Review
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.
Attachment #8382385 - Flags: feedback?(borja.bugzilla)
The PR contains tests now
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! :)
Attachment #8382385 - Flags: feedback?(borja.bugzilla) → feedback+
Attached file UX —
Comment on attachment 8382385 [details] [diff] [review] WIP 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] WIP Please make any new file jshint friendly! Thanks.
Attachment #8382385 - Flags: review?(alive)
Comment on attachment 8382385 [details] [diff] [review] WIP Thanks Alive. I've just update the PR.
Attachment #8382385 - Flags: review?(alive)
Attachment #8382385 - Flags: review?(alive) → review+
Thanks Alive! https://github.com/mozilla-b2g/gaia/commit/b3758a90b8888e9d95128846b2833b4d9444ef7f
Status: NEW → RESOLVED
Closed: 11 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.

Attachment

General

Creator:
Created:
Updated:
Size: