Maciej Korycinski

Results 56 comments of Maciej Korycinski

> `TransactionCreateInput` only accepts amountPendingRefund? > > ```graphql > """The input used in transactionCreate mutation""" > input TransactionCreateInput { > ... > """Amount that defines pending refund""" > amountPendingRefund: MoneyInput...

> > The one thing that I am worried about with this approach is: that we allow to modify the transaction (transactionUpdate mutation), which means that someone different could modify...

> If we have amount of non issued refunds we should also add non granted refunds so that we know if for example a refund was requested inside Adyen /...

> I'm not sure if we should do that - some payment methods might not allow transactions to be voided or refunded. To give you an example take a look...

> I was talking here from Dashboard perspective, not app developer. We could theoretically calculate this data in dashboard, but since we won't be doing that for non issued refund...

@witoszekdev > >Saleor will silently accept a call with the same details but no values will be updated for each next call. >Can we add some clue that the call...

> > pspReference is required. In case of missing psp reference, the webhook will be treat as undelivered. > > Does it mean that Saleor can make another request to...

> Will this change affect `transactionCreate` webhook? Will I still be able to create a transaction + an event with it? > > Another one: in the new dashboard views...

> Please change type on `transactionEvent.amount` to `PositiveDecimal` like we have on the rest of the transaction ;) PositiveDecimal is a something that we need to discuss. We need to...

@witoszekdev sorry, I forgot to mention you ☝️