Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Allow JDK8 Optional to use contentConverter #97

Closed
sdavids13 opened this issue Jan 24, 2019 · 2 comments
Closed

Allow JDK8 Optional to use contentConverter #97

sdavids13 opened this issue Jan 24, 2019 · 2 comments
Labels
test-needed

Comments

@sdavids13
Copy link

The @JsonDeserialize and @JsonSerialize should allow a contentConverter to operate on the class inside the wrapped Java Optional class. Right now the only support is for using converter but you end up writing twice the amount of Converters if you need to support both Optional and Non-Optional bindings.
@cowtowncoder
Copy link
Member

@sdavids13 I am not sure I follow why twice the amount would be needed. Can you give an example?

@cowtowncoder
Copy link
Member

This may be valid, but I think I'd just need an example to show intended usage.

@richardsonwk richardsonwk mentioned this issue Dec 8, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
test-needed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@cowtowncoder @sdavids13