show blue email confirmation banner if UnverifiedEmailException is thrown from the backend
closes #2428 (closed)
Testing
This MR is for showing the blue email confirmation banner when a certain exception is thrown from the backend, which only applies to new registered users that haven't verified their emails.
First use an old user (older than the email confirmation task), and you should be able to do all of the following:
-
Create a post -
Comment on a post -
Vote up or down -
Create group -
Create blog -
Upload images -
Upload movies
Second, create a new user, don't verify its email address, hide the blue email confirmation banner by clicking on the X icon and try to do the same. Each time you try to do any of those, the blue banner should re-appear
Merge request reports
Activity
- Resolved by Marcelo Rivera
I can't create a new user on the review site - when I fill out the register form, the 'join minds now' button never becomes enabled.
added 33 commits
-
6da671ee...266c3f0d - 31 commits from branch
master
- 066ab5e7 - Merge remote-tracking branch 'upstream/master' into feat/unverified-users-permissions
- d865f4b2 - (fix): properly show verification banner for blogs
-
6da671ee...266c3f0d - 31 commits from branch
- Resolved by Marcelo Rivera
There are merge conflicts and I suspect the backend is out of sync because I was able to post this blog while being unverified
https://feat-unverified-users-permissions.minds.io/brianhatchet10/blog/deny-me-1078738026168848399
added 56 commits
-
d865f4b2...c64a0685 - 55 commits from branch
master
- 55cabcd8 - Merge remote-tracking branch 'upstream/master' into feat/unverified-users-permissions
-
d865f4b2...c64a0685 - 55 commits from branch
added 80 commits
-
55cabcd8...f335566b - 79 commits from branch
master
- 48875f7b - Merge remote-tracking branch 'upstream/master' into feat/unverified-users-permissions
-
55cabcd8...f335566b - 79 commits from branch
Yep, but we talked with Brian about first shipping it like this. There's no problem in adding a new banner, though. Please let me know your thoughts, @brianhatchet
mentioned in commit 4657a243