1123766 - Implement email confirmation screen

Status: VERIFIED FIXED

(support.mozilla.org :: BuddyUp, defect, P2)

Comment 1

[Kadir Topal [:atopal]]

moving to next sprint

Comment 2

[Rehan Dalal (Inactive)]

Attachment: https://github.com/mozilla/buddyup/pull/77

Sorry, forgot to attach this when I filed it.

Comment 3

[Anthony Ricaud (:rik)]

Comment on attachment 8563376 [details] [review]
https://github.com/mozilla/buddyup/pull/77

I think this needs a new iframe approach to cover for other usage of this panel.

Comment 4

[Anthony Ricaud (:rik)]

I looked into registration a bit more through my work on notifications. I don't understand how this flow can work.

When we register a user, User.get_token() returns a 400 saying that 'User account is disabled'. Therefore, we don't store the user credentials in asyncStorage. So when we poll, User.get_user() is polling for the auto-generated account that we currently have. And it's already active.

Comment 5

[Anthony Ricaud (:rik)]

Comment on attachment 8563376 [details] [review]
https://github.com/mozilla/buddyup/pull/77

So we know this will need some refactoring when we display this window on app launch. But to move forward, let's already land this.

Can you open the bug for app opening and mark it as a [blocker] so we account for it?

Comment 6

[Rehan Dalal (Inactive)]

Landed:
https://github.com/mozilla/buddyup/commit/8ad03854e06ef07fa34b539fbf2334dbce5790ef