Skip to content

fix: Use rfc3339 to decode date from text#3411

Merged
abonander merged 1 commit intolaunchbadge:mainfrom
pierre-wehbe:fix/chrono_fixedoffset
Sep 2, 2024
Merged

fix: Use rfc3339 to decode date from text#3411
abonander merged 1 commit intolaunchbadge:mainfrom
pierre-wehbe:fix/chrono_fixedoffset

Conversation

@pierre-wehbe
Copy link
Copy Markdown
Contributor

@pierre-wehbe pierre-wehbe commented Aug 7, 2024

Does your PR solve an issue?

#703 (comment)

Fix inspired by https://github.com/launchbadge/sqlx/pull/1618/files

@pierre-wehbe pierre-wehbe force-pushed the fix/chrono_fixedoffset branch 3 times, most recently from de9eab2 to ca7931a Compare August 7, 2024 07:05
@pierre-wehbe
Copy link
Copy Markdown
Contributor Author

@abonander @CommanderStorm Any chance this can be reviewed soon?

@CommanderStorm
Copy link
Copy Markdown
Contributor

CommanderStorm commented Aug 7, 2024

@pierre-wehbe You opened the PR 11h ago and and are already pinging?
I think you might be having this issue in prod => You know you can override the cargo dependency resolution to point to git

Copy link
Copy Markdown
Contributor

@CommanderStorm CommanderStorm left a comment

Choose a reason for hiding this comment

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

idk why you tagged me. I am not a maintainer, just a fellow contibutor ^^

Could you attach a testcase here to ensure that no regressions on this can happen in the future (and to ensure that this acutally fixes the issue linked, not doubting that it does):

https://github.com/05storm26/sqlx/blob/c5832d90267e085d420fc6f02de5f429f33e7f7d/tests/postgres/types.rs#L221-L277

@pierre-wehbe
Copy link
Copy Markdown
Contributor Author

@CommanderStorm Already overwrote the cargo deps. Wasn't sure how to get the PR visibility since no Reviewers were assigned.

@abonander
Copy link
Copy Markdown
Collaborator

It's generally just me, and I get to it when I get to it.

@abonander abonander merged commit fd80f99 into launchbadge:main Sep 2, 2024
jrasanen pushed a commit to jrasanen/sqlx that referenced this pull request Oct 14, 2024
jrasanen pushed a commit to jrasanen/sqlx that referenced this pull request Oct 14, 2024
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