Skip to content

Commit 256279f

Browse files
committed
feat(suite-native): sort networks according to networksConfig
1 parent b3eb721 commit 256279f

File tree

4 files changed

+12
-9
lines changed

4 files changed

+12
-9
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 { orderedNetworksSymbols, 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 = orderedNetworksSymbols.indexOf(a.symbol) ?? Number.MAX_SAFE_INTEGER;
83+
const bOrder = orderedNetworksSymbols.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 { orderedNetworksSymbols } 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 = orderedNetworksSymbols.indexOf(a) ?? Number.MAX_SAFE_INTEGER;
64+
const bOrder = orderedNetworksSymbols.indexOf(b) ?? Number.MAX_SAFE_INTEGER;
6565

6666
return aOrder - bOrder;
6767
}),

suite-native/config/src/supportedNetworks.ts

+5-2
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

52+
export const orderedNetworksSymbols = Object.keys(networks) as NetworkSymbol[];
53+
5154
export const sortNetworks = (networks: Network[]) =>
5255
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;
56+
const aOrder = orderedNetworksSymbols.indexOf(a.symbol);
57+
const bOrder = orderedNetworksSymbols.indexOf(b.symbol);
5558

5659
return aOrder - bOrder;
5760
});

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)