Maciej Korycinski
Maciej Korycinski
@timuric Should we also take into account the case where we have email provided with upper case? Like [[email protected]](mailto:[email protected]), [[email protected]](mailto:[email protected]) and [email protected]?
Edit: Updated the scope
The main problem here is the migration tests that are checking the compatibility with `main` branch. It could not detect the changes, or provide a false positive result. The proper...
Looks reaaaally hot! 🔥 For POC, you could also show, where we will store the json.schema file. Do we want to serve it in the same way as for graphql...
`union AppEventRequestor = User | App` There is already common union for `User|App`. I think that you should use it, instead of adding a new one. `UserOrApp`.
There was an assumption, that `TransactionCancelationRequested` always cancels the whole transaction's amount, but you're right, we can make it better. In that case we could always send the `amount` set...
Hey @teddyondieki , Yes! Would be great! 😄 Just quickly test it, and the issue doesn't happen when using flat tax rates. It happens when using TaxApp or TaxPlugin. For...
Transaction API is a preview so we can improve the errors that we return. We should have separate errors like: - missing-app - app is misconfigured - app is disabled...
For draft orders we have a separate query: `draftOrders`. The `orders` query doesn't return the draft orders. So maybe we should have the same query for `expiredOrders`?
Currently, we don't allow for returning the error without psp-reference. We could reduce the validation, and allow to accept the the error event without pspReference. We have similar flow for...