EstimateGas endpoint - nonce should be automatically set the current nonce for the account #1819
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
The DebugTransaction, EstimateGas, and Call endpoints should not depend on the nonce provided by the user. Instead, the nonce should be automatically set to the current value for the account.
I have left the optional parameter forceNonce in DecodeTxn because, in the future, certain endpoints might rely on the nonce provided by the user.
Changes include
Breaking changes
Please complete this section if any breaking changes have been made, otherwise delete it
Checklist
Testing
Manual tests
cur_nonce
also and then both of those transactions should be included in some block