Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support submitting multiple transactions at once #194

Closed
AlicanC opened this issue Mar 20, 2022 · 4 comments
Closed

Support submitting multiple transactions at once #194

AlicanC opened this issue Mar 20, 2022 · 4 comments
Labels
feat Issue is a feature

Comments

@AlicanC
Copy link
Contributor

AlicanC commented Mar 20, 2022

When submitting multiple transactions at once, there's a risk that same coins get included in more than one txn, causing some txns to fail.

@AlicanC
Copy link
Contributor Author

AlicanC commented Mar 20, 2022

We could keep a list of pending coins in Wallet, and use that list for exclusion when calling coinsToSpend after the following is implemented: FuelLabs/fuel-core#185

@luizstacio
Copy link
Member

This would require some storage management to keep the queue safe from accidental termination, such as the exit process on node.js or closed browser?

@AlicanC
Copy link
Contributor Author

AlicanC commented Mar 24, 2022

I think we can regenerate all the data we need by querying pending transactions.

@Torres-ssf
Copy link
Contributor

Closing this as we already have multi call working

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat Issue is a feature
Projects
None yet
Development

No branches or pull requests

4 participants