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

Notification campaign status update #3499

Closed
Xfolchf opened this issue May 25, 2018 · 1 comment · Fixed by #3509
Closed

Notification campaign status update #3499

Xfolchf opened this issue May 25, 2018 · 1 comment · Fixed by #3509
Assignees
Labels
type: bug Issues that describe a bug

Comments

@Xfolchf
Copy link

Xfolchf commented May 25, 2018

Describe the bug
When notifications update and mark them as "read", the notification bell keeps appearing in active state (yellow color). Only this state disappears and it appears inactive when referring to the page.

The color should be updated immediately by marking all messages as read

https://meta.decidim.org/processes/bug-report/f/210/proposals/12575

Screenshots
If applicable, add screenshots to help explain your problem.
image

Stacktrace
If applicable, add the error stacktrace to help explain your problem.

Extra data (please complete the following information):

  • Device: [e.g. iPhone6, Desktop]
  • Device OS: [e.g. iOS8.1, Windows 10]
  • Browser: [e.g. Chrome, Firefox, Safari]
  • Decidim Version: [e.g. 0.10] 0.12.x
  • Decidim installation: [e.g. MetaDecidim]

Additional context
Add any other context about the problem here.

@Xfolchf Xfolchf added the type: bug Issues that describe a bug label May 25, 2018
@mrcasals
Copy link
Contributor

Instead of an AJAX call, we can make a normal call so we get a full refresh of the page. It's much simpler to handle dev-wise.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Issues that describe a bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants