From 65454e01f7563d483162c7d07f463622709cc79c Mon Sep 17 00:00:00 2001
From: Assunta DeSanto <assuntad23@gmail.com>
Date: Thu, 7 Dec 2023 20:30:41 -0500
Subject: [PATCH 1/3] disable wf report and generate PDF buttons if WF is not
 complete or failed

---
 .../WorkflowInvocationSummary.vue             | 27 ++++++++++++++++++-
 1 file changed, 26 insertions(+), 1 deletion(-)

diff --git a/client/src/components/WorkflowInvocationState/WorkflowInvocationSummary.vue b/client/src/components/WorkflowInvocationState/WorkflowInvocationSummary.vue
index 3b30faf87d73..2a83a2a7a325 100644
--- a/client/src/components/WorkflowInvocationState/WorkflowInvocationSummary.vue
+++ b/client/src/components/WorkflowInvocationState/WorkflowInvocationSummary.vue
@@ -2,11 +2,19 @@
     <div class="mb-3 workflow-invocation-state-component">
         <div v-if="invocationAndJobTerminal">
             <span>
-                <b-button v-b-tooltip.hover size="sm" class="invocation-report-link" :href="invocationLink">
+                <b-button
+                    v-b-tooltip.hover
+                    :title="invocationStateSuccess ? reportTooltip : disabledReportTooltip"
+                    :disabled="!invocationStateSuccess"
+                    size="sm"
+                    class="invocation-report-link"
+                    :href="invocationLink">
                     View Report
                 </b-button>
                 <b-button
                     v-b-tooltip.hover
+                    :title="invocationStateSuccess ? generatePdfTooltip : disabledPdfTooltip"
+                    :disabled="!invocationStateSuccess"
                     size="sm"
                     class="invocation-pdf-link"
                     :href="invocationPdfLink"
@@ -109,6 +117,12 @@ export default {
         return {
             stepStatesInterval: null,
             jobStatesInterval: null,
+            reportTooltip: "View report for this workflow invocation",
+            generatePdfTooltip: "Generate PDF report for this workflow invocation",
+            disabledReportTooltip:
+                "Unable to create report because this workflow invocation is not complete or was not successful.",
+            disabledPdfTooltip:
+                "Unable to generate PDF because this workflow invocation is not complete or was not successful.",
         };
     },
     computed: {
@@ -126,6 +140,17 @@ export default {
         invocationState: function () {
             return this.invocation?.state || "new";
         },
+        invocationStateSuccess: function () {
+            return (
+                this.invocationState == "scheduled" &&
+                this.errorCount === 0 &&
+                this.runningCount === 0 &&
+                this.invocationAndJobTerminal
+            );
+        },
+        displayTooltip: function () {
+            return this.invocationStateSuccess ? "" : this.disabledTooltip;
+        },
         stepCount: function () {
             return this.invocation?.steps.length;
         },

From fe5966c0d92af4266d2761d8ec1d59b3546f2d19 Mon Sep 17 00:00:00 2001
From: Assunta DeSanto <assuntad23@gmail.com>
Date: Fri, 8 Dec 2023 09:40:27 -0500
Subject: [PATCH 2/3] Update
 client/src/components/WorkflowInvocationState/WorkflowInvocationSummary.vue

Co-authored-by: Marius van den Beek <m.vandenbeek@gmail.com>
---
 .../WorkflowInvocationState/WorkflowInvocationSummary.vue      | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/client/src/components/WorkflowInvocationState/WorkflowInvocationSummary.vue b/client/src/components/WorkflowInvocationState/WorkflowInvocationSummary.vue
index 2a83a2a7a325..0bd650e786a7 100644
--- a/client/src/components/WorkflowInvocationState/WorkflowInvocationSummary.vue
+++ b/client/src/components/WorkflowInvocationState/WorkflowInvocationSummary.vue
@@ -148,9 +148,6 @@ export default {
                 this.invocationAndJobTerminal
             );
         },
-        displayTooltip: function () {
-            return this.invocationStateSuccess ? "" : this.disabledTooltip;
-        },
         stepCount: function () {
             return this.invocation?.steps.length;
         },

From fd36deddfb0920a2f6d5609d1175c62b0a6298f2 Mon Sep 17 00:00:00 2001
From: Assunta DeSanto <assuntad23@gmail.com>
Date: Fri, 8 Dec 2023 19:31:22 -0500
Subject: [PATCH 3/3] changing disabled tooltip to be more detailed for users

---
 .../WorkflowInvocationSummary.vue             | 32 ++++++++++++-------
 1 file changed, 21 insertions(+), 11 deletions(-)

diff --git a/client/src/components/WorkflowInvocationState/WorkflowInvocationSummary.vue b/client/src/components/WorkflowInvocationState/WorkflowInvocationSummary.vue
index 0bd650e786a7..bf70ae7b2899 100644
--- a/client/src/components/WorkflowInvocationState/WorkflowInvocationSummary.vue
+++ b/client/src/components/WorkflowInvocationState/WorkflowInvocationSummary.vue
@@ -13,7 +13,7 @@
                 </b-button>
                 <b-button
                     v-b-tooltip.hover
-                    :title="invocationStateSuccess ? generatePdfTooltip : disabledPdfTooltip"
+                    :title="invocationStateSuccess ? generatePdfTooltip : disabledReportTooltip"
                     :disabled="!invocationStateSuccess"
                     size="sm"
                     class="invocation-pdf-link"
@@ -119,10 +119,6 @@ export default {
             jobStatesInterval: null,
             reportTooltip: "View report for this workflow invocation",
             generatePdfTooltip: "Generate PDF report for this workflow invocation",
-            disabledReportTooltip:
-                "Unable to create report because this workflow invocation is not complete or was not successful.",
-            disabledPdfTooltip:
-                "Unable to generate PDF because this workflow invocation is not complete or was not successful.",
         };
     },
     computed: {
@@ -141,12 +137,26 @@ export default {
             return this.invocation?.state || "new";
         },
         invocationStateSuccess: function () {
-            return (
-                this.invocationState == "scheduled" &&
-                this.errorCount === 0 &&
-                this.runningCount === 0 &&
-                this.invocationAndJobTerminal
-            );
+            return this.invocationState == "scheduled" && this.runningCount === 0 && this.invocationAndJobTerminal;
+        },
+        disabledReportTooltip: function () {
+            const state = this.invocationState;
+            const runCount = this.runningCount;
+            if (this.invocationState != "scheduled") {
+                return (
+                    "This workflow is not currently scheduled. The current state is ",
+                    state,
+                    ". Once the workflow is fully scheduled and jobs have complete this option will become available."
+                );
+            } else if (runCount != 0) {
+                return (
+                    "The workflow invocation still contains ",
+                    runCount,
+                    " running job(s). Once these jobs have completed this option will become available. "
+                );
+            } else {
+                return "Steps for this workflow are still running. A report will be available once complete.";
+            }
         },
         stepCount: function () {
             return this.invocation?.steps.length;