Skip to content

Conversation

durm
Copy link
Contributor

@durm durm commented Jun 8, 2019

No description provided.

Copy link
Owner

@timols timols left a comment

Choose a reason for hiding this comment

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

Just one change and then good to merge

build.gradle Outdated
compile(group: 'org.slf4j', name: 'slf4j-api', version: '1.8.0-beta2')
compile(group: "com.fasterxml.jackson.core", name: "jackson-databind", version: "2.5.+")
compile(group: "commons-io", name: "commons-io", version: "2.4")
compile ("com.fasterxml.jackson.datatype:jackson-datatype-jsr310:2.5.1")
Copy link
Owner

Choose a reason for hiding this comment

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

could you change this to use the same form as the other deps in this file?

@cowtowncoder
Copy link

I am not sure how everything came together, but this is broken, as per test GitlabIssueDeserializationTest. Perhaps due to #355 or... something else.

But the issue is this: field dueDate is now of type java.util.Date; similarly setter and getter. But custom deserializer registered produces LocalDate.
What is needed is to make sure they all have LocalDate AND there is also serializer registered.

It would probably be cleaner to actually register JavaTimeModule and not use annotations: this would work for all LocalDate fields.

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