-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
Notifications don't disappear after action #5665
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
Comments
@yackx |
As mention the note will disappear if you click on it. I'm pretty sure there was an issue for this already, I couldn't find it :/ |
It was closed #4684 |
The referenced issues #4684 does not seem to match this one. If I click on "verify it here", I am lead to "Change email" screen. There, if I click on "Make primary", the warning/error message goes away. But that was just me guessing and clicking on a what seemed to be an unrelated button ("make primary" vs "verify" actions). I fail to see why that extra action is required after I've clicked the link in the email. That alone should verify the address (and make it primary) IMO. |
There are related to the notification, when the user fixes the problem that the notifications says, it doesn't disappear until it's clicked. |
Unfortunately, this is how our notification system works now. We are not using template-based notifications, but instead when an action is needed based on some db field, we create a notification object with the proper message. Then, the only way to make it disappear is to click on the link from the notification (which trigger an AJAX call to delete it). We are not going to change this behavior soon, so I'm going to close this issue for now. We can reopen it if we want to refactor our notification system. |
Details
A persistent error message appears on every page and claims my email address is not verified, although it is.
I have a single email address associated with my account, and it has been verified. Actually, after verification, I even saw both success and error messages.
Expected Result
No error message displayed
Actual Result
Incorrect error message:
The text was updated successfully, but these errors were encountered: