refactor: update default API server (#3106)

Author: luciorubeens

__tests__/unit/services/client.spec.js

@@ -16,7 +16,8 @@ const sessionNetwork = Object.freeze({
   },
   vendorField: {
     maxLength: 64
-  }
+  },
+  version: 23
 })
 
 jest.mock('@/store', () => ({

__tests__/unit/services/crypto/transaction-signer.spec.js

@@ -16,7 +16,8 @@ const sessionNetwork = Object.freeze({
   },
   vendorField: {
     maxLength: 64
-  }
+  },
+  version: 23
 })
 
 jest.mock('@/store', () => ({

config/networks/devnet.json

@@ -2,7 +2,7 @@
   "id": "ark.devnet",
   "name": "ARK Devnet",
   "title": "ARK Devnet",
-  "server": "https://dexplorer.ark.io:8443",
+  "server": "https://dwallets.ark.io",
   "description": "The ARK Devnet is only used for R&D by engineers.",
   "imagePath": "networks/ark.devnet.svg",
   "apiVersion": 2,

config/networks/mainnet.json

@@ -2,7 +2,7 @@
   "id": "ark.mainnet",
   "name": "ARK Mainnet",
   "title": "ARK Mainnet",
-  "server": "https://explorer.ark.io:8443",
+  "server": "http://wallets.ark.io",
   "description": "The ARK Mainnet should be chosen in most cases.",
   "imagePath": "networks/ark.png",
   "apiVersion": 2,

src/renderer/services/crypto/transaction-signer.js

@@ -27,7 +27,7 @@ export class TransactionSigner {
       network = store.getters['session/network']
     }
 
-    transaction = transaction.network(network.version)
+    transaction = transaction.network(+network.version)
 
     // TODO replace with dayjs
     const epochTime = dayjs(network.constants.epoch)

src/renderer/store/modules/network.js

@@ -53,18 +53,28 @@ export default new BaseModule(NetworkModel, {
     load ({ commit, getters, rootGetters }) {
       const all = cloneDeep(getters.all)
       if (!isEmpty(all)) {
+        // Update API server on existing networks
+        const servers = {
+          'ark.devnet': 'https://dwallets.ark.io',
+          'ark.mainnet': 'http://wallets.ark.io'
+        }
+        const sanitizedAll = all.map(network => {
+          const server = servers[network.id] || network.server
+          return {
+            ...network,
+            server
+          }
+        })
+
         // TODO: remove in future major version
         // This is a "hack" to make sure all custom networks are in state.all
-        let missingCustom = false
         for (const custom of Object.values(getters.customNetworks)) {
-          if (!all.find(network => network.name === custom.name)) {
-            all.push(custom)
-            missingCustom = true
+          if (!sanitizedAll.find(network => network.name === custom.name)) {
+            sanitizedAll.push(custom)
           }
         }
-        if (missingCustom) {
-          commit('SET_ALL', all)
-        }
+
+        commit('SET_ALL', sanitizedAll)
       } else {
         commit('SET_ALL', NETWORKS)
       }