feat: logging (#380)

* feat: add logging framework
* chore: change logger to log for ledger connect function
* chore: use electron-log for file logs

package.json

@@ -79,6 +79,7 @@
     "bip39": "^2.5.0",
     "chart.js": "^2.7.2",
     "dayjs": "^1.7.5",
+    "electron-log": "^2.2.17",
     "jsonschema": "^1.2.4",
     "localforage": "^1.7.2",
     "lodash": "^4.17.11",

src/renderer/components/Button/ButtonClipboard.vue

@@ -74,7 +74,7 @@ export default {
         document.execCommand('copy')
       } catch (err) {
         this.isCopySupported = false
-        console.error('Clipboard not supported!')
+        this.$logger.error('Clipboard not supported!')
       }
 
       document.body.removeChild(textArea)

src/renderer/components/Dashboard/DashboardTransactions.vue

@@ -60,7 +60,7 @@ export default {
           ], 'id')
         })
       } catch (error) {
-        console.error(error)
+        this.$logger.error(error)
         this.$error(this.$t('COMMON.FAILED_FETCH', {
           name: 'transactions',
           msg: error.message

src/renderer/components/Input/InputAddress.vue

@@ -141,7 +141,7 @@ export default {
 
     openQR () {
       // TODO when the QR reader is available
-      console.error('QR reader is not available yet')
+      this.$logger.error('QR reader is not available yet')
     },
 
     updateInputValue (value) {

src/renderer/components/Passphrase/PassphraseInput.vue

@@ -173,7 +173,7 @@ export default {
     },
     openQR () {
       // TODO when the QR reader is available
-      console.error('QR reader is not available yet')
+      this.$logger.error('QR reader is not available yet')
     },
 
     toggleVisible () {

src/renderer/components/Transaction/TransactionForm/TransactionFormSecondSignature.vue

@@ -135,7 +135,6 @@ export default {
 
     generateNewPassphrase () {
       this.secondPassphrase = WalletService.generateSecondPassphrase()
-      console.log(this.secondPassphrase)
     },
 
     async onSubmit () {

src/renderer/components/Transaction/TransactionModal.vue

@@ -94,8 +94,8 @@ export default {
     },
 
     successfulResponse (response) {
-      console.log(response)
       if (response.status !== 200) {
+        this.$logger.error(response)
         return false
       }
 

src/renderer/components/Wallet/WalletDelegates/WalletDelegates.vue

@@ -102,7 +102,7 @@ export default {
       try {
         this.delegates = await this.$client.fetchDelegates()
       } catch (error) {
-        console.error(error)
+        this.$logger.error(error)
         this.$error(this.$t('COMMON.FAILED_FETCH', {
           name: 'delegates',
           msg: error.message
@@ -114,7 +114,7 @@ export default {
       try {
         this.votePublicKey = await this.$client.fetchWalletVote(this.wallet_fromRoute.address)
       } catch (error) {
-        console.error(error)
+        this.$logger.error(error)
         this.$error(this.$t('COMMON.FAILED_FETCH', {
           name: 'fetch vote',
           msg: error.message

src/renderer/components/Wallet/WalletSidebar/WalletSidebar.vue

@@ -102,7 +102,7 @@ export default {
           this.$store.dispatch('wallet/update', updatedWallet)
         }
       } catch (error) {
-        console.error(error)
+        this.$logger.error(error)
         // TODO the error could mean that the wallet isn't on the blockchain yet
         // this.$error(this.$t('COMMON.FAILED_FETCH', {
         //   name: 'wallet data',

src/renderer/components/Wallet/WalletTransactions/WalletTransactions.vue

@@ -35,7 +35,7 @@ export default {
       try {
         this.transactions = await this.$client.fetchTransactions(this.wallet_fromRoute.address)
       } catch (error) {
-        console.error(error)
+        this.$logger.error(error)
         this.$error(this.$t('COMMON.FAILED_FETCH', {
           name: 'transactions',
           msg: error.message

src/renderer/main.js

@@ -5,6 +5,7 @@ import VTooltip from 'v-tooltip'
 import VueGoodTablePlugin from 'vue-good-table'
 import PortalVue from 'portal-vue'
 import axios from 'axios'
+import logger from 'electron-log'
 
 import App from './App'
 import i18n from './i18n'
@@ -19,6 +20,7 @@ import eventBus from '@/plugins/event-bus'
 
 Vue.config.productionTip = false
 Vue.http = Vue.prototype.$http = axios
+Vue.logger = Vue.prototype.$logger = logger
 Vue.prototype.$eventBus = eventBus
 
 Vue.use(filters)

src/renderer/services/crypto-compare.js

@@ -4,6 +4,7 @@ import i18n from '@/i18n'
 import alertEvents from '@/plugins/alert-events'
 import dayjs from 'dayjs'
 import { capitalize, keys, min, max } from 'lodash'
+import logger from 'electron-log'
 
 class CryptoCompare {
   /**
@@ -24,6 +25,7 @@ class CryptoCompare {
 
       return this.__transformMarketResponse(data)
     } catch (error) {
+      logger.error(error)
       alertEvents.$error(i18n.t('COMMON.FAILED_FETCH', {
         name: i18n.t('MARKET.MARKET'),
         msg: error.message
@@ -116,6 +118,7 @@ class CryptoCompare {
       const response = await axios.get(uri, { params })
       return this.__transformHistoricalResponse(response.data.Data, dateFormat)
     } catch (error) {
+      logger.error(error)
       alertEvents.$error(i18n.t('COMMON.FAILED_FETCH', {
         name: i18n.t('MARKET.HISTORICAL_DATA'),
         msg: error.message

src/renderer/services/ledger-service.js

@@ -1,6 +1,7 @@
 import LedgerTransport from '@ledgerhq/hw-transport-node-hid'
 import ArkLedger from '@arkecosystem/ledger-transport'
 import queue from 'async/queue'
+import logger from 'electron-log'
 
 class LedgerService {
   /**
@@ -31,7 +32,7 @@ class LedgerService {
 
       return this.isConnected()
     } catch (error) {
-      //
+      logger.log(error)
     }
 
     return false
@@ -46,7 +47,7 @@ class LedgerService {
     try {
       await this.transport.close()
     } catch (error) {
-      //
+      logger.error(error)
     }
   }
 
@@ -61,7 +62,7 @@ class LedgerService {
 
       return true
     } catch (error) {
-      //
+      logger.error(error)
     }
 
     return false

src/renderer/store/modules/ledger.js

@@ -1,6 +1,7 @@
 import ledgerService from '@/services/ledger-service'
 import eventBus from '@/plugins/event-bus'
 import { crypto } from '@arkecosystem/crypto'
+import logger from 'electron-log'
 
 export default {
   namespaced: true,
@@ -146,6 +147,7 @@ export default {
           try {
             wallet = await this._vm.$client.fetchWallet(ledgerAddress)
           } catch (error) {
+            logger.error(error)
             const message = error.response ? error.response.data.message : error.message
             if (message !== 'Wallet not found') {
               throw error
@@ -177,7 +179,7 @@ export default {
           }
         }
       } catch (error) {
-        //
+        logger.error(error)
       }
       commit('SET_WALLETS', [])
       eventBus.$emit('ledger:wallets-updated', [])
@@ -197,7 +199,7 @@ export default {
           accountIndex
         })
       } catch (error) {
-        //
+        logger.error(error)
       }
 
       return false
@@ -215,7 +217,7 @@ export default {
           accountIndex
         })
       } catch (error) {
-        //
+        logger.error(error)
       }
 
       return false
@@ -236,7 +238,7 @@ export default {
           data: transactionHex
         })
       } catch (error) {
-        //
+        logger.error(error)
       }
 
       return false