-
-
Notifications
You must be signed in to change notification settings - Fork 22
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
Add nullable case #3
Conversation
Awesome! Should we add an array type as well? |
not needed as array is under
|
Co-authored-by: Arthur Fiorette <github@arthur.place>
Co-authored-by: Arthur Fiorette <github@arthur.place>
Co-authored-by: Arthur Fiorette <github@arthur.place>
For some reason i have in my memories that when i tried to use I no longer have this repo set up, can you just test it out for me? 😁 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm!
gracias |
thanks for the quick review. Let me know when it's published. Kinda blocking me to deploy some of my changes. |
@emroot just released |
thanks. works like a charm |
Json can be nullable especially in the case of CreateInput. Add a couple more cases to handle that.
@arthurfiorette