Page MenuHomeDevCentral

Simplify PhabricatorNotification and allow to use notifications:payload
ClosedPublic

Authored by dereckson on Aug 24 2016, 02:02.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Mar 27, 21:47
Unknown Object (File)
Wed, Mar 27, 21:17
Unknown Object (File)
Wed, Mar 27, 20:16
Unknown Object (File)
Fri, Mar 22, 15:51
Unknown Object (File)
Fri, Mar 22, 13:25
Unknown Object (File)
Fri, Mar 22, 10:59
Unknown Object (File)
Mon, Mar 18, 23:09
Unknown Object (File)
Mon, Mar 18, 23:09
Subscribers

Details

Summary

With rNOTIF67730e913d04, Phabricator began to match other services.

We continue to simplify Phabricator classes and standardize constructors.

This change allows:

  • to use the notifications:payload command with Phabricator too
  • to use the same rawContent format (inline JSON) than other notifications
Test Plan

php artisan notifications:payload Phabricator storage/logs/PhabricatorPastePayload.json Nasqueron

Diff Detail

Repository
rNOTIF Notifications center
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

dereckson retitled this revision from to Allow to use CLI notifications:payload command with Phabricator.
dereckson updated this object.
dereckson edited the test plan for this revision. (Show Details)
dereckson added a reviewer: dereckson.
dereckson edited edge metadata.

More tests

dereckson retitled this revision from Allow to use CLI notifications:payload command with Phabricator to Simplify PhabricatorNotification and allow to use notifications:payload.Aug 24 2016, 03:00
dereckson updated this object.
dereckson edited edge metadata.
dereckson edited edge metadata.
This revision is now accepted and ready to land.Aug 24 2016, 03:02
dereckson edited edge metadata.

EOL at EOF

This revision was automatically updated to reflect the committed changes.