Page MenuHomePhabricator

Split seen timestamp for alerts and messages
Closed, ResolvedPublic

Description

Right now it's one timestamp, which means that if you view your messages, then don't view your alerts, then reload the page, your alerts will all be marked as seen.

Event Timeline

Catrope created this task.Sep 2 2015, 11:29 PM
Catrope updated the task description. (Show Details)
Catrope raised the priority of this task from to Unbreak Now!.
Catrope assigned this task to Mooeypoo.
Catrope added a subscriber: Catrope.
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptSep 2 2015, 11:29 PM

T110731 would probably require a schema change to add a notification_seen_timestamp column or something. We can fix bug this by using two keys or something.

Change 235659 had a related patch set uploaded (by Mooeypoo):
Separate 'seen time' for alerts and messages

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

DannyH set Security to None.

Change 235659 merged by jenkins-bot:
Separate 'seen time' for alerts and messages

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

Checked in betalabs.

DannyH closed this task as Resolved.Sep 8 2015, 9:12 PM
DannyH added a subscriber: DannyH.