-
Notifications
You must be signed in to change notification settings - Fork 12
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
[patch] Add support for endpoint for listing customer's redeemables #268
[patch] Add support for endpoint for listing customer's redeemables #268
Conversation
…erface as it is not available
…abel List since it is not the same as Voucher type
… not Record. Add more fields to 'RedeemableContainerVoucher'
…tomerRedeemablesListResponse'
camapign_id?: string | ||
category_id?: string | ||
type: 'GIFT_VOUCHER' | 'DISCOUNT_VOUCHER' | 'LOYALTY_CARD' | ||
discount?: DiscountAmount | DiscountPercent | DiscountUnit | DiscountFixed |
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.
It looks like we do not cover "Unit Multiple" discount type in SDK. To make it a little more complex, in docs, we have two different types, "Unit" and "Unit Multiple" which have exclusively "units" attribute:
In the Voucherify domain, there is one DTO for both cases that has the "units" key. In SDK, I don't see the "units" key in the "DiscountUnit" type nor "DiscountUnitMultiple" defined.
- types names following CONTRIBUTING.md - add missing voucher.gift.effect - adding missing voucher.loyalty_card.next_expiration_date and voucher.loyalty_card.next_expiration_points - adding missing voucher.redemption.redeemed_points - voucher.active as optional field
Change type
Patch
Changes:
customers/{id}/redeemables
endpoint - https://docs.voucherify.io/reference/list-customer-redeemables