Page MenuHomePhabricator
Differential D19867

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

Authored by epriestley on Dec 11 2018, 12:15 AM.
Tags
None
Referenced Files
F15415504: D19867.diff
Thu, Mar 20, 6:17 AM
F15309006: D19867.id47445.diff
Thu, Mar 6, 6:48 AM
F15309005: D19867.id47444.diff
Thu, Mar 6, 6:48 AM
F15309004: D19867.id.diff
Thu, Mar 6, 6:48 AM
F15309000: D19867.diff
Thu, Mar 6, 6:48 AM
Unknown Object (File)
Feb 22 2025, 7:25 PM
Unknown Object (File)
Feb 9 2025, 6:45 AM
Unknown Object (File)
Feb 9 2025, 6:45 AM
View All Files
Subscribers
None

Details

Reviewers
amckinley
Maniphest Tasks
T13222: 2018 Week 48-51 Bonus Content
Commits
rP0e067213fbb1: Make viewing a user's profile page clear notifications about that user
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
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
src/
applications/
people/
controller/
4 lines

Diff 47445

View Options

src/applications/people/controller/PhabricatorPeopleProfileViewController.php

Loading...
Phabricator · Built by Phacility