Skip to content

WPB-26538: Add PUT /meetings/:domain/:id/invitations - #5285

Merged
blackheaven merged 4 commits into
developfrom
gdifolco/WPB-26538-wire-meetings-invitation-put
Jun 23, 2026
Merged

WPB-26538: Add PUT /meetings/:domain/:id/invitations#5285
blackheaven merged 4 commits into
developfrom
gdifolco/WPB-26538-wire-meetings-invitation-put

Conversation

@blackheaven

Copy link
Copy Markdown
Contributor

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

Checklist

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

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.

Pull request overview

Adds support for replacing the full invited-email list for a meeting via a new Galley public API endpoint, wiring it through the Meetings subsystem/store layers and covering it with unit + integration tests.

Changes:

  • Add PUT /meetings/:domain/:id/invitations to replace all invited emails on a meeting.
  • Extend MeetingsSubsystem and MeetingsStore with ReplaceInvitedEmails, including Postgres implementation and service handler wiring.
  • Add unit and integration test coverage for the new replace behavior (success, empty list, not found, unauthorized).

Reviewed changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
services/galley/src/Galley/API/Public/Meetings.hs Wires the new named API handler into Galley’s public meetings API.
services/galley/src/Galley/API/Meetings.hs Adds the replaceMeetingInvitation handler that calls the Meetings subsystem.
libs/wire-subsystems/test/unit/Wire/MockInterpreters/MeetingsStore.hs Extends the in-memory MeetingsStore interpreter to support replacing invited emails.
libs/wire-subsystems/test/unit/Wire/MeetingsSubsystem/InterpreterSpec.hs Adds unit tests for replaceInvitedEmails behavior and feature-flag handling.
libs/wire-subsystems/src/Wire/MeetingsSubsystem/Interpreter.hs Implements replaceInvitedEmailsImpl and hooks it into the subsystem interpreter.
libs/wire-subsystems/src/Wire/MeetingsSubsystem.hs Adds the ReplaceInvitedEmails effect constructor (with makeSem support).
libs/wire-subsystems/src/Wire/MeetingsStore/Postgres.hs Implements the Postgres update to replace invited_emails and updates updated_at.
libs/wire-subsystems/src/Wire/MeetingsStore.hs Adds the ReplaceInvitedEmails store effect constructor (with makeSem support).
libs/wire-api/src/Wire/API/Routes/Public/Galley/Meetings.hs Exposes the new PUT route in the public API type.
integration/test/Test/Meetings.hs Adds integration tests for replacing invitations (including empty list and authorization/not-found cases).
integration/test/API/Galley.hs Adds an integration test helper for the new PUT invitation endpoint.
changelog.d/1-api-changes/WPB-26538 Documents the new endpoint in the changelog.

:<|> Named
"replace-meeting-invitation"
( Summary "Replace the invited emails"
:> From 'V16

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.

Is V16 the expected target, or V17? (I'm always confused if this is inclusive or exclusive 😅 )

@supersven supersven 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.

LGTM 👍

@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Jun 23, 2026
@blackheaven
blackheaven force-pushed the gdifolco/WPB-26538-wire-meetings-invitation-put branch from 0e695bf to 8868c7f Compare June 23, 2026 18:29
@blackheaven
blackheaven merged commit 0e5e980 into develop Jun 23, 2026
10 checks passed
@blackheaven
blackheaven deleted the gdifolco/WPB-26538-wire-meetings-invitation-put branch June 23, 2026 23:18
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.

4 participants