Don't run "aphlict-listen" behavior if real-time notifications are disabled
ClosedPublic
Actions

Authored by epriestley on Jun 7 2014, 7:45 PM.

Tags

None

Referenced Files

	F19319111: D9418.id22492.diff
	Thu, Dec 25, 7:39 AM

	F19031483: D9418.diff
	Nov 25 2025, 1:22 AM

	F19013809: D9418.diff
	Nov 22 2025, 5:05 PM

	F18999611: D9418.diff
	Nov 20 2025, 9:41 PM

	F18992629: D9418.diff
	Nov 19 2025, 3:30 AM

	F18824704: D9418.id22491.diff
	Oct 23 2025, 5:27 PM

	F18759017: D9418.id22492.diff
	Oct 6 2025, 1:12 AM

	F18759016: D9418.id22491.diff
	Oct 6 2025, 1:12 AM

View All Files

Subscribers

epriestley

Details

Reviewers

joshuaspence

Maniphest Tasks

T5277: Aphlict client is started even if notifications are disabled

Commits

Restricted Diffusion Commit
rP96d5a9b4c2ef: Don't run "aphlict-listen" behavior if real-time notifications are disabled

Summary

Fixes T5277. We incorrectly ran this unconditionally.

Test Plan

Toggled setting on/off, verified behavior ran or did not run.

Diff Detail

Repository

rP Phabricator

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

epriestley updated this revision to Diff 22491.Jun 7 2014, 7:45 PM

epriestley retitled this revision from to Don't run "aphlict-listen" behavior if real-time notifications are disabled.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a reviewer: joshuaspence.

epriestley added a task: T5277: Aphlict client is started even if notifications are disabled.

Herald added a subscriber: epriestley. · View Herald TranscriptJun 7 2014, 7:45 PM

Harbormaster completed remote builds in B917: Diff 22491.Jun 7 2014, 7:46 PM

I was thinking it would be much harder than this.

This revision is now accepted and ready to land.Jun 7 2014, 7:50 PM

Closed by commit rP96d5a9b4c2ef (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

view/

page/

PhabricatorStandardPageView.php

79 lines

Diff 22492

View Options

src/view/page/PhabricatorStandardPageView.php

Don't run "aphlict-listen" behavior if real-time notifications are disabledClosedPublicActions