You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently, transactions need to be manually updated when completing (paying) an order.
Order completion and transitions applied to its transactions should belong to the same database transaction, as mentioned before:
What if the /order-moves request fails after transitions have been applied to transactions and we have an inconsistent state as a result?
I asked about a more integrated sequence on slack but maybe we didn’t spare enough time to think about this.
I don’t promote API complexity, but here we could help Order API user to feel safer and have less work to do to add transitions to every single transaction. Extra parameters might be worth it.
What about some utility parameter (like transactionTransitionName, set to a valid transition name) in Order moves API that could automatically add transition to all transactions given transactions transactionIdsInTransition, in a Postgres transaction.
It’s a feature so it’s not required to include such a change in this PR if you don’t have time. This could also make an issue and a PR on its own.
Currently, transactions need to be manually updated when completing (paying) an order.
Order completion and transitions applied to its transactions should belong to the same database transaction, as mentioned before:
What if the
/order-moves
request fails after transitions have been applied to transactions and we have an inconsistent state as a result?I asked about a more integrated sequence on slack but maybe we didn’t spare enough time to think about this.
I don’t promote API complexity, but here we could help Order API user to feel safer and have less work to do to add transitions to every single transaction. Extra parameters might be worth it.
What about some utility parameter (like
transactionTransitionName
, set to a valid transition name) in Order moves API that could automatically add transition to all transactions given transactionstransactionIdsInTransition
, in a Postgres transaction.It’s a feature so it’s not required to include such a change in this PR if you don’t have time. This could also make an issue and a PR on its own.
Originally posted by @abruere in #297
The text was updated successfully, but these errors were encountered: