* cast chainId to hex for metrics
* Update ui/pages/settings/networks-tab/networks-form/networks-form.js
Co-authored-by: Ariella Vu <20778143+digiwand@users.noreply.github.com>
Co-authored-by: Ariella Vu <20778143+digiwand@users.noreply.github.com>
* Don't send errors to sentry if users have not opted-in to participate in metametrics
* Don't capture opt-out metrics
* Move the metrics-opt in screen to immediately after the welcome screen
* Ensure that global.getSentryState is set in the background
* Fix e2e tests after rearranging onboardin flow
* Fix unit tests
* More e2e test fixes
* Remove unnecessary wrappers around capture exception
* Updating design tokens v1.8 and shadows
* Adding missing stories
* Some fixes and updates to css and stories
* removing unneeded story
* Fixing story order
* Adding responsive props to Box component
* Updating margin array prop instances
* Updating padding array prop instances
* Updates to docs, tests and margin, padding instances
* Optimizing class name object
* Simplifying single value logic
* replacing for loop with switch statement
* Memoizing generateClassNames function
* set more appropriate default for ticker symbol when wallet_addEthereumChain is called
* throw error to dapp when site suggests network with same chainId but different ticker symbol from already added network, instead of showing error and disabled notification to user
* Removed infura v3 key from rpcUrl display
* Code refactor
* Fixed error
* Prevent submission of form when editing an featured rpc and there is a chainId error
* Add optional chaining to rpcurls to prevent type errors
Co-authored-by: Dan J Miller <danjm.com@gmail.com>
* Fix warning dialog when sending tokens to a known token contract address
Fixing after rebase
Covering missed cases
Rebased and ran yarn setup
Rebased
Fix checkContractAddress condition
Lint fix
Applied requested changes
Fix unit tests
Applying requested changes
Applied requested changes
Refactor and update
Lint fix
Use V2 of ActionableMessage component
Adding Learn More Link
Updating warning copy
Addressing review feedback
Fix up copy changes
Simplify validation of pasted addresses
Improve detection of whether this is a token contract
Refactor to leave updateRecipient unchanged, and to prevent the double calling of update recipient
Update tests
fix
* Fix unit tests
* Fix e2e tests
* Ensure next button is disabled while recipient type is loading
* Add optional chaining and a fallback to getRecipientWarningAcknowledgement
* Fix lint
* Don't reset recipient warning on asset change, because we should show recipient warnings regardless of asset
* Update unit tests
* Update unit tests
Co-authored-by: Filip Sekulic <filip.sekulic@consensys.net>
* Added 8px of margin to the top of the network display as it was touching the edge of the view
* Removed the SiteIcon component and passed the icon url to the SiteOrigin to display the icon next to the origin.
Co-authored-by: David Walsh <davidwalsh83@gmail.com>
* Ensure that addresses are properly hex-prefixed in the generate ERC token transfer functions
* Ensure hex-prefixed addresses in send input
* Update unit tests
Co-authored-by: Frederik Bolding <frederik.bolding@gmail.com>