Page MenuHomePhabricator

Fix missing link targets for "View Object" header buttons in HTML email

Authored by epriestley on Jun 18 2019, 2:25 PM.




I added "View Task" / "View Commit" buttons recently but the logic for generating URIs isn't quite right. Fix it up.

Test Plan
  • Commented on a task.
  • Used bin/mail show-outbound --id ... --dump-html > out.html to dump the HTML.
  • Previewed the HTML in a browser.
  • This time, actually clicked the button to go to the task.

Diff Detail

rP Phabricator
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley created this revision.Jun 18 2019, 2:25 PM
epriestley requested review of this revision.Jun 18 2019, 2:27 PM
amckinley accepted this revision.Jun 18 2019, 8:15 PM
This revision is now accepted and ready to land.Jun 18 2019, 8:15 PM