Skip to content

SPTECH-236: Restore booking_customer_accommodation as deprecated - #128

Merged
kingaM merged 1 commit into
mainfrom
SPTECH-236/restore-booking-customer-accommodation-deprecated
Oct 5, 2026
Merged

kingaM merged 1 commit into
mainfrom
SPTECH-236/restore-booking-customer-accommodation-deprecated

Conversation

@kingaM

@kingaM kingaM commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

Description

Reverts #120 and marks booking_customer_accommodation as deprecated in favor of booking_pick_up_location. The API still returns it for pickup tours, so its removal broke Check24's generated client.

🤖 Generated with Claude Code

The API still returns it alongside booking_pick_up_location; removing it
broke partners generating strict clients from the spec.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The changes correctly restore the removed API contract while clearly marking its replacement.

Review effort: Balanced
Findings: None

What changed in this PR

Restores backward compatibility for clients receiving booking_customer_accommodation, while directing consumers to booking_pick_up_location.

Changes:

  • Restores the deprecated booking question and request property.
  • Documents the preferred replacement.
File Description
spec/​components/​schema/​option.yaml Restores and documents the deprecated question enum value.
spec/​components/​schema/​bookings.yaml Restores the deprecated accommodation answer schema.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@kingaM
kingaM marked this pull request as ready for review October 5, 2026 10:08
@kingaM
kingaM requested a review from a team as a code owner October 5, 2026 10:08
@clue-gyg clue-gyg removed the risk:low label Oct 5, 2026
@gyg-pr-tool
gyg-pr-tool Bot requested a review from FlaviaC-gyg October 5, 2026 10:08
@kingaM
kingaM merged commit 000a084 into main Oct 5, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants