Skip to content

fix: include metadata in the BOOKING_CANCELLED webhook payload#27784

Closed
supalarry wants to merge 2 commits intomainfrom
lauris/cal-7187-fix-metadata-not-sent-in-booking-cancelled-event
Closed

fix: include metadata in the BOOKING_CANCELLED webhook payload#27784
supalarry wants to merge 2 commits intomainfrom
lauris/cal-7187-fix-metadata-not-sent-in-booking-cancelled-event

Conversation

@supalarry
Copy link
Contributor

@supalarry supalarry commented Feb 9, 2026

Fixes #27783


Summary by cubic

Include booking metadata in the BOOKING_CANCELLED webhook payload so consumers receive custom booking fields. Metadata is validated and added only when present and valid; otherwise omitted.

  • Bug Fixes
    • Safely parse booking.metadata with bookingMetadataSchema and include it in the payload when valid.
    • Update tests to assert metadata is forwarded on cancellation.
    • Adjust test helper to conditionally expect metadata instead of always null.

Written for commit 3f4ba20. Summary will update on new commits.

@supalarry supalarry requested a review from a team as a code owner February 9, 2026 10:26
@linear
Copy link

linear bot commented Feb 9, 2026

@paragon-review
Copy link

paragon-review bot commented Feb 9, 2026

Paragon Review Unavailable

Hi @supalarry! To enable Paragon reviews on this repository, please register at https://home.polarity.cc

Once registered, connect your GitHub account and Paragon will automatically review your pull requests.

@graphite-app graphite-app bot added core area: core, team members only consumer labels Feb 9, 2026
@graphite-app graphite-app bot requested a review from a team February 9, 2026 10:26
@github-actions github-actions bot added the Medium priority Created by Linear-GitHub Sync label Feb 9, 2026
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 3 files

@supalarry supalarry enabled auto-merge (squash) February 9, 2026 10:29
@github-actions
Copy link
Contributor

github-actions bot commented Feb 9, 2026

E2E results are ready!

location,
subscriberUrl,
payload,
metadata,
Copy link
Contributor

Choose a reason for hiding this comment

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

Did you mean to include metadata in cancelled booking webhook payload or did you mean to just touch on tests?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

include metadata in cancelled booking webhook payload

@supalarry supalarry requested a review from hbjORbj February 9, 2026 14:46
Ryukemeister
Ryukemeister previously approved these changes Feb 16, 2026
Copy link
Contributor

@Ryukemeister Ryukemeister left a comment

Choose a reason for hiding this comment

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

LGTM!

@github-actions
Copy link
Contributor

This PR has been marked as stale due to inactivity. If you're still working on it or need any help, please let us know or update the PR to keep it active.

@github-actions github-actions bot added the Stale label Feb 25, 2026
@supalarry supalarry closed this Mar 2, 2026
auto-merge was automatically disabled March 2, 2026 13:50

Pull request was closed

@supalarry supalarry deleted the lauris/cal-7187-fix-metadata-not-sent-in-booking-cancelled-event branch March 2, 2026 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

consumer core area: core, team members only Medium priority Created by Linear-GitHub Sync ready-for-e2e size/S Stale

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: metadata not sent in booking cancelled event

3 participants