Page MenuHomePhabricator

Reply notifications are now sent with the ID corresponing to the message
ClosedPublic

Authored by cesar on Oct 14 2019, 7:34 PM.

Details

Test Plan

When replying to a ping, the email has the proper link

Diff Detail

Repository
R2 Ping
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

cesar requested review of this revision.Oct 14 2019, 7:34 PM
cesar created this revision.

@sebastian Hey! Can you review this please? Would like to go live with the latest set of changes on Ping next week.

cesar changed the visibility from "All Users" to "Public (No Login Required)".

Works for me!
The "Go to the website" button points to the following URL:

/ping/detail/{ping-id}

and opens the right page!

This revision is now accepted and ready to land.Nov 15 2019, 9:51 AM