feat: query gas price from chain #741
Conversation
| FAST: 4, | ||
| AVERAGE: 2.5, | ||
| FAST: 1.2, | ||
| AVERAGE: 1.1, |
There was a problem hiding this comment.
Do you think this will be sufficient? (I get the reason because it's from the chain, which is more correct and reliable)
There was a problem hiding this comment.
Just tested, transactions don't go through with the suggested average value. It goes through with the fast option.
We need to find the right multiplier that's just enough to make a transaction go through, which we should use for the average option.
There was a problem hiding this comment.
Also, currently, users can send transactions even if the transaction fee is too little to send a transaction.
- We need to know the lowest, minimum value that can make transactions go through
- If the Network Fee is lower than that, then we should error out
There was a problem hiding this comment.
That's right,
sets the gasUsed data from simulating the current transaction to the minimum value.
I'll have to find the right weighting values through testing.
The handling of values less than the minimum will be covered in the migration task.
What type of PR is this?
What this PR does:
This PR implements direct gas price querying from the chain instead of using the indexer. The changes include updating the gas price calculation logic and simplifying the related interfaces.
Changes
getGasPricemethod inGnoProviderto query gas price directly from chainTechnical Details
Gas Price Query
Interface Changes
ITransactionGasRepositoryinterfaceGasPriceTierInfotype