fixup@ feat(suite-native): only download token definitions for releva…

…nt networks
trezor · Nov 6, 2024 · 6006606 · 6006606
1 parent 59c0334
commit 6006606
Show file tree

Hide file tree

Showing 6 changed files with 4 additions and 6 deletions.
diff --git a/suite-native/discovery/src/discoveryMiddleware.ts b/suite-native/discovery/src/discoveryMiddleware.ts
@@ -1,3 +1,5 @@
+import { createMiddlewareWithExtraDeps } from '@suite-common/redux-utils';
+import { periodicCheckTokenDefinitionsThunk } from '@suite-common/token-definitions';
 import {
     deviceActions,
     discoveryActions,
@@ -6,7 +8,6 @@ import {
     authorizeDeviceThunk,
     accountsActions,
 } from '@suite-common/wallet-core';
-import { createMiddlewareWithExtraDeps } from '@suite-common/redux-utils';
 import { isFirmwareVersionSupported } from '@suite-native/device';
 
 import { startDescriptorPreloadedDiscoveryThunk, discoveryCheckThunk } from './discoveryThunks';
@@ -17,7 +18,6 @@ import {
     setEnabledDiscoveryNetworkSymbols,
     toggleEnabledDiscoveryNetworkSymbol,
 } from './discoveryConfigSlice';
-import { periodicCheckTokenDefinitionsThunk } from '@suite-common/token-definitions';
 
 export const prepareDiscoveryMiddleware = createMiddlewareWithExtraDeps(
     (action, { dispatch, next, getState }) => {

diff --git a/suite-native/state/package.json b/suite-native/state/package.json
@@ -31,7 +31,6 @@
         "@suite-native/module-send": "workspace:*",
         "@suite-native/settings": "workspace:*",
         "@suite-native/storage": "workspace:*",
-        "@suite-native/tokens": "workspace:*",
         "@trezor/transport-native": "workspace:*",
         "@trezor/utils": "workspace:*",
         "expo-device": "6.0.2",

diff --git a/suite-native/state/src/extraDependencies.ts b/suite-native/state/src/extraDependencies.ts
@@ -36,7 +36,7 @@ export const extraDependencies: ExtraDependencies = mergeDeepObject(extraDepende
         // otherwise disableAccountsThunk might erase accounts not supported by current device
         selectEnabledNetworks: selectEnabledDiscoveryNetworkSymbols,
         // todo: this is temporary solution to make token definitions work on native in portfolio tracker
-        selectTokenDefinitionsEnabledNetworks: selectTokenDefinitionsEnabledNetworks,
+        selectTokenDefinitionsEnabledNetworks,
         selectBitcoinAmountUnit: selectBitcoinUnits,
         selectAreSatsAmountUnit,
         selectLocalCurrency: selectFiatCurrencyCode,

diff --git a/suite-native/state/tsconfig.json b/suite-native/state/tsconfig.json
@@ -34,7 +34,6 @@
         { "path": "../module-send" },
         { "path": "../settings" },
         { "path": "../storage" },
-        { "path": "../tokens" },
         {
             "path": "../../packages/transport-native"
         },

diff --git a/suite-native/tokens/src/tokensSelectors.ts b/suite-native/tokens/src/tokensSelectors.ts
@@ -284,5 +284,6 @@ export const selectNetworkSymbolsOfAccountsWithTokensAllowed = (state: TokensRoo
             if (!acc.includes(account.symbol)) {
                 acc.push(account.symbol);
             }
+
             return acc;
         }, new Array<NetworkSymbol>());
diff --git a/yarn.lock b/yarn.lock
@@ -10508,7 +10508,6 @@ __metadata:
     "@suite-native/module-send": "workspace:*"
     "@suite-native/settings": "workspace:*"
     "@suite-native/storage": "workspace:*"
-    "@suite-native/tokens": "workspace:*"
     "@trezor/transport-native": "workspace:*"
     "@trezor/utils": "workspace:*"
     expo-device: "npm:6.0.2"