Dan J Miller
6b08ca428b
Fix left border of the first button in the gas price button group of the gas customization modal ( #9406 )
4 years ago
Whymarrh Whitby
34b3953815
Use eth-json-rpc-middleware@5.0.3 ( #9405 )
4 years ago
dependabot[bot]
b1665dedc6
Bump node-fetch from 2.6.0 to 2.6.1 ( #9399 )
...
Bumps [node-fetch](https://github.com/bitinn/node-fetch ) from 2.6.0 to 2.6.1.
- [Release notes](https://github.com/bitinn/node-fetch/releases )
- [Changelog](https://github.com/node-fetch/node-fetch/blob/master/docs/CHANGELOG.md )
- [Commits](https://github.com/bitinn/node-fetch/compare/v2.6.0...v2.6.1 )
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
4 years ago
Whymarrh Whitby
cd7a3a687b
Update allow list for CLA, fix Dependabot ( #9400 )
...
This change updates the Dependabot name in the allow list of the CLA workflow.
4 years ago
Brad Decker
8b24f624dd
add segment implementation of metametrics ( #9382 )
...
Co-authored-by: Whymarrh Whitby <whymarrh.whitby@gmail.com>
Co-authored-by: Mark Stacey <markjstacey@gmail.com>
4 years ago
Mark Stacey
9391eac670
Update `@metamask/eth-token-tracker` from `v3.0.0` to `v3.0.1` ( #9398 )
...
`v3.0.1` of `@metamask/eth-token-tracker` fixes how token balances are
displayed when they are between 1 and 0.1. See here for more details:
https://github.com/MetaMask/eth-token-tracker/pull/47
4 years ago
Mark Stacey
ce66ddcf0d
Use `prettier` for JSON linting ( #9396 )
...
Instead of using `eslint-plugin-json` for linting JSON files,
`prettier` is now used. `prettier` is capable of detecting and
correcting more problems than `eslint-plugin-json` can, such as
indentation.
All JSON files have been run through `prettier`. The changes are all
superficial.
4 years ago
Mark Stacey
feab36710d
Remove unused connections route ( #9394 )
...
This route was used for the "Connections" page in Settings, which was
removed in #7004
4 years ago
Kevin Holder
95acee95b5
nodeify removeFromAddressBook ( #9395 )
...
Right now when editing an address in "Settings > Contact", the contact
is lost after saving. This is because the code awaits
`removeFromAddressBook()` before creating the new contact but
`removeFromAddressBook()` never resolves. This change fixes this bug.
4 years ago
Whymarrh Whitby
1757c63473
Always use null Infura project ID in testing ( #9393 )
4 years ago
Mark Stacey
359cdf744e
Include replacement of Infura Project ID in setup ( #9390 )
...
The setup steps listed in the README now explicitly include replacing
the `INFURA_PROJECT_ID` value with your own personal project ID.
4 years ago
Whymarrh Whitby
b8938c9c7a
Move ESLint ignore patterns into config ( #9392 )
4 years ago
Whymarrh Whitby
f415fd91b5
Move stylelint ignore patterns into config ( #9391 )
4 years ago
Mark Stacey
7ba2310726
Update CLA bot ( #9389 )
...
The `cla-signature-bot` has been updated to `v3.0.1`. This update
includes a bug fix for PRs that have over 100 comments.
4 years ago
Whymarrh Whitby
e2dedaacdb
Use Infura v3 API ( #9368 )
...
* Use eth-json-rpc-infura@5.0.0
* Use Infura v3 API
* Add example .metamaskrc file
4 years ago
Mark Stacey
b80ab53396
Add `radiogroup` `ButtonGroup` variant ( #9383 )
...
Add a prop to the `ButtonGroup` component to make it act as a group of
radio buttons, rather than a group of regular buttons. These commits
were written by @danjm - I just pulled them into this branch.
4 years ago
Whymarrh Whitby
24f4973afc
Delete MISSION.md ( #9387 )
...
This file has been moved to our internal docs.
4 years ago
Whymarrh Whitby
bb9eb827d6
Delete USER_AGREEMENT.md ( #9386 )
...
This is on the website now: https://metamask.io/terms.html
4 years ago
matteopey
ee36073eb5
Update italian translation ( #9293 )
4 years ago
Erik Marks
eef4ec981f
Ensure token amounts are in fixed notation ( #9381 )
...
* Ensure token amounts are in fixed notation
4 years ago
Erik Marks
b349a5c8b1
Convert ConfirmTokenTransactionBase to functional component ( #9373 )
...
* Convert ConfirmTokenTransactionBase to functional component
Co-authored-by: Mark Stacey <markjstacey@gmail.com>
4 years ago
Brad Decker
380ed78a34
add external links to github issue routing ( #9351 )
4 years ago
Erik Marks
f02ba2d561
Ensure the extension can be unlocked without network access ( #9295 )
...
move checkForLatestBlock to separate catch block
add test case
remove duplicate checkForLatestBlock call
4 years ago
Erik Marks
707c7193e0
Ensure tokenAmount is always a string ( #9374 )
...
* Ensure tokenAmount is always a string
* roundExponential: add docstring, remove unnecessary cast
4 years ago
Erik Marks
c5917aa445
roundExponential: add docstring, remove unnecessary cast
4 years ago
Erik Marks
d532cdc459
Ensure tokenAmount is always a string
4 years ago
Whymarrh Whitby
89eade97c5
Use bl@3.0.1, dedupe bl@1.x ( #9375 )
4 years ago
Erik Marks
23683c1630
Replace toNumber calls on BigNumbers with toString ( #9369 )
4 years ago
Whymarrh Whitby
253cd12bbb
Use yargs@7.1.1 ( #9364 )
...
This change updates the `yargs` dependency introduced by `gulp-cli` to the latest
`^7` version, addressing [`GHSA-p9pc-299p-vxgp`][1].
[1]:https://github.com/advisories/GHSA-p9pc-299p-vxgp
The `yarn audit` output:
```
┌───────────────┬──────────────────────────────────────────────────────────────┐
│ low │ Prototype Pollution │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Package │ yargs-parser │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Patched in │ >=13.1.2 <14.0.0 || >=15.0.1 <16.0.0 || >=18.1.2 │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Dependency of │ gulp │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Path │ gulp > gulp-cli > yargs > yargs-parser │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ More info │ https://www.npmjs.com/advisories/1500 │
└───────────────┴──────────────────────────────────────────────────────────────┘
```
4 years ago
Whymarrh Whitby
9c77f6add2
Use bl@1.2.3 ( #9349 )
4 years ago
Dan J Miller
ccb7eb3ab7
Fix useTokenTracker useEffect bug ( #9334 )
...
Co-authored-by: Mark Stacey <markjstacey@gmail.com>
Co-authored-by: Brad Decker <git@braddecker.dev>
4 years ago
Patryk Łucka
3d4f1f45bd
Add messages to Ledger connection process ( #9344 )
4 years ago
ricky
f281df3b36
Feature/hide seedphrase when restoring vault ( #9329 )
...
* hide seedphrase (by default) with option to show it
4 years ago
Whymarrh Whitby
449118dfc9
Remove unneeded globals from ESLint config ( #9342 )
4 years ago
Whymarrh Whitby
1e99a7b0c3
Migrate to scoped @metamask/jazzicon ( #9341 )
4 years ago
Whymarrh Whitby
9a711e695a
Fix lookAtDirection prop passed to Mascot ( #9343 )
4 years ago
Whymarrh Whitby
a6e93a6344
Use ganache-core/websocket@1.0.32 ( #9340 )
...
This change updates `websocket` to address a low-severity security advisory
with `yargs-parser`.
See https://www.npmjs.com/advisories/1500 for more information.
The `yarn audit` output:
```
┌───────────────┬──────────────────────────────────────────────────────────────┐
│ low │ Prototype Pollution │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Package │ yargs-parser │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Patched in │ >=13.1.2 <14.0.0 || >=15.0.1 <16.0.0 || >=18.1.2 │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Dependency of │ ganache-core │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Path │ ganache-core > websocket > gulp > gulp-cli > yargs > │
│ │ yargs-parser │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ More info │ https://www.npmjs.com/advisories/1500 │
└───────────────┴──────────────────────────────────────────────────────────────┘
```
4 years ago
Thomas Huang
f3a37dc1e2
Fix alignment of restore vault screen ( #9339 )
...
* Fix alignment of restore vault screen
Fixes #9324
Possibly missed when #9183 was merged. The issue is that the same classname is overridding the intended flex-direction.
46ba1ef100/ui/app/css/itcss/components/newui-sections.scss (L98-L101)
4 years ago
Thomas Huang
a3f9e6d37b
Show token symbol when token amount string is long ( #9333 )
...
* Show token symbol when token amount string is long
Fixes #9318
This essentially separates the token cell list item title/token amount and the token symbol.
I do request advice/recomendations on how to better handle the new `subTtitle` which I set the token symbol to and the `subtitle` https://github.com/MetaMask/metamask-extension/blob/develop/ui/app/components/app/asset-list-item/asset-list-item.js#L99 .
* Individually test token balance and token symbol and their associated css element
* Feedback commit
* Classname and css overflow for title with string.
4 years ago
Whymarrh Whitby
72313f011d
Use derequire@2.1.1 ( #9332 )
...
This change updates `derequire` to address a low-severity security advisory
with `yargs-parser`.
See https://www.npmjs.com/advisories/1500 for more information.
The `yarn audit` output:
```
┌───────────────┬──────────────────────────────────────────────────────────────┐
│ low │ Prototype Pollution │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Package │ yargs-parser │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Patched in │ >=13.1.2 <14.0.0 || >=15.0.1 <16.0.0 || >=18.1.2 │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Dependency of │ browserify-derequire │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Path │ browserify-derequire > derequire > yargs > yargs-parser │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ More info │ https://www.npmjs.com/advisories/1500 │
└───────────────┴──────────────────────────────────────────────────────────────┘
```
4 years ago
Whymarrh Whitby
ba9af7d7bf
Use react-inspector@4.0.1 ( #9331 )
...
This change addresses a low-severity security advisory for `yargs-parser`.
See https://www.npmjs.com/advisories/1500 for more information.
The `yarn audit` output:
```
┌───────────────┬──────────────────────────────────────────────────────────────┐
│ low │ Prototype Pollution │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Package │ yargs-parser │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Patched in │ >=13.1.2 <14.0.0 || >=15.0.1 <16.0.0 || >=18.1.2 │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Dependency of │ @storybook/addon-actions │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Path │ @storybook/addon-actions > react-inspector > │
│ │ storybook-chromatic > @chromaui/localtunnel > yargs > │
│ │ yargs-parser │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ More info │ https://www.npmjs.com/advisories/1500 │
└───────────────┴──────────────────────────────────────────────────────────────┘
```
4 years ago
Whymarrh Whitby
1024f49275
Use @metamask/eslint-config@3.2.0 ( #9330 )
4 years ago
Patryk Łucka
cd86d00cb7
Fix send warning dialog ( #9321 )
...
This change moves warning message from `add-recipient` component to `send-content`. Currently
whenever provided address is a valid eth address `send-content` is rendered instead of `add-recipient`
this is why warnings never popped up.
4 years ago
Thomas Huang
79d477009e
Use verifyPassword instead of submitPassword when exporting priv key ( #9288 )
...
* Use verifyPassword instead of submitPassword when exporting priv key
Fixes #9287 which was when submitPassword is called will fully clear the keyring state
https://github.com/MetaMask/KeyringController/blob/master/index.js#L155
ad823d0ac1/index.js (L562)
ad823d0ac1/index.js (L726)
Also, pass hideWarning action prop so it will clear the appState.warning if a correct password is never provided on componentWillUnmount
* Hide Warning on componentWillUnmount
* Update exportAccount tests to verifyPassword
* Update ui/app/store/actions.js
Co-authored-by: Mark Stacey <markjstacey@gmail.com>
4 years ago
Patryk Łucka
20b0e66cc9
fix ens error text on mainnet ( #9314 )
4 years ago
Thomas Huang
9e6ba089d9
Remove mountWithStore enzyme component wrapper ( #9309 )
...
* Remove mountWithStore enzyme component wrapper in favor for renderWithProvider testing-library/react for tests
Change dropdown component tests to testing-library/react
4 years ago
Mark Stacey
885bd13160
Add CLA Signature bot ( #9310 )
...
The CLA signature bot will check the authors of each PR to ensure they
have all signed the CLA. If any authors still need to sign the CLA, it
will leave a comment explaining how it can be signed, and will check
back upon each comment to see if it has been signed.
The bot used is `MetaMask/cla-signature-bot`, which is a fork of
`Roblox/cla-signature-bot`. The fork has a couple of improvements, and
it updated the PR comment text to be more appropriate for our usage.
Currently the only user on the `allowlist` is `dependabot`, but any public
members of the MetaMask organization will also be exempt from needing
to sign the CLA due to the `allow-organization-members` setting.
The signatures are stored in `cla.json` on the `cla-signatures` branch,
which is in this repository as a distinct root. We can consider moving
this to a separate repository in the future - this was just easier to
setup.
4 years ago
Erik Marks
867dd33c1e
Remove 'Contact us' exclamation mark ( #9307 )
4 years ago
Erik Marks
81a1d0a23d
Remove unused diagnostics property ( #9296 )
...
* Remove unused diagnostics property
4 years ago
Erik Marks
29550e3e65
Remove 'Contact us' exclamation mark
4 years ago