-
Notifications
You must be signed in to change notification settings - Fork 117
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
@JsonFormat
overriden features don't apply when there are no other options while deserializing ZonedDateTime
#196
Milestone
Comments
MaciejDobrowolski
changed the title
@JsonFormat overriden features don't apply when there are no other options
@JsonFormat overriden features don't apply when there are no other options while deserializng ZonedDateTime
Jan 7, 2021
MaciejDobrowolski
added a commit
to MaciejDobrowolski/jackson-modules-java8
that referenced
this issue
Jan 7, 2021
…eatures override if there were no other options defined on it
And this is with 2.12.0 or an older version? |
MaciejDobrowolski
added a commit
to MaciejDobrowolski/jackson-modules-java8
that referenced
this issue
Jan 8, 2021
…eatures override if there were no other options defined on it
This is for 2.12, but I believe it applies to all the previous versions as well |
Ok thanks, sounds good. Just wanted to know if it was a regression (new thing in 2.12) or was already there; makes sense it was already there. |
cowtowncoder
pushed a commit
that referenced
this issue
Jan 8, 2021
…verride if there were no other options defined on it (#198)
cowtowncoder
changed the title
@JsonFormat overriden features don't apply when there are no other options while deserializng ZonedDateTime
Jan 8, 2021
@JsonFormat
overriden features don't apply when there are no other options while deserializing ZonedDateTime
cowtowncoder
added a commit
that referenced
this issue
Jan 8, 2021
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
After deserializing a following class:
value
field should preserve timezone from the source - but it doesn't. Following test fails:The text was updated successfully, but these errors were encountered: