Page MenuHomePhabricator

Minerva initial notification count does not include messages
Open, Needs TriagePublicBUG REPORT

Description

List of steps to reproduce (step by step, including full links if applicable):

  • Generate notifications e.g. php extensions/Echo/maintenance/generateSampleNotifications.php --user Admin --agent Jdlrobson --other Jdlrobson making sure you have both alerts AND notices
  • Load any page in Minerva as a logged in user
  • Click the notification icon
  • Close the overlay

What happens?:
The initially displayed count is not the same as the post-overlay count. This is because the initial count is only the number of alerts NOT the number of messages

What should have happened instead?:

Software version (if not a Wikimedia wiki), browser information, screenshots, other information, etc.:

Event Timeline

Restricted Application added subscribers: Masumrezarock100, Aklapper. · View Herald Transcript

Change 805485 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/extensions/Echo@master] Add badge that combines notices and alerts

https://gerrit.wikimedia.org/r/805485

Test wiki created on Patch demo by Jdlrobson using patch(es) linked to this task:
https://patchdemo.wmflabs.org/wikis/9225eb4c81/w/

Test wiki on Patch demo by Jdlrobson using patch(es) linked to this task was deleted:

https://patchdemo.wmflabs.org/wikis/9225eb4c81/w/