Skip to content
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

Notifications don't disappear after action #5665

Closed
yackx opened this issue May 6, 2019 · 6 comments
Closed

Notifications don't disappear after action #5665

yackx opened this issue May 6, 2019 · 6 comments
Labels
Improvement Minor improvement to code Needed: design decision A core team decision is required

Comments

@yackx
Copy link

yackx commented May 6, 2019

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.

readthedocs-bug

Expected Result

No error message displayed

Actual Result

Incorrect error message:

Your primary email address is not verified. Please verify it here.

@dojutsu-user
Copy link
Member

@yackx
It will go away once you click on verify it here.

@stsewd
Copy link
Member

stsewd commented May 6, 2019

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 :/

@stsewd
Copy link
Member

stsewd commented May 6, 2019

It was closed #4684

@yackx
Copy link
Author

yackx commented May 7, 2019

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.

@stsewd
Copy link
Member

stsewd commented May 7, 2019

There are related to the notification, when the user fixes the problem that the notifications says, it doesn't disappear until it's clicked.

@stsewd stsewd changed the title Incorrect error msg - Your primary email address is not verified Notifications don't disappear after action May 7, 2019
@stsewd stsewd added Improvement Minor improvement to code Needed: design decision A core team decision is required labels May 7, 2019
@humitos
Copy link
Member

humitos commented May 23, 2019

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Improvement Minor improvement to code Needed: design decision A core team decision is required
Projects
None yet
Development

No branches or pull requests

4 participants