From 85b6091332dbf47a19348f1b6cf44de758420a9c Mon Sep 17 00:00:00 2001 From: Alex Date: Tue, 15 Oct 2024 17:08:02 -0500 Subject: [PATCH] fix bug where chain permissions were not properly represented in the permission confirmation --- ui/components/app/permission-cell/permission-cell.js | 6 +++--- .../permissions-connect-permission-list.js | 5 ++++- 2 files changed, 7 insertions(+), 4 deletions(-) diff --git a/ui/components/app/permission-cell/permission-cell.js b/ui/components/app/permission-cell/permission-cell.js index 4df1dc256765..7d85f48d2a54 100644 --- 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 index da15d384849c..4917e25a1315 100644 --- 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 ( ); }