Page MenuHomePhabricator

Return more transaction types in Conpherence notification menu
ClosedPublic

Authored by chad on Oct 3 2016, 8:03 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 15, 9:37 PM
Unknown Object (File)
Fri, Dec 13, 4:53 PM
Unknown Object (File)
Wed, Dec 11, 10:21 PM
Unknown Object (File)
Wed, Dec 11, 11:24 AM
Unknown Object (File)
Mon, Dec 9, 2:44 AM
Unknown Object (File)
Sun, Dec 8, 9:52 AM
Unknown Object (File)
Fri, Dec 6, 12:19 PM
Unknown Object (File)
Thu, Dec 5, 4:15 AM
Subscribers

Details

Summary

Unclear these are worth sending, but mostly seems useful. Returns getTitle for the transaction if it's not a message. Fixes T10683

Test Plan

Leave rooms, change names, add pictures.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad retitled this revision from to Return more transaction types in Conpherence notification menu.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
epriestley edited edge metadata.

Personally, it seems weird that join/leave mark the room unread. I wouldn't expect title or picture changes to, either -- really, anything but a new message.

Regardless, you should be able to get a text rendering of the action like this, I think:

$text = id(clone $action_transaction)
  ->setRenderingTarget(PhabricatorApplicationTransaction::TARGET_TEXT)
  ->getTitle();

Try that instead of strip_tags()?

This revision now requires changes to proceed.Oct 3 2016, 8:12 PM
chad edited edge metadata.
  • use a rendering target
epriestley edited edge metadata.
This revision is now accepted and ready to land.Oct 3 2016, 9:26 PM
This revision was automatically updated to reflect the committed changes.