Move message posting to PhabricatorNotificationClient
ClosedPublic
Actions

Authored by epriestley on Feb 16 2014, 9:08 PM.

Tags

None

Referenced Files

	F15465162: D8252.id.diff
	Wed, Apr 2, 3:20 PM

	F15464965: D8252.id19665.diff
	Wed, Apr 2, 1:13 PM

	F15463191: D8252.diff
	Tue, Apr 1, 9:14 PM

	F15462799: D8252.id19665.diff
	Tue, Apr 1, 5:53 PM

	F15434945: D8252.id19665.diff
	Tue, Mar 25, 6:09 AM

	F15425071: D8252.id19632.diff
	Sun, Mar 23, 2:22 AM

	F15409064: D8252.id19665.diff
	Wed, Mar 19, 3:07 AM

	F15388800: D8252.id19632.diff
	Sat, Mar 15, 4:24 AM

View All Files

Subscribers

aran

Details

Reviewers

btrahan

Maniphest Tasks

T4324: Change notification server so it only alerts appropriate clients

Commits

Restricted Diffusion Commit
rP1b8e129145d1: Move message posting to PhabricatorNotificationClient

Summary

Ref T4324. Centralize communication with the notification server. This will probably get less messy eventually.

Test Plan

Posted some messages.

Diff Detail

Repository

rP Phabricator

Branch

aphlict2

Lint

Lint Passed

Unit

No Test Coverage

Event Timeline

btrahan accepted this revision.Feb 17 2014, 5:49 PM

Closed by commit rP1b8e129145d1 (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

feed/

PhabricatorFeedStoryPublisher.php

10 lines

notification/

client/

PhabricatorNotificationClient.php

9 lines

Diff 19632

View Options

src/applications/feed/PhabricatorFeedStoryPublisher.php