diff --git a/ui/components/app/permission-page-container/permission-page-container.component.js b/ui/components/app/permission-page-container/permission-page-container.component.js index da7719f6d4dc..f5f69da8f947 100644 --- a/ui/components/app/permission-page-container/permission-page-container.component.js +++ b/ui/components/app/permission-page-container/permission-page-container.component.js @@ -147,7 +147,7 @@ export default class PermissionPageContainer extends Component { ); const permittedChainsPermission = - _request.permissions[PermissionNames.permittedChains]; + _request.permissions?.[PermissionNames.permittedChains]; const approvedChainIds = permittedChainsPermission?.caveats.find( (caveat) => caveat.type === CaveatTypes.restrictNetworkSwitching, )?.value; @@ -155,8 +155,8 @@ export default class PermissionPageContainer extends Component { const request = { ..._request, permissions: { ..._request.permissions }, - ...(_request.permissions.eth_accounts && { approvedAccounts }), - ...(_request.permissions[PermissionNames.permittedChains] && { + ...(_request.permissions?.eth_accounts && { approvedAccounts }), + ...(_request.permissions?.[PermissionNames.permittedChains] && { approvedChainIds, }), }; diff --git a/ui/pages/permissions-connect/permissions-connect.component.js b/ui/pages/permissions-connect/permissions-connect.component.js index 403c431330b1..09befa7218cd 100644 --- a/ui/pages/permissions-connect/permissions-connect.component.js +++ b/ui/pages/permissions-connect/permissions-connect.component.js @@ -148,9 +148,6 @@ export default class PermissionConnect extends Component { history.replace(DEFAULT_ROUTE); return; } - if (process.env.CHAIN_PERMISSIONS) { - history.replace(confirmPermissionPath); - } // if this is an incremental permission request for permitted chains, skip the account selection if ( permissionsRequest?.diff?.permissionDiffMap?.[ @@ -341,33 +338,8 @@ export default class PermissionConnect extends Component { ( - this.selectAccounts(addresses)} - selectNewAccountViaModal={(handleAccountClick) => { - showNewAccountModal({ - onCreateNewAccount: (address) => - handleAccountClick(address), - newAccountNumber, - }); - }} - addressLastConnectedMap={addressLastConnectedMap} - cancelPermissionsRequest={(requestId) => - this.cancelPermissionsRequest(requestId) - } - permissionsRequestId={permissionsRequestId} - selectedAccountAddresses={selectedAccountAddresses} - targetSubjectMetadata={targetSubjectMetadata} - /> - )} - /> - - process.env.CHAIN_PERMISSIONS && !permissionsRequest?.diff ? ( + process.env.CHAIN_PERMISSIONS ? ( this.cancelPermissionsRequest(requestId) @@ -378,31 +350,58 @@ export default class PermissionConnect extends Component { approveConnection={this.approveConnection} /> ) : ( - { - approvePermissionsRequest(...args); - this.redirect(true); + + this.selectAccounts(addresses) + } + selectNewAccountViaModal={(handleAccountClick) => { + showNewAccountModal({ + onCreateNewAccount: (address) => + handleAccountClick(address), + newAccountNumber, + }); }} - rejectPermissionsRequest={(requestId) => + addressLastConnectedMap={addressLastConnectedMap} + cancelPermissionsRequest={(requestId) => this.cancelPermissionsRequest(requestId) } - selectedAccounts={accounts.filter((account) => - selectedAccountAddresses.has(account.address), - )} + permissionsRequestId={permissionsRequestId} + selectedAccountAddresses={selectedAccountAddresses} targetSubjectMetadata={targetSubjectMetadata} - history={this.props.history} - connectPath={connectPath} - snapsInstallPrivacyWarningShown={ - snapsInstallPrivacyWarningShown - } - setSnapsInstallPrivacyWarningShownStatus={ - setSnapsInstallPrivacyWarningShownStatus - } /> ) } /> + ( + { + approvePermissionsRequest(...args); + this.redirect(true); + }} + rejectPermissionsRequest={(requestId) => + this.cancelPermissionsRequest(requestId) + } + selectedAccounts={accounts.filter((account) => + selectedAccountAddresses.has(account.address), + )} + targetSubjectMetadata={targetSubjectMetadata} + history={this.props.history} + connectPath={connectPath} + snapsInstallPrivacyWarningShown={ + snapsInstallPrivacyWarningShown + } + setSnapsInstallPrivacyWarningShownStatus={ + setSnapsInstallPrivacyWarningShownStatus + } + /> + )} + />