Page MenuHomePhabricator

Make viewing a user's profile page clear notifications about that user
ClosedPublic

Authored by epriestley on Dec 11 2018, 12:15 AM.
Tags
None
Referenced Files
F15509491: D19867.diff
Wed, Apr 16, 12:50 PM
F15506654: D19867.id47445.diff
Tue, Apr 15, 1:05 PM
F15505742: D19867.diff
Tue, Apr 15, 5:38 AM
F15505572: D19867.id47444.diff
Tue, Apr 15, 4:08 AM
F15496223: D19867.id.diff
Sun, Apr 13, 6:19 AM
F15492115: D19867.diff
Sat, Apr 12, 11:15 AM
F15480853: D19867.id47444.diff
Tue, Apr 8, 4:57 PM
F15469652: D19867.id47444.diff
Fri, Apr 4, 3:14 PM
Subscribers
None

Details

Summary

Ref T13222. See PHI996. This is a general correctness improvement, but also allows you to clear test notifications by clicking on them (since their default destination is the recipient's profile page).

Test Plan

Clicked a test notification, got taken to my profile page, saw notification marked as read.

Diff Detail

Repository
rP Phabricator
Branch
notify6
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 21297
Build 28974: Run Core Tests
Build 28973: arc lint + arc unit