-
Notifications
You must be signed in to change notification settings - Fork 1.2k
androidpublisher: PaymentState should be a pointer type #727
Copy link
Copy link
Closed
Labels
type: feature request‘Nice-to-have’ improvement, new feature or different behavior or design.‘Nice-to-have’ improvement, new feature or different behavior or design.
Description
Working with the api, I notice that PendingState is not using a pointer, but the value 0 is a valid information to business:
| PaymentState int64 `json:"paymentState,omitempty"` |
I believe will be best to change it to a pointer like the PurchaseType (example).
I notice that because when I call the .MarshalJSON() the field disappeared.
If allowed, I can submit this PR.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
type: feature request‘Nice-to-have’ improvement, new feature or different behavior or design.‘Nice-to-have’ improvement, new feature or different behavior or design.