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

Set nonce as "next including unconfirmed transactions" #588

Closed
iamdefinitelyahuman opened this issue Jun 1, 2020 · 0 comments · Fixed by #597
Closed

Set nonce as "next including unconfirmed transactions" #588

iamdefinitelyahuman opened this issue Jun 1, 2020 · 0 comments · Fixed by #597
Assignees
Labels
enhancement New feature or request

Comments

@iamdefinitelyahuman
Copy link
Member

Overview

It should be possible to broadcast a tx and set the nonce as "the next available nonce, including all the pending transactions".

Specification

Nonce is already exposed since #488, but the value is currently given as an integer.

I have a few ideas but not sure which is the best. Open to comments or further suggestions.

accounts[0].transfer(accounts[1], nonce="next")
accounts[0].transfer(accounts[1], nonce="pending")
accounts[0].transfer(accoutns[1], nonce=accounts[0].pending_nonce)

Dependencies

Requires #587 before it will be useful.

@iamdefinitelyahuman iamdefinitelyahuman added the enhancement New feature or request label Jun 1, 2020
@iamdefinitelyahuman iamdefinitelyahuman added this to the Brownie 1.9.0 milestone Jun 1, 2020
@matnad matnad mentioned this issue Jun 3, 2020
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants