Skip to content

misc(customer): Expose error_details on API#383

Merged
vincent-pochet merged 1 commit intomainfrom
misc-customer-error-details
Mar 20, 2026
Merged

misc(customer): Expose error_details on API#383
vincent-pochet merged 1 commit intomainfrom
misc-customer-error-details

Conversation

@vincent-pochet
Copy link
Contributor

This PR is related to

It exposes the error_details on the customer related end-points

Copilot AI review requested due to automatic review settings March 19, 2026 09:56
Copy link

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 updates the Python client + test fixtures to surface error_details on customer-related API responses (aligned with the upstream lago-api / OpenAPI changes), and updates the error detail model to match the payload shape.

Changes:

  • Add error_details to the CustomerResponse model.
  • Update ErrorDetailResponse to use lago_id (instead of organization_id).
  • Update JSON fixtures (customer.json, invoice.json) to include error_details and apply minor formatting changes.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
tests/fixtures/invoice.json Adds an error_details array to the invoice fixture and minor JSON formatting tweaks.
tests/fixtures/customer.json Adds an error_details array to the customer fixture and minor JSON formatting tweaks.
lago_python_client/models/error_detail.py Renames organization_idlago_id on ErrorDetailResponse.
lago_python_client/models/customer.py Exposes error_details on CustomerResponse (typed as ErrorDetailsResponse).

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

@vincent-pochet vincent-pochet merged commit 070117c into main Mar 20, 2026
11 checks passed
@vincent-pochet vincent-pochet deleted the misc-customer-error-details branch March 20, 2026 09:04
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.

3 participants