When looking at our fetchBasicGasEstimates function, I found that we are manually managing cache time:
https://github.com/MetaMask/metamask-extension/blob/develop/ui/app/ducks/gas/gas.duck.js#L110-L134
Our fetchWithCache resource can do this, so we should move to fetchWithCache to make fetchBasicGasEstimates much more simple.
When looking at our
fetchBasicGasEstimatesfunction, I found that we are manually managing cache time:https://github.com/MetaMask/metamask-extension/blob/develop/ui/app/ducks/gas/gas.duck.js#L110-L134
Our
fetchWithCacheresource can do this, so we should move tofetchWithCacheto makefetchBasicGasEstimatesmuch more simple.