X Tutup
Skip to content

Allow none filter in chargefilterusage (default case)#353

Merged
vincent-pochet merged 2 commits intogetlago:mainfrom
harshitgupta412:patch-1
Jan 8, 2026
Merged

Allow none filter in chargefilterusage (default case)#353
vincent-pochet merged 2 commits intogetlago:mainfrom
harshitgupta412:patch-1

Conversation

@harshitgupta412
Copy link
Contributor

@harshitgupta412 harshitgupta412 commented Jan 6, 2026

when getting current usage, the current api returns None in value for Default case. ChargeFilterUsage BaseModel does not allow this and so the function client.customers().current_usage errors out with

ValidationError: 1 validation error for CustomerUsageResponse
charges_usage -> 0 -> filters -> 2 -> values
  none is not an allowed value (type=type_error.none.not_allowed)

@vincent-pochet
Copy link
Contributor

vincent-pochet commented Jan 8, 2026

Thank you @harshitgupta412 for this pull request.
I'm about to merge it and it will be part of the next release.

@vincent-pochet vincent-pochet merged commit 0ecf074 into getlago:main Jan 8, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vincent-pochet vincent-pochet vincent-pochet approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@harshitgupta412 @vincent-pochet
X Tutup