Skip to content

Season object status can be overridden per season#164

Merged
mroloux merged 1 commit into
masterfrom
matti/season-status-overridde-by-season
Apr 8, 2026
Merged

Season object status can be overridden per season#164
mroloux merged 1 commit into
masterfrom
matti/season-status-overridde-by-season

Conversation

@mroloux
Copy link
Copy Markdown
Member

@mroloux mroloux commented Apr 8, 2026

No description provided.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds support for overriding/using “season object status” scoped to a specific season, enabling season-specific behavior when an event can be interpreted in a seasonal context.

Changes:

  • Extend EventsClient#override_season_object_status and EventsClient#use_season_object_status with an optional season: keyword argument and pass it through to the API when provided.
  • Add test coverage for the new season: parameter in both the dedicated methods and the batch status-change API.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
lib/seatsio/events.rb Adds optional season: parameter to season-status override/use calls and includes it in the request payload when present.
test/events/use_season_object_status_test.rb Adds coverage for use_season_object_status(..., season: ...).
test/events/override_season_object_status_test.rb Adds coverage for override_season_object_status(..., season: ...).
test/events/change_object_status_in_batch_test.rb Adds batch-API coverage for passing :season for OVERRIDE_SEASON_STATUS and USE_SEASON_STATUS.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mroloux mroloux merged commit 74d4e85 into master Apr 8, 2026
6 checks passed
@mroloux mroloux deleted the matti/season-status-overridde-by-season branch April 8, 2026 15:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants