Skip to content

Commit bf62819

Browse files
committed
feat(suite-native): sort networks according to networksConfig
1 parent e4536e2 commit bf62819

File tree

4 files changed

+19
-13
lines changed

4 files changed

+19
-13
lines changed

suite-native/accounts/src/utils.ts

+3-3
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ import { A, D, G } from '@mobily/ts-belt';
33
import { AccountType, getNetwork, networks } from '@suite-common/wallet-config';
44
import { formattedAccountTypeMap } from '@suite-common/wallet-core';
55
import { Account } from '@suite-common/wallet-types';
6-
import { discoverySupportedNetworks, orderedAccountTypes } from '@suite-native/config';
6+
import { orderedNetworkSymbols, orderedAccountTypes } from '@suite-native/config';
77

88
const accountTypeToSectionHeader: Readonly<Partial<Record<AccountType, string>>> = {
99
normal: 'default',
@@ -79,8 +79,8 @@ export const groupAccountsByNetworkAccountType = A.groupBy((account: Account) =>
7979

8080
export const sortAccountsByNetworksAndAccountTypes = (accounts: readonly Account[]) => {
8181
return A.sort(accounts, (a, b) => {
82-
const aOrder = discoverySupportedNetworks.indexOf(a.symbol) ?? Number.MAX_SAFE_INTEGER;
83-
const bOrder = discoverySupportedNetworks.indexOf(b.symbol) ?? Number.MAX_SAFE_INTEGER;
82+
const aOrder = orderedNetworkSymbols.indexOf(a.symbol) ?? Number.MAX_SAFE_INTEGER;
83+
const bOrder = orderedNetworkSymbols.indexOf(b.symbol) ?? Number.MAX_SAFE_INTEGER;
8484

8585
if (aOrder === bOrder) {
8686
const aAccountTypeOrder =

suite-native/assets/src/assetsSelectors.ts

+3-3
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ import {
1616
import { getAccountFiatBalance } from '@suite-common/wallet-utils';
1717
import { selectAccountListSections } from '@suite-native/accounts';
1818
import { sortAccountsByNetworksAndAccountTypes } from '@suite-native/accounts/src/utils';
19-
import { discoverySupportedNetworks } from '@suite-native/config';
19+
import { orderedNetworkSymbols } from '@suite-native/config';
2020
import { selectFiatCurrencyCode, SettingsSliceRootState } from '@suite-native/settings';
2121
import {
2222
NativeStakingRootState,
@@ -60,8 +60,8 @@ export const selectDeviceNetworksWithAssets = (state: AssetsRootState) => {
6060
A.map(account => account.symbol),
6161
A.uniq,
6262
A.sort((a, b) => {
63-
const aOrder = discoverySupportedNetworks.indexOf(a) ?? Number.MAX_SAFE_INTEGER;
64-
const bOrder = discoverySupportedNetworks.indexOf(b) ?? Number.MAX_SAFE_INTEGER;
63+
const aOrder = orderedNetworkSymbols.indexOf(a) ?? Number.MAX_SAFE_INTEGER;
64+
const bOrder = orderedNetworkSymbols.indexOf(b) ?? Number.MAX_SAFE_INTEGER;
6565

6666
return aOrder - bOrder;
6767
}),

suite-native/config/src/supportedNetworks.ts

+12-6
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ import { A } from '@mobily/ts-belt';
22

33
import { isTestnet } from '@suite-common/wallet-utils';
44
import {
5+
networks,
56
AccountType,
67
Network,
78
NetworkSymbol,
@@ -48,10 +49,12 @@ export const discoverySupportedNetworks = [
4849
...networkSymbolsWhitelistMap.testnet,
4950
];
5051

51-
export const sortNetworks = (networks: Network[]) =>
52-
A.sort(networks, (a, b) => {
53-
const aOrder = discoverySupportedNetworks.indexOf(a.symbol) ?? Number.MAX_SAFE_INTEGER;
54-
const bOrder = discoverySupportedNetworks.indexOf(b.symbol) ?? Number.MAX_SAFE_INTEGER;
52+
export const orderedNetworkSymbols = Object.keys(networks) as NetworkSymbol[];
53+
54+
export const sortNetworks = (networksToSort: Network[]) =>
55+
A.sort(networksToSort, (a, b) => {
56+
const aOrder = orderedNetworkSymbols.indexOf(a.symbol);
57+
const bOrder = orderedNetworkSymbols.indexOf(b.symbol);
5558

5659
return aOrder - bOrder;
5760
});
@@ -65,8 +68,11 @@ export const filterTestnetNetworks = (
6568
return networkSymbols.filter(networkSymbol => !isTestnet(networkSymbol));
6669
};
6770

68-
export const filterBlacklistedNetworks = (networks: Network[], allowList: NetworkSymbol[]) =>
69-
networks.filter(
71+
export const filterBlacklistedNetworks = (
72+
networksToFilter: Network[],
73+
allowList: NetworkSymbol[],
74+
) =>
75+
networksToFilter.filter(
7076
network =>
7177
!discoveryBlacklist.includes(network.symbol) || allowList.includes(network.symbol),
7278
);

suite-native/module-add-accounts/src/hooks/useAddCoinAccount.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,7 @@ export const useAddCoinAccount = () => {
111111
availableTypes.set(symbol as NetworkSymbol, [
112112
NORMAL_ACCOUNT_TYPE,
113113
// For Cardano and EVMs allow only normal account type
114-
...(['ada', 'eth', 'pol', 'bnb', 'sol'].includes(symbol) ? [] : types),
114+
...(['ada', 'eth', 'pol', 'bnb', 'sol', 'op'].includes(symbol) ? [] : types),
115115
]);
116116
});
117117

0 commit comments

Comments
 (0)