[FLASK] Add updated version of the Snaps list UI #18284
Closed
+16,644
−7,329
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…disable domain binding (#18200) * siwe: re-enable warning UI for mismatched domains - unblocks mismatched domain support - we may re-add error handling here #18184 - reverts logic from #16616 * siwe: fix mismatch domain warning msg UI * lint: rm whitespace EOL * siwe: rm unit test * lint: fix whitespace * Revert "siwe: rm unit test" This reverts commit c80a4a2. --------- Co-authored-by: legobeat <[email protected]>
* security: patch request for CVE-2023-28155 GHSA-p8p7-x288-28g6 Ported from request/request#3444 * add iyarc exclusion
Version v10.26.2 RC
|
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
david0xd
added
flask
team-snaps
master sync following v10.26.2
* bump contract metadata package version * bump contract metadata package version
* feat: add the consensys zkEVM as a default network * fix: change infuraNetworkStatus in navigate-txs file * fix: remove account tracker for zkEVM + remove zkEVM from infura list * fix: change consensys zkevm name to linea + change rpc url for linea network * fix: rebase conflicts * feat: add new colors for linea goerli network * feat: add new function inside network dropdown to render non infura networks * feat: add feature toggle for linea network * fix: add new unit test --------- Co-authored-by: Dan J Miller <[email protected]>
* Adding ModalContent component * Using different component api for ref * use imperative handle * Updating size * Updating stories and docs as well as component api * Fixing import
* added redesign storybook * updated token list * updated css * fixed lint error * updated the new token list component * fixed redesign folder error * reverted changes in settings.json * updated redesign to multichain * added feature flag * reverted settings.json * added detect token banner * added button componeny * fixed lint errors * removed settings * fixed lint errors * added stories for multichain * updated no token found string * updated lint error * updated padding values * updated padding values * updated tabs with role button * updated hover state * updated components with multichain * fixed lint errors * updated multichain import token link * updated a tag * updated fixes * updated onClick to handleClick * updated setShowDetectedTokens proptype * updated multichain tokenlist with item suffix * updated tests * updated tests * updated token list css * updated snapshot * updated text * reverted story * added story for multichain token list * updated story * updated tooltip * updated the new token list component * fixed redesign folder error * added feature flag * reverted unused setting change * removed token list * fixed lint error * updated status * updated tooltip * updated token-list-item changes * updated actionbutton click for detect token banner * updated snapshot * updated symbol * updated styles * updated eth decimal and token url * updated snapshot * updated scripts * updated snapshots
…ot display the pointer anymore (#18046) * Signed Type Data address hover fix * Snapshots fix * Snapshots update fix * Update snapshot
* signature-req: fix overflow ui * signature-req: fix uneven btn height
* add header base component * fix resizing issue * add center * add demo * header base using flexbox * fix button issue * header base clean up * update tests * add readme description * add docs * update snapshot * add more to readme * convert to TS * fix file name * fix types and colors * fix classname error * fix boxprops import * fix boxprops * prop fix * fix errors * Update ui/components/component-library/header-base/header-base.stories.tsx Co-authored-by: George Marshall <[email protected]> * Update ui/components/component-library/header-base/header-base.types.ts Co-authored-by: George Marshall <[email protected]> * Update ui/components/component-library/header-base/header-base.types.ts Co-authored-by: George Marshall <[email protected]> * headerbase fixes * fix export * remove Math.max * change order for index on storybook to prep build * revert back to order * remove type from export * add type to export * change export of headerbase function * export update * revert back to normal * add type to export * Removing interface export from index --------- Co-authored-by: George Marshall <[email protected]>
* Ensure that NonInfuraDefaultNetworks are only selected in the dropdown if they are the currently selected network * Ensure Linea Goerli network appears in network settings tab if added manually
* Ensure that NonInfuraDefaultNetworks are only selected in the dropdown if they are the currently selected network * Ensure Linea Goerli network appears in network settings tab if added manually
* PermissionsConnectHeader: unlock SiteOrigin title * SignatureRequestOriginal: unlock SiteOrigin title * signature-req: update snapshots
* NFTs: Use Unknown Collection instead of first NFT name * Use localization in hook * Get localization for previously owned * Fix tests --------- Co-authored-by: Dan Miller <[email protected]>
* adds component with locales and test * adds feedback from review * adds storeis file * update snapshots * prettier * clean up stories file * relocation * review fixes * Update ui/pages/institutional/confirm-add-custodian-token/confirm-add-custodian-token.js Co-authored-by: George Marshall <[email protected]> * Update ui/pages/institutional/confirm-add-custodian-token/confirm-add-custodian-token.js Co-authored-by: George Marshall <[email protected]> * Update ui/pages/institutional/confirm-add-custodian-token/confirm-add-custodian-token.js Co-authored-by: George Marshall <[email protected]> * Button path remove * Update ui/pages/institutional/confirm-add-custodian-token/confirm-add-custodian-token.js Co-authored-by: George Marshall <[email protected]> * pulled * stories file update location --------- Co-authored-by: George Marshall <[email protected]>
* Added code fencing in info tab component * Fixed storybook urls * Removed isBeta mmi code fencing
* adds build type mmi * run prettier
* fix dapp interaction e2e test * wip * add sentry post request mock * fix console errors * fix scripting console error and remove e2e test unnecessary check * clean up * remove e2e test * stop skipping test * fixing build mv3 job * fixing unit tests * fixing unit tests * fixing unit tests * update coverages * revert skip mv3 e2e test * remove IN_TEST on the npm script * remove console.log * revert aria label changes * revert aria label changes * revert permission changes * revert permission changes * implement sw restart delay tracking * fix rebase
#18395) * Allow specific origin to direct to second page of token allowance flow * Remove accidentally committed git diff text --------- Co-authored-by: Erik Marks <[email protected]>
We want to convert NetworkController to TypeScript in order to be able to compare differences in the controller between in this repo and the core repo. To do this, however, we need to convert the dependencies of the controller to TypeScript. As a part of this effort, this commit converts `shared/modules/fetch-with-timeout.js` to TypeScript.
We want to convert NetworkController to TypeScript in order to be able to compare differences in the controller between in this repo and the core repo. To do this, however, we need to convert the dependencies of the controller to TypeScript. As a part of this effort, this commit converts `shared/modules/network.utils.js` to TypeScript, and also adds tests. Co-authored-by: Mark Stacey <[email protected]>
* added site connection menu component * reverted change for unlock page * updated snapshot * updated state with useSelector * updated state for connected * updated icons * updated test * updated snapshot * moved component to multichain folder * updated color * added multichain connection to menu bar * updated default color * updated css * updated multichain site with connected site info * updated ui for not connected state * removed scripts * updated lint errors * updated lint errors * updated stories * updated story for not connected to current state * updated story for not connected to current state * updated badge to 16px * updated badge position * updated snapshot * fixed lint errors * updated not connected state icon * updated constants for status and added new locale string
#18403) * Ensure that firefox linting of beta builds only happens on beta build commits * Fix script
* Fix in opensea message * Further copy changes --------- Co-authored-by: Dan J Miller <[email protected]>
The MMI build type is still a work in progress, and should not be expected to build successfully yet, so we haven't committed a LavaMoat policy to the repository for it yet. The policy update script has been updated to skip MMI by default for now; we can include it after that build has reached a point where we are confident it will build successfully.
We want to convert NetworkController to TypeScript in order to be able to compare differences in the controller between in this repo and the core repo. To do this, however, we need to convert the dependencies of the controller to TypeScript. As a part of this effort, this commit converts `shared/constants/metametrics` to TypeScript. Note that simple objects have been largely replaced with enums. There are some cases where I even split up some of these objects into multiple enums. Co-authored-by: Mark Stacey <[email protected]>
* Add permission cell component Add storybook and handling for revoked permission colors * Fix things after conflict resolve after rebase * Add code refactoring and minor UI changes * Add permission cell component to snap-update flow * Update storybook * Add unit tests for permission cell * Update component padding * Fix spacing between permission cells * Fix main icon color for approved permissions * Update width value with constant
* convert text component to TS * invisible character fix * storybook fix * fix types export * update ButtonBase * add typeof to objects in TextProps * fix linting issues * fix implicit conversion * lint fix * add deprecated Text back * change box ref to any * fix classnames issue * account details to use deprecated text reference * ref update * make RefObject * remove RefObject and go back to Ref * react.ref change to box
* update tabs to functional component * updated Tabs to use Box * updated css for tab * updated css on home screen * updated snapshot * fixed defaultkey and active key value for new keys
`@metamask/obs-store` has been updated from v5 to v8. The breaking changes include updating the minimum supported Node.js version to v12, and removing an unused class `LocalStorageStore`. v8 includes a bugfix that resolves an accidental breaking change in v6.0.2; it's non- breaking when updating from older versions. This update includes type improvements. The main improvement is that now we can type the observational store using generic parameters. The previous version used `unknown` for the store contents.
…8396) * disabled button for zero token * addressed changes
|
Closing this due to messed up git history after git's rebase feature. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closed and will be reopened in another PR.
Explanation
Fixes: MetaMask/snaps#1170
Screenshots/Screencaps
Before
After
Manual Testing Steps
Pre-merge author checklist
Pre-merge reviewer checklist
If further QA is required (e.g. new feature, complex testing steps, large refactor), add the
Extension QA Boardlabel.In this case, a QA Engineer approval will be be required.