-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add notification-dot for user with unverified account_status #724
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #724 +/- ##
=======================================
Coverage 66.89% 66.89%
=======================================
Files 62 62
Lines 3727 3727
=======================================
Hits 2493 2493
Misses 1234 1234 |
Passing run #1763 ↗︎
Details:
Review all test suite changes for PR #724 ↗︎ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good so far, one minor adjustment
pre-commit.ci autofix |
…ip with popper.js Remove accidental yarn files
204f38b
to
757a178
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work @BenjaminCharmes, thanks!
Related to #710
Add more UI hints for unverified users (red dot next to their name in account settings), and the same hint for admins that a user has asked to be verified (this could also trigger an email send at some point).