Skip to content

Commit

Permalink
Merge remote-tracking branch 'origin/main' into asynckeyexchange
Browse files Browse the repository at this point in the history
  • Loading branch information
abretonc7s committed Aug 22, 2024
2 parents 7bd210f + aa2a0db commit 586c4b1
Show file tree
Hide file tree
Showing 73 changed files with 2,485 additions and 1,073 deletions.
89 changes: 0 additions & 89 deletions .github/scripts/add-team-label.ts

This file was deleted.

4 changes: 3 additions & 1 deletion .github/workflows/add-team-label.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,8 @@ on:

jobs:
add-team-label:
uses: metamask/github-tools/.github/workflows/add-team-label.yml@7c10eb3bafb3f221111a9e4309ae5dcaee171de5
uses: metamask/github-tools/.github/workflows/add-team-label.yml@058012b49ff2fbd9649c566ba43b29497f93b21d
permissions:
pull-requests: write
secrets:
PERSONAL_ACCESS_TOKEN: ${{ secrets.RELEASE_LABEL_TOKEN }}
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ export interface PickerAccountProps extends Omit<PickerBaseProps, 'children'> {
/**
* Account type label.
*/
accountTypeLabel?: string;
accountTypeLabel?: string | null;
}

/**
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ const SignatureApproval = () => {
const onSignConfirm = useCallback(async () => {
await onConfirm({
waitForResult: true,
//@ts-expect-error - this is added via patch of approval controller
deleteAfterResult: true,
handleErrors: false,
});
Expand Down
10 changes: 7 additions & 3 deletions app/components/Nav/App/index.test.tsx
Original file line number Diff line number Diff line change
@@ -1,12 +1,16 @@
import { renderScreen } from '../../../util/test/renderWithProvider';
import {
DeepPartial,
renderScreen,
} from '../../../util/test/renderWithProvider';
import { backgroundState } from '../../../util/test/initial-root-state';
import App from './';
import { MetaMetrics } from '../../../core/Analytics';
import { waitFor } from '@testing-library/react-native';
import { RootState } from '../../../reducers';

const initialState = {
const initialState: DeepPartial<RootState> = {
user: {
loggedIn: true,
userLoggedIn: true,
},
engine: {
backgroundState,
Expand Down
5 changes: 4 additions & 1 deletion app/components/Nav/Main/index.test.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,10 @@ function renderUseSwapConfirmedEventHook({
state: {
engine: {
backgroundState: {
TransactionController: { swapsTransactions },
TransactionController: {
//@ts-expect-error - swaps transactions is something we do not have implemented on TransacitonController yet
swapsTransactions,
},
},
},
},
Expand Down
7 changes: 5 additions & 2 deletions app/components/UI/AccountInfoCard/index.test.tsx
Original file line number Diff line number Diff line change
@@ -1,11 +1,14 @@
import React from 'react';
import AccountInfoCard from './';
import renderWithProvider from '../../../util/test/renderWithProvider';
import renderWithProvider, {
DeepPartial,
} from '../../../util/test/renderWithProvider';
import { backgroundState } from '../../../util/test/initial-root-state';
import {
MOCK_ACCOUNTS_CONTROLLER_STATE,
MOCK_ADDRESS_1,
} from '../../../util/test/accountsControllerTestUtils';
import { RootState } from '../../../reducers';

jest.mock('../../../core/Engine', () => ({
resetState: jest.fn(),
Expand All @@ -21,7 +24,7 @@ jest.mock('../../../core/Engine', () => ({
},
}));

const mockInitialState = {
const mockInitialState: DeepPartial<RootState> = {
settings: {
useBlockieIcon: false,
},
Expand Down
11 changes: 9 additions & 2 deletions app/components/UI/AssetSearch/index.tsx
Original file line number Diff line number Diff line change
@@ -1,5 +1,12 @@
import React, { memo, useEffect, useState, useCallback } from 'react';
import { TextInput, View, StyleSheet, Platform, TextStyle } from 'react-native';
import {
TextInput,
View,
StyleSheet,
Platform,
TextStyle,
DimensionValue,
} from 'react-native';
import { fontStyles } from '../../../styles/common';
import { strings } from '../../../../locales/i18n';
import Fuse from 'fuse.js';
Expand Down Expand Up @@ -102,7 +109,7 @@ interface Props {
// eslint-disable-next-line react/display-name
const AssetSearch = memo(({ onSearch, onFocus, onBlur }: Props) => {
const [searchQuery, setSearchQuery] = useState('');
const [inputDimensions, setInputDimensions] = useState('85%');
const [inputDimensions, setInputDimensions] = useState<DimensionValue>('85%');
const [isFocus, setIsFocus] = useState(false);
const tokenList = useSelector(selectTokenListArray);
const { colors, themeAppearance } = useTheme();
Expand Down
Loading

0 comments on commit 586c4b1

Please sign in to comment.