Skip to content

WPB-25932: Send meeting emails on create/update/link-refresh/delete - #5606

Open
blackheaven wants to merge 4 commits into
developfrom
gdifolco/WPB-25932-meetings-send-emails
Open

blackheaven wants to merge 4 commits into
developfrom
gdifolco/WPB-25932-meetings-send-emails

Conversation

@blackheaven

@blackheaven blackheaven commented Oct 19, 2026 •

Copy link
Copy Markdown
Contributor

https://wearezeta.atlassian.net/browse/WPB-25932

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d
  • Read and follow the PR guidelines

@blackheaven
blackheaven requested review from a team as code owners October 19, 2026 12:04
@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Oct 19, 2026
Base automatically changed from gdifolco/WPB-25930-wire-emails-operations to develop October 19, 2026 20:19
@blackheaven
blackheaven force-pushed the gdifolco/WPB-25932-meetings-send-emails branch from 16715ed to 5fe8101 Compare October 19, 2026 20:42
@blackheaven blackheaven changed the title [WPB-25932] Send meeting emails on create/update/link-refresh/delete WPB-25932: Send meeting emails on create/update/link-refresh/delete Oct 19, 2026
@blackheaven
blackheaven force-pushed the gdifolco/WPB-25932-meetings-send-emails branch from 5fe8101 to 9d6987b Compare October 19, 2026 21:02
Galley now sends meeting invitation, update and cancellation emails to
invited email addresses via Wire.EmailSubsystem when
settings.meetings.email is configured (scheduled meetings only; creator
locale; delivery failures logged, never surfaced):

- create/addInvitedEmails/replaceInvitedEmails (newly-added only) ->
  invitation emails
- updateMeeting -> update email per actually-changed stored field
- refreshMeetingLink -> 'Join link changed' update email
- deleteMeeting -> cancellation emails after the deletion preflight

Infrastructure: reusable meetingEmailSubsystemInterpreter and
errorBrigAPIAccess; MeetingsEmailConfig gains templateDir + branding;
galley MeetingEmailEnv wiring with fire-and-forget best-effort transport;
meetings cleanup worker satisfies the new constraints with error
interpreters. Sends are async best-effort (pushAsyncImpl idiom);
cancellation emails go out only after the fallible deletion preflight;
empty recipient lists skip brig/code-store lookups entirely.
@blackheaven
blackheaven force-pushed the gdifolco/WPB-25932-meetings-send-emails branch from 9d6987b to 6537276 Compare October 19, 2026 22:00
@blackheaven
blackheaven requested a balanced review from Copilot October 20, 2026 06:41

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@blackheaven
blackheaven force-pushed the gdifolco/WPB-25932-meetings-send-emails branch from c3152d9 to bcfef0f Compare October 20, 2026 23:42

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants