| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| | |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
not included in a block (#6388)
* transactions/pending - check nonce against the network and mark as dropped if not included in a block
* transactions/pending - unifiy "dropped" txs
* transactions/pending - test - fix for new expected behavior
* fix comment
* transactions/pending - clean up dropped event
* fix spelling
Co-Authored-By: frankiebee <frankie.diamond@gmail.com>
* nit fix
* test/tx-pending - clarify test description
|
| |
|
|
|
|
|
|
|
|
| |
* Set a default value for code in _determineTransactionCategory
* Adds e2e tests that fail when token txs without gas param are not properly handled.
* Adds unit tests for _determineTransactionCategory
* Base error throwing and simple gas setting in estimateTxGas on transactionCategory
|
| |
|
|
|
|
| |
* Adds a transactionCategory to txMeta for use in UI
* Update transaction controller and tx-gas-util documentation on new code param in multiple functions.
|
| | |
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Fixed Vivek's bug
Fixes #5850
What was happening:
It seems that his MetaMask had crashed while some new transactions had
been loading defaults. He probably had a network connectivity issue to
Infura (which we are working with Infura to address).
As a result of this network cutout, his three unapproved transactions
were not marked failed, and were not marked as `loadingDefaults =
false`, as their gas prices had not yet been estimated.
Normally this behavior is supposed to clean itself up when the
transaction controller starts up, via the
`TransactionController._onBootCleanUp()` function, but in this case,
during unlock, that function was unable to do its job because when it
requested the transaction list, the current network was in the `loading`
state, making it proceed as if there were no pending transactions.
To fix this, I am doing two things:
- Setting transactions to loadingDefaults = false in more catch blocks.
- Calling `onBootCleanUp()` when the network store's status changes, so
that it will re-trigger when loading completes.
* Fixed reference
* Fixed infinite loop bug
Was refreshing the tx list on every tx state change instead of just
network changes, creating an infinite loop.
* Add notes to tx updates to clarify logs
|
| |
|
|
|
|
|
|
| |
a %10 bump (#5786)
* transactions - autofill gasPrice for retry attempts with either the recomened gasprice or a %10 bump
* lint
|
| |
|
|
|
|
|
|
|
|
|
|
| |
* transactions - use safe-event-emitter over events
* tests - pass a platform object on init with a noop showTransactionNotification
* test - fix for tx-state-history-helper trying to reduce an empty array
* deps - safe-event-emitter
* lint
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Add beginning of test
* Resubmit approved transactions on new block
May fix #4343 and related issues, where an error could leave
transactions stranded in the approved state.
* Remove unused test
* Re-approve transactions when retrying approved
* Add retry approved test
* Include approved in pending tx count
* Fix getPendingTxs()
* Linted
* Only throw hash error in submitted state
* Only check submitted txs for block inclusion
* Fix test expectations
|
| | |
|
| | |
|
| |
|
|
|
|
| |
(#5375)"
This reverts commit ff67293a8ef61308d602d09f26b163b9b9ec90d3.
|
| | |
|
| |
|
|
|
|
|
|
|
| |
Transaction statuses were not being properly updated when:
- MetaMask was unlocked
- The network was changed
This PR fixes both of those. Fixes #5174
|
| |
|
|
| |
component. Add CancelTransaction modal
|
| | |
|
| |\
| |
| |
| | |
network-remove-provider-engine
|
| |\|
| |
| |
| | |
network-remove-provider-engine
|
| | | |
|
| | | |
|
| |\|
| |
| |
| | |
network-remove-provider-engine
|
| | | |
|
| | | |
|
| | | |
|
| |\|
| |
| |
| | |
network-remove-provider-engine
|
| | |
| |
| |
| | |
more errors traces cleaned up
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| |\ \
| | |
| | |
| | | |
https://github.com/MetaMask/metamask-extension into transactions-use-new-block-tracker
|
| | |\ \
| | | |
| | | |
| | | | |
network-remove-provider-engine
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
* add getFilteredTxList from txController to getApi
* transactions - remove dead code (isNonceTaken)
|
| |/ / / |
|
| |/ / |
|
| | | |
|
| |/ |
|
| | |
|
| |
|
|
|
|
|
|
|
|
| |
Nonce tracker is not fully documented yet.
Have not yet touched:
- tx-state-manager
- tx-state-history-helper
- util
- tx-gas-utils
- pending-tx-tracker
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
|
|
relevant files into it
|
