Page MenuHomePhabricator
Differential D15610

Upate notifications for newPage
ClosedPublic
Actions

Authored by chad on Apr 4 2016, 7:13 PM.
Tags
None
Referenced Files
F15469834: D15610.id37621.diff
Fri, Apr 4, 5:02 PM
F15461880: D15610.id37621.diff
Tue, Apr 1, 12:06 PM
F15445771: D15610.id37622.diff
Thu, Mar 27, 3:58 PM
F15445533: D15610.id37621.diff
Thu, Mar 27, 2:25 PM
F15413910: D15610.diff
Wed, Mar 19, 9:57 PM
F15381087: D15610.diff
Fri, Mar 14, 5:47 AM
Unknown Object (File)
Feb 16 2025, 6:36 AM
Unknown Object (File)
Feb 12 2025, 1:09 PM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Commits
rP23979a05aa95: Upate notifications for newPage
Summary

Single callsite, swap to newPage

Test Plan

Visit page, see same status message. Also remove device ready flag.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad updated this revision to Diff 37621.Apr 4 2016, 7:13 PM
chad retitled this revision from to Upate notifications for newPage.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
Herald added a subscriber: Korvin. · View Herald TranscriptApr 4 2016, 7:13 PM
epriestley accepted this revision.Apr 4 2016, 7:21 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Apr 4 2016, 7:21 PM
Closed by commit rP23979a05aa95: Upate notifications for newPage (authored by chad, committed by chad). · Explain WhyApr 4 2016, 7:26 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
notification/
controller/
15 lines

Diff 37622

View Options

src/applications/notification/controller/PhabricatorNotificationStatusController.php

Loading...
Phabricator · Built by Phacility