Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | transactions/deps - use broken out nonce-tracker module (#6555) | Frankie | 2019-05-21 | 2 | -162/+1 |
| | |||||
* | drop transactions who's nonce is lower then the known network nonce but were ↵ | Frankie | 2019-05-16 | 2 | -8/+46 |
| | | | | | | | | | | | | | | | | | | | | | | 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 (#6604) | Dan J Miller | 2019-05-15 | 2 | -13/+13 |
| | | | | | | | | | | * 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 | ||||
* | Check for unused function arguments (#6583) | Whymarrh Whitby | 2019-05-09 | 1 | -2/+2 |
| | | | | | | | | | | * eslint: Check for unused function arguments * eslint: Ignore unused '_' in argument list Also allow any number of '_' e.g., '__' or '___' which is to be used sparingly * Remove and rename unused arguments | ||||
* | Adds a transactionCategory to txMeta for use in UI (#6567) | Dan J Miller | 2019-05-09 | 2 | -7/+59 |
| | | | | | | * Adds a transactionCategory to txMeta for use in UI * Update transaction controller and tx-gas-util documentation on new code param in multiple functions. | ||||
* | Folder restructure (#6304) | Chi Kei Chan | 2019-03-22 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Remove ui/app/keychains/ * Remove ui/app/img/ (unused images) * Move conversion-util to helpers/utils/ * Move token-util to helpers/utils/ * Move /helpers/*.js inside /helpers/utils/ * Move util tests inside /helpers/utils/ * Renameand move confirm-transaction/util.js to helpers/utils/ * Move higher-order-components to helpers/higher-order-components/ * Move infura-conversion.json to helpers/constants/ * Move all utility functions to helpers/utils/ * Move pages directory to top-level * Move all constants to helpers/constants/ * Move metametrics inside helpers/ * Move app and root inside pages/ * Move routes inside helpers/ * Re-organize ducks/ * Move reducers to ducks/ * Move selectors inside selectors/ * Move test out of test folder * Move action, reducer, store inside store/ * Move ui components inside ui/ * Move UI components inside ui/ * Move connected components inside components/app/ * Move i18n-helper inside helpers/ * Fix unit tests * Fix unit test * Move pages components * Rename routes component * Move reducers to ducks/index * Fix bad path in unit test | ||||
* | Disallow loading as metamaskNetworkId (#5924) | Frankie | 2018-12-14 | 1 | -1/+3 |
| | | | | | | * transactions - throw an error if a transaction is generated while the network is loading * add tests for failing when netId is loading | ||||
* | Group transactions by nonce (#5886) | Alexander Tseung | 2018-12-10 | 2 | -2/+6 |
| | |||||
* | Adds createSpeedUpTransaction to txController | Dan Miller | 2018-12-04 | 1 | -0/+23 |
| | |||||
* | Fixed Vivek's bug (#5860) | Dan Finlay | 2018-11-30 | 1 | -2/+10 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * 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 | ||||
* | transactions/tx-gas-utils - add the acctual response for eth_getCode for ↵ | Frankie | 2018-11-30 | 1 | -0/+9 |
| | | | | NO_CONTRACT_ERROR's && add a debug object to simulationFailed (#5840) | ||||
* | Autofill gasPrice for retry attempts with either the recommended gasprice or ↵ | Frankie | 2018-11-27 | 1 | -0/+9 |
| | | | | | | | | a %10 bump (#5786) * transactions - autofill gasPrice for retry attempts with either the recomened gasprice or a %10 bump * lint | ||||
* | transactions - ensure err is defined when setting tx failed (#5801) | Frankie | 2018-11-27 | 1 | -4/+6 |
| | |||||
* | ESLint fixes (#5775) | Whymarrh Whitby | 2018-11-17 | 1 | -1/+1 |
| | | | | | | * eslint . --fix * Upgrade all ESLint warnings to errors | ||||
* | Tx controller now uses safe event emitter (#5769) | Frankie | 2018-11-17 | 3 | -3/+3 |
| | | | | | | | | | | | | * 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 | ||||
* | Resubmit approved transactions on new block (#5752) | Dan Finlay | 2018-11-15 | 3 | -2/+22 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | * 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 | ||||
* | tx-gas-util - set error message when invalidating tx based on tx data but no ↵ | kumavis | 2018-10-21 | 1 | -4/+5 |
| | | | | contract code | ||||
* | Merge branch 'develop' into develop | kumavis | 2018-10-21 | 2 | -1/+43 |
|\ | |||||
| * | Add txReceipt data to transaction details (#5513) | Alexander Tseung | 2018-10-16 | 2 | -1/+39 |
| | | |||||
| * | Ensure that new transactions added are using the selected address | Whymarrh Whitby | 2018-10-11 | 1 | -0/+4 |
| | | |||||
| * | Revert "transactions - add txReceipt to the txMeta body for confirmed txs ↵ | Whymarrh Whitby | 2018-10-11 | 1 | -23/+1 |
| | | | | | | | | | | | | (#5375)" This reverts commit ff67293a8ef61308d602d09f26b163b9b9ec90d3. | ||||
| * | transactions - add txReceipt to the txMeta body for confirmed txs (#5375) | Frankie | 2018-10-10 | 1 | -1/+23 |
| | | |||||
* | | tx-gas-utils - improve format + comments | kumavis | 2018-10-10 | 1 | -12/+18 |
| | | |||||
* | | ganache-core merged my PR, so I changed some comments to clarify that ↵ | Howard Braham | 2018-10-10 | 1 | -5/+6 |
| | | | | | | | | ganache-core v2.2.1 and below will return the non-standard '0x0' | ||||
* | | Bug Fix: #1789 and #4525 eth.getCode() with no contract | Howard Braham | 2018-10-10 | 1 | -7/+16 |
|/ | |||||
* | Fix updating of pending transactions | Dan Finlay | 2018-10-06 | 1 | -0/+1 |
| | | | | | | | | | Transaction statuses were not being properly updated when: - MetaMask was unlocked - The network was changed This PR fixes both of those. Fixes #5174 | ||||
* | Switch existing modals from using Notification to Modal. Remove Notification ↵ | Alexander Tseung | 2018-09-20 | 1 | -2/+3 |
| | | | | component. Add CancelTransaction modal | ||||
* | Merge pull request #5216 from MetaMask/tx-cancel | Frankie | 2018-09-11 | 2 | -2/+59 |
|\ | | | | | Add createCancelTransaction method | ||||
| * | Add createCancelTransaction method | Alexander Tseung | 2018-09-11 | 2 | -2/+59 |
| | | |||||
* | | Attach the RPC error value to txMeta | Whymarrh Whitby | 2018-09-09 | 1 | -0/+1 |
|/ | |||||
* | Merge branch 'develop' into network-remove-provider-engine-tests | Thomas Huang | 2018-08-02 | 1 | -7/+3 |
|\ | |||||
| * | tx-gas-utils - catch all errors from gas estimation | Jenny Pollack | 2018-07-25 | 1 | -7/+3 |
| | | |||||
* | | Merge branch 'develop' into network-remove-provider-engine | Thomas | 2018-07-26 | 2 | -13/+6 |
|\| | |||||
| * | Merge pull request #4669 from MetaMask/removeNonceTrackerFunction | Thomas Huang | 2018-07-13 | 1 | -13/+0 |
| |\ | | | | | | | Remove unsued nonce tracker function | ||||
| | * | Merge branch 'develop' into removeNonceTrackerFunction | Thomas | 2018-07-11 | 1 | -1/+1 |
| | |\ | |||||
| | * | | Remove unsued nonce tracker function | tmashuang | 2018-06-26 | 1 | -13/+0 |
| | | | | |||||
| * | | | Merge branch 'develop' of https://github.com/MetaMask/metamask-extension ↵ | frankiebee | 2018-07-11 | 1 | -1/+1 |
| |\ \ \ | | | |/ | | |/| | | | | | into i#3896 | ||||
| * | | | trandsactions - remove rejected transactions from history | frankiebee | 2018-06-26 | 1 | -0/+6 |
| | |/ | |/| | |||||
* | | | Merge branch 'develop' of github.com:MetaMask/metamask-extension into ↵ | kumavis | 2018-07-04 | 1 | -1/+1 |
|\ \ \ | | |/ | |/| | | | | network-remove-provider-engine | ||||
| * | | eslint --fix . | brunobar79 | 2018-07-03 | 1 | -1/+1 |
| |/ | |||||
* | | Merge branch 'develop' of github.com:MetaMask/metamask-extension into ↵ | kumavis | 2018-07-03 | 5 | -19/+27 |
|\| | | | | | | | network-remove-provider-engine | ||||
| * | lint - fix recipient-blacklist.js | kumavis | 2018-06-16 | 1 | -13/+13 |
| | | |||||
| * | blacklist - recipient blacklist as js for inline comments | kumavis | 2018-06-16 | 2 | -2/+2 |
| | | |||||
| * | Update recipient-blacklist-config.json | kumavis | 2018-06-16 | 1 | -0/+2 |
| | | |||||
| * | Add apparent phishing address to block list | Dan Finlay | 2018-06-15 | 1 | -0/+1 |
| | | | | | | In [this reddit post](https://www.reddit.com/r/Metamask/comments/8r3nsu/help_me_please_somebody_stole_my_ethers/) a user suggests they got some ether stolen after visiting IDEX. Their ether was sent to [this address](https://etherscan.io/address/0x9bcb0a9d99d815bb87ee3191b1399b1bcc46dc77), which is full of comments of people telling similar stories of being phished on IDEX. I think we can safely block this, and probably safe some people some money. | ||||
| * | controllers - transaction - prefer log over console | kumavis | 2018-06-13 | 1 | -2/+2 |
| | | |||||
| * | metamask - ensure all nonce locks are released | kumavis | 2018-06-13 | 3 | -5/+10 |
| | | |||||
| * | nonce-tracker - wrap nonce calculations in try-catch and release lock on error | kumavis | 2018-06-13 | 1 | -22/+28 |
| | | |||||
* | | nonce-tracker - wrap nonce calculations in try-catch and release lock on error | kumavis | 2018-06-13 | 1 | -22/+28 |
| | | |||||
* | | Merge branch 'develop' of github.com:MetaMask/metamask-extension into ↵ | kumavis | 2018-06-07 | 3 | -1/+43 |
|\| | | | | | | | network-remove-provider-engine | ||||
| * | remove generating blocked accounts and use a config file instead | Csaba Solya | 2018-05-31 | 3 | -18/+20 |
| | | |||||
| * | fix documentation | Csaba Solya | 2018-05-30 | 1 | -1/+1 |
| | | |||||
| * | adding tests for recipient blacklist checker | Csaba Solya | 2018-05-30 | 2 | -5/+5 |
| | | |||||
| * | adding transaction controller tests | Csaba Solya | 2018-05-30 | 1 | -1/+1 |
| | | |||||
| * | initial implementation | Csaba Solya | 2018-05-30 | 2 | -1/+41 |
| | | |||||
* | | Merge branch 'develop' of github.com:MetaMask/metamask-extension into ↵ | kumavis | 2018-06-07 | 2 | -10/+19 |
|\| | | | | | | | network-remove-provider-engine | ||||
| * | Merge branch 'master' of github.com:MetaMask/metamask-extension into ↵ | kumavis | 2018-05-30 | 2 | -10/+19 |
| |\ | | | | | | | | | | merge-master | ||||
| | * | cleanErrorStack moved to separate library module | Anton | 2018-05-29 | 1 | -12/+4 |
| | | | | | | | | | | | | more errors traces cleaned up | ||||
| | * | error message fix | Anton | 2018-05-29 | 1 | -2/+1 |
| | | | |||||
| | * | Clean up user rejection error message | Anton | 2018-05-28 | 1 | -3/+14 |
| | | | |||||
| | * | require log | frankiebee | 2018-05-01 | 1 | -0/+1 |
| | | | |||||
| | * | log emitter errors | frankiebee | 2018-04-30 | 1 | -6/+10 |
| | | | |||||
| | * | transactions - run event emitters outside context of _setTxStatus | frankiebee | 2018-04-30 | 1 | -7/+9 |
| | | | |||||
* | | | controllers - transactions - fix tx status update on boot | kumavis | 2018-05-29 | 2 | -2/+13 |
| | | | |||||
* | | | controllers - transactions - fix tx confirmation | kumavis | 2018-05-29 | 2 | -3/+3 |
| | | | |||||
* | | | controllers - transaction - pending-tx-tracker - lint fix | kumavis | 2018-05-29 | 1 | -1/+1 |
| | | | |||||
* | | | controllers - transactions - merge @frankiebee's work with mine | kumavis | 2018-05-29 | 3 | -96/+55 |
| | | | |||||
* | | | controllers - transactions - pending-tx-tracker - _getBlock - poll until ↵ | kumavis | 2018-05-26 | 1 | -1/+10 |
| | | | | | | | | | | | | block is truthy | ||||
* | | | tx-gas-utils - query for block without tx bodies | kumavis | 2018-05-26 | 1 | -2/+2 |
| | | | |||||
* | | | Merge branch 'network-remove-provider-engine' of ↵ | frankiebee | 2018-05-23 | 1 | -15/+0 |
|\ \ \ | | | | | | | | | | | | | https://github.com/MetaMask/metamask-extension into transactions-use-new-block-tracker | ||||
| * | | | Merge branch 'develop' of github.com:MetaMask/metamask-extension into ↵ | kumavis | 2018-05-23 | 1 | -15/+0 |
| |\| | | | | | | | | | | | | | | network-remove-provider-engine | ||||
| | * | | Reveal get filtered tx list (#4332) | Frankie | 2018-05-22 | 1 | -15/+0 |
| | | | | | | | | | | | | | | | | | | | | | | | | * add getFilteredTxList from txController to getApi * transactions - remove dead code (isNonceTaken) | ||||
* | | | | transactions - update pending-tx-tracker to use the new block tracker | frankiebee | 2018-05-23 | 1 | -7/+18 |
| | | | | |||||
* | | | | remove dependance on the even tx:confirmed | frankiebee | 2018-05-23 | 1 | -2/+6 |
|/ / / | |||||
* / / | network - create provider and block-tracker via json-rpc-engine | kumavis | 2018-05-17 | 2 | -14/+5 |
|/ / | |||||
* | | tx-state-history-helper - use more readable Date.now method | kumavis | 2018-05-17 | 1 | -1/+1 |
| | | |||||
* | | fix lint issues | Csaba Solya | 2018-05-10 | 2 | -2/+2 |
| | | |||||
* | | remove unnecessary lib | Csaba Solya | 2018-05-10 | 1 | -1/+0 |
| | | |||||
* | | revert unnecessary change in state manager | Csaba Solya | 2018-05-10 | 1 | -13/+7 |
| | | |||||
* | | add time stamps to transaction history log entries | Csaba Solya | 2018-05-10 | 2 | -16/+28 |
| | | |||||
* | | Fix documentation typo | bitpshr | 2018-05-08 | 1 | -1/+1 |
| | | |||||
* | | transactions - add a nonce check utility for ui use | frankiebee | 2018-05-02 | 1 | -0/+15 |
| | | |||||
* | | fix - getTxsByMetaData check if the key is in the object not if the value is ↵ | frankiebee | 2018-05-02 | 1 | -1/+1 |
|/ | | | | truthy | ||||
* | transactions - more docs and clean ups | frankiebee | 2018-04-26 | 6 | -55/+116 |
| | |||||
* | Cleaned up some typos and JSDocs in Transactions | Dan Finlay | 2018-04-24 | 3 | -38/+40 |
| | | | | | | | | | | 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 | ||||
* | transactions - fix refernces | frankiebee | 2018-04-20 | 2 | -1/+2 |
| | |||||
* | Merge branch 'master' into tx-controller-rewrite-v3 | frankiebee | 2018-04-20 | 1 | -0/+1 |
| | |||||
* | meta - transactions - docs yo! | frankiebee | 2018-04-20 | 8 | -73/+351 |
| | |||||
* | fix up - more docs | frankiebee | 2018-04-14 | 3 | -38/+77 |
| | |||||
* | meta - transactions - code clean up and jsDoc | frankiebee | 2018-04-14 | 1 | -41/+109 |
| | |||||
* | transactions - code cleanup | frankiebee | 2018-04-13 | 2 | -58/+59 |
| | |||||
* | transactions - lint fixes | frankiebee | 2018-04-11 | 3 | -12/+12 |
| | |||||
* | meta - transactions - create a transactions dir in controller and move ↵ | frankiebee | 2018-04-11 | 7 | -0/+1203 |
relevant files into it |