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

Allowing explicitly putting null #52

Open
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@GRiMe2D
Copy link

commented Jan 18, 2018

Sometimes we need JSON null values, especially when API requests with PATCH method where null values mean "delete" the property

/**
* The style that wrap should ignore nil values or explicitly put JSON null value
*/
var wrapNilStyle: WrapNilStyle { get }

This comment has been minimized.

Copy link
@GRiMe2D

GRiMe2D Jan 18, 2018

Author

However, using bool value provides easier integration but using enum makes possible to add more options in future

@hebertialmeida

This comment has been minimized.

Copy link

commented Mar 15, 2018

That would be useful for me too, any idea if this will be merged or any workaround for that? I need to send null for a specific endpoint...

@GRiMe2D

This comment has been minimized.

Copy link
Author

commented Mar 15, 2018

@hebertialmeida
You actually can grab my fork for ability to send null

In Cartfile just replace Wrap to

github "aabbcc-double/Wrap" "a80b5f40454"

I'll tag it later

@hebertialmeida

This comment has been minimized.

Copy link

commented Mar 15, 2018

@GRiMe2D I actually ended up forking it and make a customization based on #36, it's less code change and works great...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.