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.

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

Jeison doesn't seem to have a way to specify optional fields. #1

Open
smcallis opened this issue Jun 21, 2020 · 4 comments
Open

Jeison doesn't seem to have a way to specify optional fields. #1

smcallis opened this issue Jun 21, 2020 · 4 comments

Comments

@smcallis
Copy link

This would be a really nice feature to have so we can support things like eg: JSON encoded proto buffers where fields with their default value simply aren't sent. Something like an :optional keyword with a default value for the field.

@smcallis
Copy link
Author

I think this likely amounts to honoring the :initform keyword when a given slot is unset after parsing the json, I'll try to write a pull request.

@smcallis
Copy link
Author

Submitted PR #2

@bscottm
Copy link

bscottm commented Feb 17, 2023

@smcallis: I think my pull request can satisfy this issue, either through your patch or through the (or null type) type constraint.

@smcallis
Copy link
Author

smcallis commented Feb 18, 2023 via email

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

No branches or pull requests

2 participants