Skip to content

Add Itinerary schema to Tour#116

Merged
kingaM merged 2 commits into
mainfrom
add-itinerary-schema
Jun 10, 2026
Merged

Add Itinerary schema to Tour#116
kingaM merged 2 commits into
mainfrom
add-itinerary-schema

Conversation

@kingaM

@kingaM kingaM commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Adds a new itinerary.yaml defining Itinerary, ItineraryItem, ItineraryItemLocation, and ItineraryItemDuration
  • Wires a nullable itineraries array into the Tour schema
  • Mirrors the itinerary schema already merged in public-partner-api so it shows up on the public PPAPI docs site (https://code.getyourguide.com/partner-api-spec/)

Context

Per discussion in Slack, public schemas live in this repo and are what render on the docs site. The itinerary fields exist in public-partner-api's yaml but weren't surfaced publicly because they weren't ported here.

Test plan

  • Verify the rendered docs include the itineraries field on the Tour schema
  • Verify the Itinerary, ItineraryItem, ItineraryItemLocation, and ItineraryItemDuration schemas render correctly

🤖 Generated with Claude Code

Mirrors the itinerary schema from public-partner-api so structured
itineraries appear in the public PPAPI docs site. Adds a new
itinerary.yaml with Itinerary, ItineraryItem, ItineraryItemLocation,
and ItineraryItemDuration schemas, and wires a nullable itineraries
array into the Tour schema.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@kingaM kingaM requested a review from a team as a code owner June 10, 2026 09:33
@gyg-pr-tool gyg-pr-tool Bot requested a review from lrcarrera June 10, 2026 09:33
The unquoted description ended with "Day 1: Arrival in Cusco", and the
inner ": " was being parsed as a YAML mapping by SnakeYAML, failing the
openapi-generator step.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@kingaM kingaM merged commit 5f9aa8d into main Jun 10, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants