Page MenuHomePhabricator

Flow: Notifications for "created a new topic" and "responded on" are not marked as read by visiting the topic
Closed, ResolvedPublic

Description

(In reply to Helder from bug 69879 comment 0)

Also, the red icon for new notifications doesn't change to gray until I open
Special:Notifications

Visiting the topic, from the Echo flyout, should mark the notification as read.


Version: unspecified
Severity: major

Details

Reference
bz69913

Event Timeline

bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 3:36 AM
bzimport set Reference to bz69913.
bzimport added a subscriber: Unknown Object (MLST).

See attachment 16263 for a screencast of this bug. (and in the second half, bug 69879)

Tracked on trello at https://trello.com/c/JdQdXpSE/

  • Bug 69900 has been marked as a duplicate of this bug. ***

bsitu wrote:

Seem to work for me. Not sure if the problematic notification was created during the code deploy which doesn't create target page records.

Still happening for new notifications. I visited one which pointed to
https://www.mediawiki.org/w/index.php?title=Topic:S0yyhkul3i2qj8oz&fromnotif=1
and the red [1] didn't disappear. I had to go to
https://www.mediawiki.org/wiki/Special:Notifications
for it to disappear.

Is becoming annoying to keep Flow pages on my watchlist.

bsitu wrote:

This is probably related to database replication lag, I will submit a patch to fix this. This is based on the observation that target_page records sometimes get created in metawiki and mediawikiwiki, while testwiki and test2wiki work fine. echo database is hosted in the main application database for metawiki and mediawikiwiki, new title may not made it to the slave database yet. echo database is hosted in an external database for testwiki/test2wiki, with an network request delay new title will probably made it to the slave

Change 156466 had a related patch set uploaded by Bsitu:
Fallback to master lookup for title when lookup in slave fails

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

Change 156466 merged by jenkins-bot:
Fallback to master lookup for title when lookup in slave fails

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

Appears to be fixed now. Tested on MediaWiki.