Skip to content

Conversation

@l-armstrong
Copy link
Contributor

We want to update the sdk to use the new API version. This PR also contains updates to some of the billing types reflected in the new API version.

We want to update the sdk to use the new API version. This PR also
contains updates to some of the billing types reflected in the new
API version.
@l-armstrong l-armstrong marked this pull request as ready for review November 4, 2025 19:37
@l-armstrong l-armstrong requested a review from a team as a code owner November 4, 2025 19:37
@l-armstrong l-armstrong marked this pull request as draft November 4, 2025 19:41
@l-armstrong l-armstrong marked this pull request as ready for review November 4, 2025 19:49
Copy link
Contributor

@paddycarver paddycarver left a comment

Choose a reason for hiding this comment

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

Mostly just tweaks around naming and omitempty. Pointers would be more appropriate in some cases, I think. And even when serializing, omitempty has no effect on non-pointer structs. :(

Copy link
Member

@mzhong9723 mzhong9723 left a comment

Choose a reason for hiding this comment

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

@paddycarver may have to give it an approval before merging is unblocked though!

@l-armstrong l-armstrong enabled auto-merge (squash) November 10, 2025 18:38
@l-armstrong l-armstrong disabled auto-merge November 10, 2025 18:39
@l-armstrong l-armstrong merged commit 2477281 into v2 Nov 10, 2025
7 checks passed
@l-armstrong l-armstrong deleted the lamone/bill-1381-update-clerk-sdk-go-to-use-new-api-version branch November 10, 2025 18:41
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.

6 participants