search

franckverrot / terraform-provider-stripe

A Terraform Provider for Stripe
https://registry.terraform.io/providers/franckverrot/stripe/latest
Mozilla Public License 2.0
242 stars 51 forks source link

Use GetOkExists for price amounts #49

Open adamstegman opened 3 years ago

adamstegman commented 3 years ago

This allows for a 0 price to be set as a price amount in Stripe.

GetOk considers 0 as a null value for integers, because it's unclear whether it was intentionally set or unset. GetOkExists allows us to get whatever value was set, even if it's 0.

Fixes #42

Dragemil commented 3 years ago

Hi, that works for me!

Could you please check if you can add this to the next release?

adamstegman commented 3 years ago

@franckverrot what do you think of this fix?

fgoyer commented 2 years ago

This fix would be great. @franckverrot could we get this reviewed and merged?

dmaevac commented 2 years ago

I am also waiting on this