Skip to content
This repository has been archived by the owner on Nov 7, 2019. It is now read-only.

output LocalDate json schema format as DATE #78

Merged
merged 1 commit into from May 23, 2016

Conversation

dtanner
Copy link
Contributor

@dtanner dtanner commented May 23, 2016

What I want to happen is for LocalDate types' generated swagger spec to output
"format" : "date" instead of "format" : "date-time".

This works locally, and it seems like LocalDates should always be formatted as a date in the spec. Need a test for this? If so, can you point me to an example existing test?

Otherwise, thoughts? Thanks.

@cowtowncoder
Copy link
Member

Looks good to me. There doesn't seem to be schema tests for this module, although I know some exist elsewhere; unfortunately I do not have link handy. jackson-databind for one does have some tests.

I'll just go ahead and merge this, makes sense to me.

@cowtowncoder cowtowncoder merged commit a41b19c into FasterXML:master May 23, 2016
@cowtowncoder cowtowncoder added this to the 2.5.0 milestone May 23, 2016
@dtanner
Copy link
Contributor Author

dtanner commented May 23, 2016

Thanks! Any rough estimates on release timing? I'm wondering if I'll need to implement a workaround before the release (is there a way to force the format on the current version?)

@cowtowncoder
Copy link
Member

@dtanner for 2.8 it will be a few weeks until first release candidate, so you may need a work-around.
What you can do is to just cut'n paste serializer, register it via SimpleModule for LocalDate.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants