Page MenuHomePhabricator

Render Remarkup in emails
ClosedPublic

Authored by joshuaspence on Nov 18 2015, 8:52 PM.
Tags
None
Referenced Files
F15526682: D14511.diff
Mon, Apr 21, 10:16 PM
F15526368: D14511.id35208.diff
Mon, Apr 21, 8:40 PM
F15524455: D14511.id.diff
Mon, Apr 21, 8:06 AM
F15521593: D14511.id35208.diff
Sun, Apr 20, 3:19 PM
F15521592: D14511.id35102.diff
Sun, Apr 20, 3:19 PM
F15521591: D14511.id.diff
Sun, Apr 20, 3:19 PM
F15521590: D14511.diff
Sun, Apr 20, 3:19 PM
F15521426: D14511.diff
Sun, Apr 20, 2:00 PM
Subscribers

Details

Summary

Ref T992. I noticed that ManiphestTask mail doesn't render Remarkup properly (instead, it renders Remarkup literally). I think this is because the code calls addTextSection() rather than addRemarkupSection().

Test Plan

Created a new Maniphest Task and saw Remarkup in the generated self-email (inspect the email contents with ./bin/mail show-outbound). I didn't test the other affected applications.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

joshuaspence retitled this revision from to Render Remarkup in emails.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Nov 23 2015, 3:29 PM
This revision was automatically updated to reflect the committed changes.