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)
Fri, May 3, 12:35 AM
Unknown Object (File)
Mon, Apr 29, 3:42 PM
Unknown Object (File)
Thu, Apr 25, 11:44 AM
Unknown Object (File)
Wed, Apr 24, 11:41 PM
Unknown Object (File)
Mon, Apr 15, 8:18 PM
Unknown Object (File)
Thu, Apr 11, 12:06 PM
Unknown Object (File)
Thu, Apr 11, 7:58 AM
Unknown Object (File)
Mar 4 2024, 11:07 PM
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
Branch
conph-participant (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 13989
Build 18132: Run Core Tests
Build 18131: arc lint + arc unit

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.