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.

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
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley created this revision.Dec 11 2018, 12:15 AM
epriestley requested review of this revision.Dec 11 2018, 12:17 AM
amckinley accepted this revision.Dec 11 2018, 12:26 AM
This revision is now accepted and ready to land.Dec 11 2018, 12:26 AM
This revision was automatically updated to reflect the committed changes.