fix: topup wallet was showing NaN (#2504)

* fix: topup wallet was showing NaN

lnbits/static/js/base.js

@@ -145,7 +145,7 @@ window.LNbits = {
       )
     },
     updateBalance: function (credit, wallet_id) {
-      return LNbits.api.request('PUT', '/users/api/v1/topup/', null, {
+      return LNbits.api.request('PUT', '/users/api/v1/topup', null, {
         amount: credit,
         id: wallet_id
       })

lnbits/static/js/components.js

@@ -702,7 +702,14 @@ Vue.component('lnbits-update-balance', {
       LNbits.api
         .updateBalance(credit, this.wallet_id)
         .then(res => {
-          this.callback({value: res, wallet_id: this.wallet_id})
+          if (res.data.status !== 'Success') {
+            throw new Error(res.data)
+          }
+          this.callback({
+            success: true,
+            credit: parseInt(credit),
+            wallet_id: this.wallet_id
+          })
         })
         .then(_ => {
           credit = parseInt(credit)

lnbits/static/js/users.js

@@ -351,12 +351,14 @@ new Vue({
       console.log('export users')
     },
     topupCallback(res) {
-      this.wallets.forEach(wallet => {
-        if (res.wallet_id === wallet.id) {
-          wallet.balance_msat += res.value * 1000
-        }
-      })
-      this.fetchUsers()
+      if (res.success) {
+        this.wallets.forEach(wallet => {
+          if (res.wallet_id === wallet.id) {
+            wallet.balance_msat += res.credit * 1000
+          }
+        })
+        this.fetchUsers()
+      }
     },
     topupWallet() {
       LNbits.api

lnbits/static/js/wallet.js

@@ -535,7 +535,9 @@ new Vue({
         .catch(e => console.error(e))
     },
     updateBalanceCallback: function (res) {
-      this.balance += res.value
+      if (res.success && wallet.id === res.wallet_id) {
+        this.balance += res.credit
+      }
     },
     pasteToTextArea: function () {
       this.$refs.textArea.focus() // Set cursor to textarea