Merge pull request #40 from rkshaon/rkshaon

updt: frontend - page number twice api called
rkshaon · Sep 13, 2024 · be41e91 · be41e91
2 parents 6039c61 + 83fc1e2
commit be41e91
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/frontend/src/pages/HomePage.vue b/frontend/src/pages/HomePage.vue
@@ -51,13 +51,17 @@ export default {
     }
   },
   mounted () {
-    this.fetchBooks()
+    // this.fetchBooks()
+    const currentPage = this.$route.query.page || 1
+    if (!this.$route.query.page) {
+      this.fetchBooks({ page: currentPage, pageSize: this.pageSize })
+    }
   },
   methods: {
     ...mapActions(['fetchBooks']),
     changePage (page) {
       this.$router.push({ query: { page } })
-      this.fetchBooks({ page, pageSize: this.pageSize })
+      // this.fetchBooks({ page, pageSize: this.pageSize })
     }
   }
 }

diff --git a/frontend/src/services/v1/bookAPIService.js b/frontend/src/services/v1/bookAPIService.js
@@ -6,8 +6,6 @@ const API_BASE_URL = process.env.VUE_APP_BACKEND_URL
 
 export default {
   async fetchV1Books (page = 1, pageSize = 8) {
-    // console.log(page, pageSize);
-
     page = page ?? 1
     pageSize = pageSize ?? 8
     console.log(page, pageSize)