You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
fix(email): SMTP scheduled-purchase notification goes to NOTIFY_EMAIL instead of the approver #1590
SMTPSender.SendScheduledPurchaseNotification (internal/email/smtp_sender.go, ~line 475) sends the "Purchase Scheduled" email, which includes the purchase's cancel link, to the shared NOTIFY_EMAIL address. #111 moved the other per-purchase emails to the approver or the plan's recipients. This one was missed, and it isn't one of #111's listed items.
Fix: send it to the same recipients as the other per-purchase emails (the approver or the plan's notification list), as the other senders do since #1275. Add a test that asserts the recipient.
Claimed by orchestrator run 2026-11-04 (platform-burndown). The plan has passed two adversarial review passes; implementation follows in a PR that closes this issue.
Fixed by #1671, merged as d868820. The SMTP sender now delivers the scheduled-purchase notice to the configured recipient (plus CC), like the SES sender, and returns ErrNoRecipient when none is set. Verified against a real Mailpit SMTP server: the notice reached the global notification address (main sent it to the notify inbox). Release note: deployments whose NOTIFY_EMAIL differs from the global notification address will now get these notices at the global address.
Activity
cristim commentedon Nov 4, 2026
Claimed by orchestrator run 2026-11-04 (platform-burndown). The plan has passed two adversarial review passes; implementation follows in a PR that closes this issue.
cristim commentedon Nov 4, 2026
Fixed by #1671, merged as d868820. The SMTP sender now delivers the scheduled-purchase notice to the configured recipient (plus CC), like the SES sender, and returns ErrNoRecipient when none is set. Verified against a real Mailpit SMTP server: the notice reached the global notification address (main sent it to the notify inbox). Release note: deployments whose NOTIFY_EMAIL differs from the global notification address will now get these notices at the global address.
fix(email): route SMTP scheduled-purchase notification to the recipie…