Skip to content

Commit a83e8ce

Browse files
committed
update
1 parent 5cbd4bd commit a83e8ce

File tree

4 files changed

+28
-19
lines changed

4 files changed

+28
-19
lines changed

ui/pages/confirmations/confirmation/alerts/useAlertsActions.tsx

+8-2
Original file line numberDiff line numberDiff line change
@@ -3,15 +3,21 @@ import { useCallback } from 'react';
33
import { useSelector } from 'react-redux';
44

55
import { AlertActionKey } from '../../../../components/app/confirm/info/row/constants';
6-
import { getApprovalsByOrigin } from '../../../../selectors';
76
import { useConfirmationNavigation } from '../../hooks/useConfirmationNavigation';
7+
import {
8+
ApprovalsMetaMaskState,
9+
getApprovalsByOrigin,
10+
} from '../../../../selectors';
811

912
export const useAlertsActions = (
1013
hideAlertModal: () => void,
1114
pendingConfirmation: ApprovalRequest<{ id: string }>,
1215
) => {
1316
const pendingConfirmationsFromOrigin = useSelector((state) =>
14-
getApprovalsByOrigin(state, pendingConfirmation?.origin),
17+
getApprovalsByOrigin(
18+
state as ApprovalsMetaMaskState,
19+
pendingConfirmation?.origin,
20+
),
1521
);
1622

1723
const { getIndex, navigateToIndex } = useConfirmationNavigation();

ui/pages/confirmations/confirmation/alerts/useUpdateEthereumChainAlerts.ts

+8-2
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,10 @@ import { useSelector } from 'react-redux';
66
import { Alert } from '../../../../ducks/confirm-alerts/confirm-alerts';
77
import { AlertActionKey } from '../../../../components/app/confirm/info/row/constants';
88
import { Severity } from '../../../../helpers/constants/design-system';
9-
import { getApprovalsByOrigin } from '../../../../selectors';
9+
import {
10+
ApprovalsMetaMaskState,
11+
getApprovalsByOrigin,
12+
} from '../../../../selectors';
1013
import { useI18nContext } from '../../../../hooks/useI18nContext';
1114

1215
const VALIDATED_APPROVAL_TYPES = [
@@ -18,7 +21,10 @@ export function useUpdateEthereumChainAlerts(
1821
pendingConfirmation: ApprovalRequest<{ id: string }>,
1922
): Alert[] {
2023
const pendingConfirmationsFromOrigin = useSelector((state) =>
21-
getApprovalsByOrigin(state, pendingConfirmation?.origin),
24+
getApprovalsByOrigin(
25+
state as ApprovalsMetaMaskState,
26+
pendingConfirmation?.origin,
27+
),
2228
);
2329

2430
const t = useI18nContext();

ui/selectors/approvals.ts

+12
Original file line numberDiff line numberDiff line change
@@ -99,6 +99,18 @@ export const selectPendingApproval = createDeepEqualSelector(
9999
(approval) => approval,
100100
);
101101

102+
export const getApprovalsByOrigin = (
103+
state: ApprovalsMetaMaskState,
104+
origin: string,
105+
) => {
106+
const pendingApprovals = getPendingApprovals(state);
107+
108+
return pendingApprovals?.filter(
109+
(confirmation: ApprovalRequest<Record<string, Json>>) =>
110+
confirmation.origin === origin,
111+
);
112+
};
113+
102114
function isWatchTokenApproval(approval: ApprovalRequest<Record<string, Json>>) {
103115
const tokenId = (approval.requestData?.asset as Record<string, string>)
104116
?.tokenId;

ui/selectors/selectors.js

-15
Original file line numberDiff line numberDiff line change
@@ -1502,21 +1502,6 @@ export const getMemoizedUnapprovedConfirmations = createDeepEqualSelector(
15021502
(confirmations) => confirmations,
15031503
);
15041504

1505-
/**
1506-
* Get pending confirmation from specified origin.
1507-
*
1508-
* @param state - Redux state object.
1509-
* @param origin - Origin to ger approvals from.
1510-
* @returns array of approvals from an origin
1511-
*/
1512-
export const getApprovalsByOrigin = (state, origin) => {
1513-
const pendingApprovals = getMemoizedUnapprovedConfirmations(state);
1514-
1515-
return pendingApprovals?.filter(
1516-
(confirmation) => confirmation.origin === origin,
1517-
);
1518-
};
1519-
15201505
/**
15211506
* Get a memoized version of the unapproved templated confirmations.
15221507
*/

0 commit comments

Comments
 (0)