Open karolinatrzcionka opened 1 year ago
Some 'bigger' issues I have found so far:
1.Create segment - should there be a drop-down list and possibility to select ‘empty’ slots? Shouldn’t it be fixed and set to ‘static’ in Static and ‘auto-update’ in Dynamic? Because now you can select 'empty' field which causes API call to fail. Also when you switch to Dynamic we already have 'static' selected inside it at the beginning. It doesn't cause error, however, as in the code window it says 'auto update'.
Also, there is some bigger issue that causes the call to fail all the time - you cannot add customers in both cases. In static there are no fields to be filled with information whereas in dynamic filter object is lacking some information as well.
Metadata only limited to strings so far.
In orders API 'Download Export' is missing.
Problems while pasting customer source_id, in a lot of cases it adds additional spaces or weird signs. (videos on basecamp)
In update voucher API call in percentage discount, 'type' field is missing causing the whole call to fail (because of this missing field the 'value' is borrowed from the amount discount). All of other categories have this field so when you do amount, there is amount in the code. In unit, there is unit. But because of this issue percentage always shows amount and this is the error you get: "details": "It is not possible to change Voucher discount type".
Export gift card transactions - in the dashboard resource is shown as “-s”.
The issue with the App Id and App Token has not been fixed. They still disappear after switching between calls so you either have to refresh the page or paste the keys again.
In most of the Validations, Redemptions, and Stackable Discounts APIs within the order body there is another customer object called 'customer object object' same goes for the referrer - 'referrer object object'. This has to be checked again, think there are some mistakes there. In other calls from these categories (redemptions) there is - 'order object object'.
Not sure if its a mistake or not but in rollback stackable redemptions I have noticed 2x order object in 200 section, they are one below another.
@MaciekVoucheirfy @karolinatrzcionka I have addressed the above concerns as follows:
default
value, but I don't know if that what we want to do because promotion_tier
and promotion_stack
are are viable options. We could also go with an example value (as we do in the customer object - customer ID
)default
under the amount definition because it seem to me (please test this if you can to confirm) that this is required for the AMOUNT type discount. I have fixed this by defining it as an example value and it should not affect the other discount types.@janiksam regarding Validate Stackable Discounts issue, I think that setting one of the dropdown values as default should do the trick. The rest of the values (promo tier and stack) will still be there and the user will have the possibility to switch to one of them, right?
Just writing a note here from our conversation on slack so we can track progress. I've addressed the concerns listed in the above git hub bug report and my comments can be found here. And the updates in this pull request. We tested the login today with Maciek and it seems to be working fine except when we have metrics enabled (history). I have disabled the history to get rid of undesired behaviors. can share a summary of what we found during our sync today when the history is enabled (which enables a replay request button). I've also went through the endpoint comments left by Maciek and addressed all 18 except these 2: https://github.com/voucherifyio/voucherify-openapi/issues/84#issuecomment-1514778874 https://github.com/voucherifyio/voucherify-openapi/issues/283#issuecomment-1505291878
@karolinatrzcionka, regarding above comment for Validate Stackable Discounts. You are correct. I have updated the default value to be voucher
When the Validate Stackable Discount is used for the first time, the redeemables -> object is set to "voucher" but the object itself is not present in the API request. When user pastes the voucher id and tries to run the request the payload comes as invalid (see attachment)
User needs to change the object to anything and then back to voucher to get this working