From cc9826077540dfdcd12ebe0f57470e7419f8c922 Mon Sep 17 00:00:00 2001
From: AuroraHuang22 <aurora90376@gmail.com>
Date: Thu, 26 Sep 2024 16:42:57 +0800
Subject: [PATCH 1/4] =?UTF-8?q?=F0=9F=A5=85=20Throw=20connection=20error?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 components/AppHeader.vue | 19 +++++++++++++++++--
 layouts/wallet.vue       | 17 +++++++++++++++--
 pages/auth/redirect.vue  |  8 ++++++--
 3 files changed, 38 insertions(+), 6 deletions(-)

diff --git a/components/AppHeader.vue b/components/AppHeader.vue
index de9b2ee4..b20d2267 100644
--- a/components/AppHeader.vue
+++ b/components/AppHeader.vue
@@ -152,6 +152,13 @@
     >
       <IconDangerStripe />
     </div>
+    <Snackbar
+      :open="shouldShowError"
+      class="mx-auto"
+      :text="errorMessage"
+      preset="warn"
+      @close="shouldShowError=false"
+    />
   </div>
 </template>
 
@@ -179,6 +186,8 @@ export default class AppHeader extends Vue {
   @bookApiModule.Action('clearSession') clearSession!: () => void
 
   isLoading = false
+  errorMessage: string | null = null
+  shouldShowError = false
 
   // eslint-disable-next-line class-methods-use-this
   get isTestnet() {
@@ -218,7 +227,9 @@ export default class AppHeader extends Vue {
         method,
       })
       await this.initWallet(connection)
-      if (!this.currentAddress || !this.signer) return
+      if (!this.currentAddress || !this.signer) {
+        throw new Error('FAILED_TO_CONNECT_WALLET')
+      }
       const signature = await this.signMessageMemo('authorize', [
         'profile',
         'read:nftbook',
@@ -226,7 +237,9 @@ export default class AppHeader extends Vue {
         'read:nftcollection',
         'write:nftcollection',
       ])
-      if (!signature) { return }
+      if (!signature) {
+        throw new Error('SIGNING_REJECTED')
+      }
       await this.authenticate({ inputWallet: this.currentAddress, signature })
     }
     } catch (error) {
@@ -234,6 +247,8 @@ export default class AppHeader extends Vue {
       this.clearSession()
       // eslint-disable-next-line no-console
       console.error('handleConnectWalletButtonClick error', error)
+      this.shouldShowError = true
+      this.errorMessage = error as string
     }
     finally {
       this.isLoading = false
diff --git a/layouts/wallet.vue b/layouts/wallet.vue
index 8773e5c4..1f5f6099 100644
--- a/layouts/wallet.vue
+++ b/layouts/wallet.vue
@@ -15,6 +15,13 @@
         <Label :text="$t('general.signIn.loading')" />
       </Card>
     </div>
+    <Snackbar
+      :open="shouldShowError"
+      class="mx-auto"
+      :text="errorMessage"
+      preset="warn"
+      @close="shouldShowError=false"
+    />
   </RootLayout>
 </template>
 
@@ -48,6 +55,8 @@ export default class WalletLayout extends Vue {
   ) => void
 
   isSignInLoading = false
+  errorMessage: string | null = null
+  shouldShowError = false
 
   get isHideFooter() {
     return this.$route.path.includes('/nft/purchase/') || this.$route.query.layout === 'popup';
@@ -77,7 +86,9 @@ export default class WalletLayout extends Vue {
             method,
           })
           await this.initWallet(connection)
-          if (!this.walletAddress || !this.signer) return
+          if (!this.walletAddress || !this.signer) {
+            throw new Error('FAILED_TO_CONNECT_WALLET')
+          }
           const signature = await this.signMessageMemo('authorize', [
             'profile',
             'read:nftbook',
@@ -86,7 +97,7 @@ export default class WalletLayout extends Vue {
             'write:nftcollection',
           ])
           if (!signature) {
-            return
+            throw new Error('SIGNING_REJECTED')
           }
           await this.authenticate({
             inputWallet: this.walletAddress,
@@ -101,6 +112,8 @@ export default class WalletLayout extends Vue {
         this.clearSession()
         // eslint-disable-next-line no-console
         console.error('handleConnectWalletButtonClick error', error)
+        this.shouldShowError = true
+        this.errorMessage = error as string
       }
       finally {
         this.isSignInLoading = false
diff --git a/pages/auth/redirect.vue b/pages/auth/redirect.vue
index 6580b47b..0df14d59 100644
--- a/pages/auth/redirect.vue
+++ b/pages/auth/redirect.vue
@@ -49,7 +49,9 @@ export default class RedirectPage extends Vue {
             method: method as string,
           })
           await this.initWallet(connection)
-          if (!this.currentAddress || !this.signer) return
+          if (!this.currentAddress || !this.signer) {
+            throw new Error('FAILED_TO_CONNECT_WALLET')
+          }
           const signature = await this.signMessageMemo('authorize', [
             'profile',
             'read:nftbook',
@@ -57,7 +59,9 @@ export default class RedirectPage extends Vue {
             'read:nftcollection',
             'write:nftcollection',
           ])
-          if (!signature) { return }
+          if (!signature) {
+            throw new Error('SIGNING_REJECTED')
+          }
           await this.authenticate({ inputWallet: this.currentAddress, signature })
         }
         let postAuthRoute = '/';

From 5c2c13370611651bc56f227ddcdf07466b7ea02a Mon Sep 17 00:00:00 2001
From: AuroraHuang22 <aurora90376@gmail.com>
Date: Thu, 26 Sep 2024 17:00:44 +0800
Subject: [PATCH 2/4] =?UTF-8?q?=F0=9F=8E=A8=20Add=20constant=20for=20autho?=
 =?UTF-8?q?rization=20permission?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 components/AppHeader.vue | 10 ++--------
 constant/index.ts        |  8 ++++++++
 layouts/wallet.vue       |  9 ++-------
 pages/auth/redirect.vue  |  9 ++-------
 4 files changed, 14 insertions(+), 22 deletions(-)

diff --git a/components/AppHeader.vue b/components/AppHeader.vue
index b20d2267..8983d3c8 100644
--- a/components/AppHeader.vue
+++ b/components/AppHeader.vue
@@ -167,7 +167,7 @@ import { Vue, Component } from 'vue-property-decorator'
 import { namespace } from 'vuex-class'
 import logTrackerEvent, { setLoggerUser } from '~/utils/logger'
 
-import { IS_TESTNET } from '~/constant'
+import { IS_TESTNET, SIGN_AUTHORIZATION_PERMISSIONS } from '~/constant'
 
 const walletModule = namespace('wallet')
 const bookApiModule = namespace('book-api')
@@ -230,13 +230,7 @@ export default class AppHeader extends Vue {
       if (!this.currentAddress || !this.signer) {
         throw new Error('FAILED_TO_CONNECT_WALLET')
       }
-      const signature = await this.signMessageMemo('authorize', [
-        'profile',
-        'read:nftbook',
-        'write:nftbook',
-        'read:nftcollection',
-        'write:nftcollection',
-      ])
+      const signature = await this.signMessageMemo('authorize', SIGN_AUTHORIZATION_PERMISSIONS)
       if (!signature) {
         throw new Error('SIGNING_REJECTED')
       }
diff --git a/constant/index.ts b/constant/index.ts
index e7ae619b..c74d90fc 100644
--- a/constant/index.ts
+++ b/constant/index.ts
@@ -111,3 +111,11 @@ export const LIKECOIN_CHAIN_STAKING_ENDPOINT = IS_TESTNET
 export const UPLOAD_FILESIZE_MAX = 200 * 1024 * 1024; // 200MB
 
 export const NFT_BOOK_PRESS_URL = IS_TESTNET ? 'https://likecoin-nft-book-press-testnet.netlify.app/' : 'https://likecoin.github.io/nft-book-press/';
+
+export const SIGN_AUTHORIZATION_PERMISSIONS = [
+  'profile',
+  'read:nftbook',
+  'write:nftbook',
+  'read:nftcollection',
+  'write:nftcollection',
+]
diff --git a/layouts/wallet.vue b/layouts/wallet.vue
index 1f5f6099..a72735c8 100644
--- a/layouts/wallet.vue
+++ b/layouts/wallet.vue
@@ -29,6 +29,7 @@
 import { Vue, Component, Watch } from 'vue-property-decorator'
 import { namespace } from 'vuex-class'
 import logTrackerEvent, { setLoggerUser } from '~/utils/logger'
+import { SIGN_AUTHORIZATION_PERMISSIONS } from '~/constant'
 
 
 const walletModule = namespace('wallet')
@@ -89,13 +90,7 @@ export default class WalletLayout extends Vue {
           if (!this.walletAddress || !this.signer) {
             throw new Error('FAILED_TO_CONNECT_WALLET')
           }
-          const signature = await this.signMessageMemo('authorize', [
-            'profile',
-            'read:nftbook',
-            'write:nftbook',
-            'read:nftcollection',
-            'write:nftcollection',
-          ])
+          const signature = await this.signMessageMemo('authorize', SIGN_AUTHORIZATION_PERMISSIONS)
           if (!signature) {
             throw new Error('SIGNING_REJECTED')
           }
diff --git a/pages/auth/redirect.vue b/pages/auth/redirect.vue
index 0df14d59..825100af 100644
--- a/pages/auth/redirect.vue
+++ b/pages/auth/redirect.vue
@@ -11,6 +11,7 @@
 import { Vue, Component } from 'vue-property-decorator'
 import { namespace } from 'vuex-class'
 import logTrackerEvent, { setLoggerUser } from '~/utils/logger'
+import { SIGN_AUTHORIZATION_PERMISSIONS } from '~/constant'
 
 const walletModule = namespace('wallet')
 const bookApiModule = namespace('book-api')
@@ -52,13 +53,7 @@ export default class RedirectPage extends Vue {
           if (!this.currentAddress || !this.signer) {
             throw new Error('FAILED_TO_CONNECT_WALLET')
           }
-          const signature = await this.signMessageMemo('authorize', [
-            'profile',
-            'read:nftbook',
-            'write:nftbook',
-            'read:nftcollection',
-            'write:nftcollection',
-          ])
+          const signature = await this.signMessageMemo('authorize', SIGN_AUTHORIZATION_PERMISSIONS)
           if (!signature) {
             throw new Error('SIGNING_REJECTED')
           }

From 11d36aca5a66a60714fe5a71bd340513cb7fad3b Mon Sep 17 00:00:00 2001
From: AuroraHuang22 <aurora90376@gmail.com>
Date: Thu, 26 Sep 2024 17:11:03 +0800
Subject: [PATCH 3/4] =?UTF-8?q?=F0=9F=A5=85=20Throw=20error=20with=20axios?=
 =?UTF-8?q?=20response=20data?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 store/book-api.ts | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/store/book-api.ts b/store/book-api.ts
index c898d535..1377c3d4 100644
--- a/store/book-api.ts
+++ b/store/book-api.ts
@@ -48,7 +48,11 @@ export default class BookAPI extends VuexModule {
     } catch (error) {
       // eslint-disable-next-line no-console
       console.error(error)
-      throw new Error('AUTHENTICATION_FAILED')
+      if (axios.isAxiosError(error)) {
+        throw new Error(`AUTHENTICATION_FAILED: ${error.response?.data}`)
+      }
+      throw new Error(`AUTHENTICATION_FAILED`)
+
     }
   }
 

From 3c503f279abf7736b110d71552bdd0c5ac72018e Mon Sep 17 00:00:00 2001
From: AuroraHuang22 <aurora90376@gmail.com>
Date: Thu, 26 Sep 2024 17:12:40 +0800
Subject: [PATCH 4/4] =?UTF-8?q?=F0=9F=94=A5=20Remove=20unused=20eslint-dis?=
 =?UTF-8?q?able=20comments?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 store/wallet.ts | 1 -
 1 file changed, 1 deletion(-)

diff --git a/store/wallet.ts b/store/wallet.ts
index 9169df79..4e4e18d2 100644
--- a/store/wallet.ts
+++ b/store/wallet.ts
@@ -221,7 +221,6 @@ export default class Wallet extends VuexModule {
   }
 
   @Action
-  // eslint-disable-next-line class-methods-use-this
   async signMessageMemo(action: string, permissions?: string[]) {
     if (!this.signer || !this.address) {
       await this.initIfNecessary()