Email Confirmation
Summary
Users should receive a confirmation email to verify it's a real address when signing up or updating the email address.
Testing
- Go to https://goal-email-verification.minds.io/
- Create a new user using a REAL email address
- You should be logged in automatically and a blue banner asking for email confirmation will be shown
- Reload the page and ensure the banner is still there
- Check your email inbox (probably on Spam folder, might need to wait a bit). DO NOT CLICK ON THE LINK YET
- Go back to Minds tab, click on the Re-Send link
- Ensure you get another email. You might need to wait a couple of minutes.
- Click on the latest email's "Complete Setup" button
- Ensure the banner is not being shown on this new opened tab
- Reload the page and ensure the banner is not appearing
Issues
- Closes #1229 (closed)
added scoped label
added 2 commits
added 1 commit
- f4d604b3 - (refactor): Move delegate to its own class; (test): Spec tests
changed the description
mentioned in merge request front!699 (merged)
- Resolved by Emiliano Balbuena
- Resolved by Emiliano Balbuena
added 1 commit
- 82d496f3 - (feat): Re-send confirmation email; (feat): Reset when email changes
resolved all threads
- Developer
added 1 commit
- ee7dbe4b - (fix): Template fixes; ditch intermediate queue for confirm email
added 12 commits
-
89955644...bd60d664 - 11 commits from branch
master
- 936c5d6d - Merge remote-tracking branch 'origin/master' into goal/email-verification
-
89955644...bd60d664 - 11 commits from branch
added 1 commit
- 5ddf644e - (feat): Export when source is email confirmation
unmarked as a Work In Progress
assigned to @benhayward.ben
changed the description
changed the description
- Developer
On my todo list for approval. Not marked for review yet. No action taken.
mentioned in merge request mobile-native!463
added scoped label
changed the description
added scoped label and automatically removed label
- Developer
Moved to Requires Changes due to mobile deeplinking compatibility
mentioned in merge request mobile-native!467
added 12 commits
-
5ddf644e...42417e78 - 11 commits from branch
master
- 5a100825 - Merge remote-tracking branch 'origin/master' into goal/email-verification
-
5ddf644e...42417e78 - 11 commits from branch
added 30m of time spent at 2019-12-24
approved this merge request
- Developer
I received the verification email. However
- No blue banner for email verification
- None of the assets loaded in the email
- No banner on reload since no banner initially
- No resend email link, was it suppose to be on the blue banner?
No assets
No banner
Attempted in both Brave and Chrome
Also MR is still labelled requires changes.
added scoped label and automatically removed label
approved this merge request
approved this merge request
- Developer
Approved assuming the missing assets will not be an issue on production
assigned to @brianhatchet and unassigned @benhayward.ben
added scoped label and automatically removed label
merged
mentioned in commit a8f2ccf4