Skip to content

Season 2a server operations (part A) - #12

Merged
pietechsolution merged 14 commits into
mainfrom
season-2a-server
Nov 11, 2026
Merged

pietechsolution merged 14 commits into
mainfrom
season-2a-server

Conversation

@pietechsolution

Copy link
Copy Markdown
Contributor

Draft for CI only. Do not merge: the season 2a SDK changes merge and release together at the end of 2a, on the owner's yes. No version bump, no tag.

What was added (71 operations)

  • seasons: the 32 missing season methods (48 -> 80), named exactly as their operationIds.
  • holders (new resource, $seatlayer->holders): 11 methods, including listCustomers and holder ticket transfers.
  • orders (new resource, $seatlayer->orders): 9 methods for /v1/orders... and /v1/events/{eventKey}/orders....
  • events (existing resource): 14 event sales methods (buyer checkout, payment gateway, sales readiness/preflight/transition, booking fee, discount codes).
  • eventSeries (new resource, $seatlayer->eventSeries): 5 methods.
  • Transport: HttpClient::getText() for the three CSV exports (returns the raw text) and HttpClient::putObject() so an optional PUT body is sent as {}.
  • The 22 new header-replay operations send an Idempotency-Key (the caller's, or a generated one reused on retry); every other new mutation is sent once with no key. The two 204 deletes return void.
  • Tests: SeasonTest now asserts all 80 season calls (verb, path, escaping, replay classes) plus bodies, CSV text and the 204; new SalesOperationsTest covers the other 39 operations, CSV text, a CSV error mapping, the 204, and retry behaviour (replay retried with one key, unsupported sent once).
  • README (season count, replay list now 36, API surface table) and an Unreleased CHANGELOG entry.

Local checks

  • Route coverage check (seatmap's check-sdk-claims.mjs reader): php: 71/71 sent.
  • Test expectations cross-checked by script against the spec (verb, path, replay class for all 71).
  • PHP, Composer, phpstan and phpunit are not available on the machine this was written on, so phpstan (level 8) and phpunit on PHP 8.1-8.4 run in CI only.

…tle (2.16c)

createEventSeries takes fromEventKey: an event already set up becomes the first date under its own key and every other date is a copy of it; name and chartKey are then left out. Every date of a series carries the series' plain name. Matches seatmap slice 2.16c.
Adds releaseSeasonLapsedOffers (POST /v1/seasons/{key}/renewal-offers/release-lapsed, no body, header replay, 202 with the job). retrieveSeasonHostedRenewals pages with cursor and limit (1-500) and answers nextCursor, and listSeasonRenewalOffers pages the same way (1-100). createSeasonRenewalOffers answers 201 {offers} for 100 contracts or fewer and 202 {job, batchId} above that. The bring-across (createSeasonHolderImportFromSeason) answers 202 with a holder_import job above 100 holders. Renewal emails take up to 20,000 offer ids; above that, 422 too_many_offer_ids.
…e_abandoned (p2b)

The public API changes in p2b (b88ed794e..ee7df2921): Seasons::dateCancelVoidJobId names the date_cancel_void job a date cancel starts; Seasons::holderImportRowsPerCall sizes an import call by its dates (a call over it answers 422 invalid_rows with maxRows); updateSeason documents singleTicketDates.jobRefused; createEventSeries documents 409 create_abandoned.
…b audit)

updateEventSeries sends only the prices that change: each date keeps its
own price for every category or tier left out. A season's null transfer
or return cut-off means the club default from Settings > Ticket site, or
until the date starts when the club has none. Holder transfers and
returns through the secret key are not stopped by the club's customer
switches; cut-offs still apply.
…part C)

A date its season cancelled never opens again: moving its sales to
testable or live, or reopening it, answers 409 season_date_cancelled,
and an event series "sales on" leaves it closed as a failed item of the
job (season_date_cancelled; not in failedDates). A cancelled date can't
be moved, postponed, replaced or marked as taken place: 409
performance_cancelled.
@pietechsolution
pietechsolution marked this pull request as ready for review November 11, 2026 10:14
@pietechsolution
pietechsolution merged commit c69ce34 into main Nov 11, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant