Merge pull request #3752 from Emurgo/ruslan/relevant-networks

protocol params only for relevant networks

packages/yoroi-extension/app/api/ada/lib/storage/database/prepackaged/networks.js

@@ -51,6 +51,7 @@ export const networks = Object.freeze({
     ]: CardanoHaskellBaseConfig),
     CoinType: CoinTypes.CARDANO,
     Fork: CardanoForks.Haskell,
+    isInProduction: true,
   }: NetworkRow),
   CardanoPreprodTestnet: ({
     NetworkId: 2_50,
@@ -179,6 +180,14 @@ export const networks = Object.freeze({
   }: NetworkRow),
 });
 
+export function listRelevantNetworkNamesForEnvironment(): Array<string> {
+  const keys = Object.keys(networks);
+  if (environment.isProduction() && !environment.isNightly()) {
+    return keys.filter(k => networks[k].isInProduction);
+  }
+  return keys;
+}
+
 export function isTestnet(
   network: $ReadOnly<NetworkRow>,
 ): boolean {

packages/yoroi-extension/app/api/ada/lib/storage/database/primitives/tables.js

@@ -91,6 +91,7 @@ export type NetworkRow = {|
    * To differentiate these, we need some identifier of the fork
    */
   Fork: number,
+  isInProduction?: boolean,
 |};
 export const NetworkSchema: {|
   +name: 'Network',

packages/yoroi-extension/app/api/ada/lib/storage/tests/__snapshots__/index.test.js.snap

@@ -1263,6 +1263,7 @@ Object {
       "NetworkFeatureName": "mainnet",
       "NetworkId": 0,
       "NetworkName": "Cardano Mainnet",
+      "isInProduction": true,
     },
     Object {
       "Backend": Object {

packages/yoroi-extension/app/containers/wallet/WalletSendPage.js

@@ -98,8 +98,10 @@ class WalletSendPage extends Component<AllProps> {
         this.props.stores.substores.ada.addresses.getSupportedAddressDomainBannerState();
     });
     const { loadProtocolParametersRequest } = this.props.stores.protocolParameters;
-    loadProtocolParametersRequest.reset();
-    loadProtocolParametersRequest.execute();
+    if (!loadProtocolParametersRequest.wasExecuted && !loadProtocolParametersRequest.isExecuting) {
+      loadProtocolParametersRequest.reset();
+      loadProtocolParametersRequest.execute();
+    }
     ampli.sendInitiated();
   }
 

packages/yoroi-extension/app/stores/toplevel/ProtocolParametersStore.js

@@ -5,7 +5,10 @@
 import Store from '../base/Store';
 import type { ProtocolParameters } from '@emurgo/yoroi-lib/dist/protocol-parameters/models';
 import { getProtocolParameters } from '../../api/thunk';
-import { networks } from '../../api/ada/lib/storage/database/prepackaged/networks';
+import {
+  listRelevantNetworkNamesForEnvironment,
+  networks
+} from '../../api/ada/lib/storage/database/prepackaged/networks';
 import LocalizedRequest from '../lib/LocalizedRequest';
 import { observable } from 'mobx';
 
@@ -17,8 +20,12 @@ export default class ProtocolParametersStore<
 
   cache: Map<number, ProtocolParameters> = new Map();
 
+  setup(): void {
+    this.loadProtocolParametersRequest.execute();
+  }
+
   async loadProtocolParameters(): Promise<void> {
-    for (const key of Object.keys(networks)) {
+    for (const key of listRelevantNetworkNamesForEnvironment()) {
       const networkId = networks[key].NetworkId;
       if (networkId === networks.CardanoSanchoTestnet.NetworkId) continue;
       const protocolParameters = await getProtocolParameters({ networkId });

packages/yoroi-extension/chrome/extension/background/handlers/yoroi/wallet.js

@@ -65,6 +65,7 @@ export const CreateWallet: HandlerType<CreateWalletRequest, CreateWalletResponse
         publicDeriverId: publicDerivers[0].getPublicDeriverId(),
       }
     });
+    // noinspection ES6MissingAwait
     syncWallet(publicDerivers[0], 'new wallet', 1);
     return await getPlaceHolderWalletState(publicDerivers[0]);
   },

run-dev.sh

@@ -1 +1 @@
-nvm i && . install-all.sh && npm run dev:stable --prefix packages/yoroi-extension
+nvm i && . install-all.sh && npm run "dev${1:-:stable}" --prefix packages/yoroi-extension