Page MenuHomePhabricator

Allow updating user's last seen time in action=echomarkread API
Closed, DeclinedPublic


Instead of making two API requests to echomarkread and echomarkseen at the same time, lets have echomarkread handle all the functionality. Probably should be a follow up to


Related Gerrit Patches:

Event Timeline

Legoktm created this task.Aug 25 2015, 5:28 PM
Legoktm raised the priority of this task from to Needs Triage.
Legoktm updated the task description. (Show Details)
Legoktm added a project: Notifications.
Legoktm added a subscriber: Legoktm.
Restricted Application added a project: Collaboration-Team-Triage. · View Herald TranscriptAug 25 2015, 5:28 PM
Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Change 233757 had a related patch set uploaded (by Legoktm):
[WIP] Add markseen support to API action=echomarkread

Catrope assigned this task to Legoktm.Aug 26 2015, 11:18 PM
Catrope triaged this task as Medium priority.
Catrope set Security to None.
Catrope added a subscriber: Catrope.

We should really change the way we calculate seen time in general, which would solve both the technical issue and the conceptual issue.

Removing Patch to Review, since the attached patch depends on an abandoned patch.

jmatazzoni closed this task as Declined.May 9 2016, 9:41 PM

We only needed this because we mark alerts as read when seen, and we're getting rid of that behavior, so we won't need this any more.