Skip to content

Fix #196 ZonedDateTime deserialization skipped @JsonFormat features o… #198

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

Conversation

MaciejDobrowolski
Copy link
Contributor

@MaciejDobrowolski MaciejDobrowolski commented Jan 8, 2021

…verride if there were no other options defined on it. Added a fixed English Locale to tests b/c they were not passing in non-English environment.

CLA has been signed

…eatures override if there were no other options defined on it
@cowtowncoder
Copy link
Member

Ok this makes sense. I am not sure why the guard statement was like it was but I can see why that prevented correct working.
I think I'll make some slight changes beyond this PR, but will get it merged first. Thanks!

@cowtowncoder cowtowncoder merged commit 3d20752 into FasterXML:2.12 Jan 8, 2021
@cowtowncoder cowtowncoder added this to the 2.12.1 milestone Jan 8, 2021
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.

2 participants