Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(client): do not strip date-time 00 seconds when getQueryParams() is called #211

Merged
merged 1 commit into from
Apr 29, 2024

Conversation

stainless-app[bot]
Copy link
Contributor

@stainless-app stainless-app bot commented Apr 29, 2024

The default string formatter for OffsetDateTime does not output seconds when they are set to 0 — i.e 2022-02-01T05:00Z instead of 2022-02-01T05:00:00Z. That can cause issues when a backend handler has strict parsing of the date-time value.

…is called

The default string formatter for OffsetDateTime does not output seconds when they are set to `0` — i.e `2022-02-01T05:00Z` instead of `2022-02-01T05:00:00Z`. That can cause issues when a backend handler has strict parsing of the date-time value.
@stainless-app stainless-app bot merged commit 888aef3 into generated Apr 29, 2024
@stainless-app stainless-app bot deleted the fix-client-do-not-strip-date-tim branch April 29, 2024 15:31
stainless-app bot added a commit that referenced this pull request Apr 29, 2024
…is called (#211)

The default string formatter for OffsetDateTime does not output seconds when they are set to `0` — i.e `2022-02-01T05:00Z` instead of `2022-02-01T05:00:00Z`. That can cause issues when a backend handler has strict parsing of the date-time value.
@stainless-app stainless-app bot mentioned this pull request Apr 29, 2024
stainless-bot pushed a commit that referenced this pull request May 1, 2024
…is called (#211)

The default string formatter for OffsetDateTime does not output seconds when they are set to `0` — i.e `2022-02-01T05:00Z` instead of `2022-02-01T05:00:00Z`. That can cause issues when a backend handler has strict parsing of the date-time value.
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.

1 participant