Fujicracy / fuji-v2

Cross-chain money market aggregator
https://fuji-v2-frontend.vercel.app
15 stars 10 forks source link

Connext tx details #424

Closed brozorec closed 1 year ago

brozorec commented 1 year ago

@ferostar @NikolaiYurchenko I propose in this PR a new method sdk.getConnextTxDetails. The point is to stop using sdk.watchTxStatus for tracking all kind of transactions but to split the flow in 2, one flow for single-chain txns and one for cross-chain txns.

ferostabio commented 1 year ago

@ferostar @NikolaiYurchenko I propose in this PR a new method sdk.getConnextTxDetails. The point is to stop using sdk.watchTxStatus for tracking all kind of transactions but to split the flow in 2, one flow for single-chain txns and one for cross-chain txns.

And we'll poll each couple of seconds and update the UI until having results, right? Makes sense @brozorec . I'm making a couple of small but breaking changes, going to update the branch as soon as I finish with them.

ferostabio commented 1 year ago

@brozorec PR is already up to date with sdk/feat/error-handling, should we merge it?