Skip to content

Add flat list and CSV report methods for event reports#155

Merged
mroloux merged 1 commit into
masterfrom
matti/event-repot-flat-list
Jul 8, 2026
Merged

Add flat list and CSV report methods for event reports#155
mroloux merged 1 commit into
masterfrom
matti/event-repot-flat-list

Conversation

@mroloux

@mroloux mroloux commented Jul 8, 2026

Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings July 8, 2026 12:09

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds support in the Seats.io Python client for retrieving an event report as a flat list and as a CSV string, along with tests to validate both response types and basic object mapping.

Changes:

  • Add flat_list(event_key) to fetch a flat list of EventObjectInfo from /reports/events/{key}.
  • Add flat_list_csv(event_key) to fetch a CSV export from /reports/events/{key}.csv and decode it as UTF-8.
  • Introduce new unit tests covering list ordering/basic properties and CSV content.

Reviewed changes

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

File Description
seatsio/reports/events/eventReports.py Adds two new event report retrieval methods (JSON flat list + CSV).
tests/reports/events/testEventReportsFlatList.py Adds tests validating the new flat list and CSV report methods.

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

@mroloux mroloux marked this pull request as ready for review July 8, 2026 12:13
@mroloux mroloux requested a review from a team as a code owner July 8, 2026 12:13
@mroloux mroloux requested review from bverbeken and schaloner and removed request for a team July 8, 2026 12:13
@mroloux mroloux merged commit f2ff5ac into master Jul 8, 2026
3 checks passed
@mroloux mroloux deleted the matti/event-repot-flat-list branch July 8, 2026 14:37
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.

3 participants