* Stop GasFeeController polling when pop closes
* Stop estimate gas polling on window unload
* lint + comments
* Improve client closed logic
* lint
* Add back _beforeUnload on unmount in gas-modal-page-container
* Add full check and call onClientClosed method for notifcation environment
* Add gas pollingToken tracking to appStateController and use to disconnect polling for each environment type
* remove unused method
* move controller manipulation logic from background.js to metamask-controller, disaggregate methods
* add beforeunload handling to reset gas polling tokens from root of send page
* cleanup, lint and address feedback
* clear appState gasPollingTokens when all instances of all env types are closed, fix pollingTokenType arg from onEnvironmentTypeClosed call in metamask-controller
* mock new methods to fix tests
* final bit of cleanup + comments
Co-authored-by: Dan Miller <danjm.com@gmail.com>
* Stop GasFeeController polling when pop closes
* Stop estimate gas polling on window unload
* lint + comments
* Improve client closed logic
* lint
* Add back _beforeUnload on unmount in gas-modal-page-container
* Add full check and call onClientClosed method for notifcation environment
* Add gas pollingToken tracking to appStateController and use to disconnect polling for each environment type
* remove unused method
* move controller manipulation logic from background.js to metamask-controller, disaggregate methods
* add beforeunload handling to reset gas polling tokens from root of send page
* cleanup, lint and address feedback
* clear appState gasPollingTokens when all instances of all env types are closed, fix pollingTokenType arg from onEnvironmentTypeClosed call in metamask-controller
* mock new methods to fix tests
* final bit of cleanup + comments
Co-authored-by: Dan Miller <danjm.com@gmail.com>
* Ensure that gas fee/prices params are not updated while in send edit stage/mode
* Allow editing of txparams gas price when not on eip1559 network, in send flow
wip
Documentation improvements for send slice support of EIP1559
Remove console.log in send duck
Property lookup safety improvement in selectors/confirm-transaction
Add code accidentally removed in rebase
Update addTxGasDefaults and _getDefaultGasFees to work with new estimate types, and ensure we correctly handle gas price estimates when on EIP1559 networks (#11615)
* Fix typo
Remove console.log in send duck
* Update addTxGasDefaults and _getDefaultGasFees to work correctly with all new gas fee estimate types
* Don't show gas timing support when not on eip1559 compatible network
* Hide gas timing component on transaction screen when on a non-1559 network
* Improve comments, tests and edge case handling
* Ensure eip1559 fees are applied and updated correctly when eip1559 estimate api fails
* Lint fix
Co-authored-by: Brad Decker <git@braddecker.dev>
Remove console.log
Handle possible gasEstimateType undefined
Remove unnecessary nonce field position change in confirm-page-container-content__details
* Ensure simple send gas estimation happens after the recipient is identified
* Update ui/ducks/send/send.test.js
Co-authored-by: Alex Donesky <adonesky@gmail.com>
* Improve test name
* Lint fix
Co-authored-by: Alex Donesky <adonesky@gmail.com>
* Ensure simple send gas estimation happens after the recipient is identified
* Update ui/ducks/send/send.test.js
Co-authored-by: Alex Donesky <adonesky@gmail.com>
* Improve test name
* Lint fix
Co-authored-by: Alex Donesky <adonesky@gmail.com>
* Ensure that eth_estimateGas is called to estimate gas limit for simple sends on custom networks
* getIsNonStandardEthChain returns false when in test
* Add comment explaining gas limit buffer multipliers in estimateGasLimitForSend
* Ensure that eth_estimateGas is called to estimate gas limit for simple sends on custom networks
* getIsNonStandardEthChain returns false when in test
* Add comment explaining gas limit buffer multipliers in estimateGasLimitForSend
The `SET_CURRENT_FIAT` action has been removed. It has been replaced
by a call to `forceUpdateMetamaskState`. The only purpose of this
action was to eagerly update the current fiat currency settings before
the next state update. Forcing a state update instead is simpler and
safer.
The `setCurrentCurrency` function in the background has been updated to
no longer return the state, now that it's no longer needed.
Alex Donesky
David Walsh
Dan J Miller
Brad Decker
Daniel
ryanml
Dan J Miller
Mark Stacey