Repository navigation
fix(mail): send the Reply-To header - #86
Merged
Merged
Conversation
POST /sequences/:id/emails/:emailId/test (and the MCP tool send_sequence_email_test) renders the email exactly like a real send, through the delivery source the send would use, with a [Test] subject. It defaults to the signed-in user, takes up to five addresses, skips suppressed ones and is capped at 50 recipients per team per hour. Tracking tokens carry a contact id that matches nobody and the unsubscribe link uses a test token whose page explains it was a test. The email records lastTestSentAt. The broadcast editor and the full-screen email editor get a Send test dialog. The per-recipient render pipeline moves to mail/campaign.ts so real and test sends share it. Refs #34, #79
Every send now carries a text/plain alternative generated from the final
HTML (links as "text (url)", bulleted lists, no images, styles or
preheader), or the caller's own text for transactional sends (new
optional `text`, stored on the row).
Marketing emails link a hosted web version from the managed footer, also
available as the {{ view_in_browser_url }} merge tag. GET /view/:token
serves it for the signed recipient with their merge data and no
tracking, as a sandboxed document. The editor default no longer starts
with a plain "View in browser" line that linked nowhere.
Refs #35, #79
sendMail had no replyTo, so a transactional send's stored Reply-To (and the organization grant's) never reached the recipient. It is now sent, and team ESPs get an optional reply-to (REST, MCP, the ESP dialog) used for broadcasts, sequences, tests and transactional sends without their own. Organization grants fall back to their ESP's reply-to. Refs #47, #79
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #47. Part of #79.
sendMailhad noreplyTo, so a transactional send's stored Reply-To (and an organization grant's) never reached the recipient.sendMailsendsreplyTo; the transactional worker passes the row's value.Testing
replyTo, else the ESP's; clearing it removes the header.