HomePhabricator

Fix Phriction document linking in mail bodies

Tags
None
Referenced Files
F1238267: Screen Shot 2016-04-20 at 6.06.04 AM.png
Apr 20 2016, 1:55 PM
F1238283: Screen Shot 2016-04-20 at 6.05.54 AM.png
Apr 20 2016, 1:55 PM
F1238266: Screen Shot 2016-04-20 at 6.06.08 AM.png
Apr 20 2016, 1:55 PM
Subscribers
None

Description

Fix Phriction document linking in mail bodies

Summary:
Fixes T10840. When rendering mail, this rule wasn't falling through in quite the right way.

Also adjust where the rules are for this so the special styles show up in Maniphest, etc.

Test Plan:
Made this comment:

Screen Shot 2016-04-20 at 6.06.08 AM.png (330×579 px, 22 KB)

Which produced this HTML:

Screen Shot 2016-04-20 at 6.06.04 AM.png (228×614 px, 25 KB)

...and sent this mail:

Screen Shot 2016-04-20 at 6.05.54 AM.png (772×1 px, 93 KB)

Reviewers: hach-que, chad

Reviewed By: chad

Maniphest Tasks: T10840

Differential Revision: https://secure.phabricator.com/D15767

Details

Provenance
epriestleyAuthored on Apr 20 2016, 1:00 PM
epriestleyPushed on Apr 20 2016, 1:55 PM
Reviewer
chad
Differential Revision
D15767: Fix Phriction document linking in mail bodies
Parents
rPb9cf9e6f0db9: Fix an issue with PHID/handle management in push logs
Branches
Unknown
Tags
Unknown
Tasks
T10840: Phriction documents don't link correctly in mail
Build Status
Buildable 11860
Build 14888: Run Core Tests