fix bug where chain permissions were not properly represented in the …

…permission confirmation
MetaMask · Oct 15, 2024 · 85b6091 · 85b6091
1 parent a770169
commit 85b6091
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/ui/components/app/permission-cell/permission-cell.js b/ui/components/app/permission-cell/permission-cell.js
@@ -42,7 +42,7 @@ const PermissionCell = ({
   showOptions,
   hideStatus,
   accounts,
-  permissionValue,
+  chainIds,
 }) => {
   const infoIcon = IconName.Info;
   let infoIconColor = IconColor.iconMuted;
@@ -71,7 +71,7 @@ const PermissionCell = ({
   }
 
   const networksInfo = useSelector((state) =>
-    getRequestingNetworkInfo(state, permissionValue),
+    getRequestingNetworkInfo(state, chainIds),
   );
 
   return (
@@ -171,7 +171,7 @@ PermissionCell.propTypes = {
   showOptions: PropTypes.bool,
   hideStatus: PropTypes.bool,
   accounts: PropTypes.array,
-  permissionValue: PropTypes.array,
+  chainIds: PropTypes.array,
 };
 
 export default PermissionCell;
diff --git a/ui/components/app/permissions-connect-permission-list/permissions-connect-permission-list.js b/ui/components/app/permissions-connect-permission-list/permissions-connect-permission-list.js
@@ -17,6 +17,9 @@ import { Box } from '../../component-library';
  * @returns {JSX.Element} A permission description node.
  */
 function getDescriptionNode(permission, index, accounts) {
+  const chainIds = permission.permissionValue.caveats.find(
+    (caveat) => caveat.type === 'restrictNetworkSwitching',
+  )?.value;
   return (
     <PermissionCell
       permissionName={permission.name}
@@ -26,7 +29,7 @@ function getDescriptionNode(permission, index, accounts) {
       avatarIcon={permission.leftIcon}
       key={`${permission.permissionName}-${index}`}
       accounts={accounts}
-      permissionValue={permission.permissionValue.restrictNetworkSwitching}
+      chainIds={chainIds}
     />
   );
 }