From 4570a03b0c33a3e5a0de3d40bbc5a5a2d66ed58b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Eva=20Mill=C3=A1n?= <evamillan@bitergia.com>
Date: Wed, 6 Nov 2024 17:01:06 +0100
Subject: [PATCH] [ui] Skip recommendation
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Adds a button on the Review recommendations modal
to skip the current recommendation and load the
next one.

Signed-off-by: Eva Millán <evamillan@bitergia.com>
---
 releases/unreleased/skip-recommendations.yml |  9 +++++++++
 ui/src/components/Recommendations.vue        | 20 ++++++++++++++++----
 2 files changed, 25 insertions(+), 4 deletions(-)
 create mode 100644 releases/unreleased/skip-recommendations.yml

diff --git a/releases/unreleased/skip-recommendations.yml b/releases/unreleased/skip-recommendations.yml
new file mode 100644
index 00000000..e756b5f2
--- /dev/null
+++ b/releases/unreleased/skip-recommendations.yml
@@ -0,0 +1,9 @@
+---
+title: Skip recommendations
+category: added
+author: Eva Millán <evamillan@bitergia.com>
+issue: 937
+notes: >
+  Users can now skip a recommendation and review
+  the next one if they are unsure of whether to
+  apply it or dismiss it.
diff --git a/ui/src/components/Recommendations.vue b/ui/src/components/Recommendations.vue
index ae133930..9d9464e0 100644
--- a/ui/src/components/Recommendations.vue
+++ b/ui/src/components/Recommendations.vue
@@ -116,7 +116,14 @@
           {{ errorMessage }}
         </v-alert>
 
-        <v-card-actions class="pr-0 mt-4">
+        <v-card-actions class="px-0 mt-4">
+          <v-btn
+            color="primary darken-1"
+            variant="text"
+            @click.prevent="fetchItem(page + 1)"
+          >
+            Skip
+          </v-btn>
           <v-spacer></v-spacer>
           <v-btn
             color="primary darken-1"
@@ -158,6 +165,7 @@ export default {
       isOpen: false,
       errorMessage: null,
       menu: null,
+      page: 1,
     };
   },
   methods: {
@@ -169,9 +177,12 @@ export default {
         this.$logger.error(`Error fetching recommendations: ${error}`);
       }
     },
-    async fetchItem() {
+    async fetchItem(page = 1) {
+      if (this.currentItem && !this.currentItem.pageInfo.hasNext) {
+        return this.onClose();
+      }
       try {
-        const response = await this.getRecommendations(1, 1);
+        const response = await this.getRecommendations(page, 1);
         if (response.data.recommendedMerge.entities) {
           const recommendation = response.data.recommendedMerge.entities[0];
           this.currentItem = {
@@ -181,6 +192,7 @@ export default {
             pageInfo: response.data.recommendedMerge.pageInfo,
           };
           this.count = this.currentItem.pageInfo.totalResults;
+          this.page = this.currentItem.pageInfo.page;
         }
       } catch (error) {
         this.errorMessage = this.$getErrorMessage(error);
@@ -199,7 +211,7 @@ export default {
         if (!this.currentItem.pageInfo.hasNext) {
           this.onClose();
         } else {
-          this.fetchItem();
+          this.fetchItem(this.page);
         }
       } catch (error) {
         this.errorMessage = this.$getErrorMessage(error);