From c9a8cab8feca9863ca4b677b669f026c9231a85e Mon Sep 17 00:00:00 2001
From: Jont828 <jt572@cornell.edu>
Date: Tue, 15 Aug 2023 16:43:12 -0400
Subject: [PATCH] Fetch version using mixin on page load to avoid lag

---
 web/src/components/SettingsCard.vue | 18 +++-------
 web/src/mixins/setVersion.js        | 24 +++++++++++++
 web/src/views/DescribeCluster.vue   | 53 ++++++++++++++++-------------
 web/src/views/ManagementCluster.vue | 24 +++++++------
 4 files changed, 70 insertions(+), 49 deletions(-)
 create mode 100644 web/src/mixins/setVersion.js

diff --git a/web/src/components/SettingsCard.vue b/web/src/components/SettingsCard.vue
index f6f39d2..6e40ee9 100644
--- a/web/src/components/SettingsCard.vue
+++ b/web/src/components/SettingsCard.vue
@@ -31,7 +31,7 @@
           <v-list-item-title>Version</v-list-item-title>
         </v-list-item-content>
         <v-list-item-action>
-          {{ gitVersion }}
+          {{ version }}
         </v-list-item-action>
       </v-list-item>
       <v-list-item
@@ -143,23 +143,14 @@ import { useSettingsStore } from "../stores/settings.js";
 export default {
   name: "SettingsCard",
   components: {},
-  async beforeMount() {
-    await this.fetchVersion();
+  props: {
+    version: String,
   },
   methods: {
     toggleDarkTheme(val) {
       this.$vuetify.theme.dark = val;
     },
-    async fetchVersion() {
-      const response = await Vue.axios.get("/version");
-      if (response.data == null) {
-        console.error("failed getting git version info");
-        return;
-      }
-
-      console.log("Git version is", response.data);
-      this.gitVersion = response.data.gitVersion;
-    }
+    // TODO: Do this once on page load so this doesn't take a minute and lag.
   },
   setup() {
     const store = useSettingsStore();
@@ -170,7 +161,6 @@ export default {
     return {
       fileTypes: ["YAML", "JSON"],
       pollingInterval: ["1s", "5s", "10s", "30s", "1m", "5m", "Off"],
-      gitVersion: "",
     };
   },
 };
diff --git a/web/src/mixins/setVersion.js b/web/src/mixins/setVersion.js
new file mode 100644
index 0000000..d07d637
--- /dev/null
+++ b/web/src/mixins/setVersion.js
@@ -0,0 +1,24 @@
+import Vue from "vue";
+
+export const setVersion = {
+  data() {
+    return {
+      gitVersion: ""
+    }
+  },
+  async beforeMount() {
+    await this.fetchVersion();
+  },
+  methods: {
+    async fetchVersion() {
+      const response = await Vue.axios.get("/version");
+      if (response.data == null) {
+        console.error("Failed getting git version info");
+        return;
+      }
+
+      console.log("Git version:", response.data);
+      this.gitVersion = response.data.gitVersion;
+    }
+  }
+}
\ No newline at end of file
diff --git a/web/src/views/DescribeCluster.vue b/web/src/views/DescribeCluster.vue
index 1e45af0..1d1c9a8 100644
--- a/web/src/views/DescribeCluster.vue
+++ b/web/src/views/DescribeCluster.vue
@@ -69,6 +69,7 @@
       <SettingsCard
         @close="() => { showSettingsOverlay = !showSettingsOverlay }"
         class="settingsCard"
+        :version="gitVersion"
       />
     </v-overlay>
   </div>
@@ -84,6 +85,7 @@ import AlertError from "../components/AlertError.vue";
 import SettingsCard from "../components/SettingsCard.vue";
 
 import { useSettingsStore } from "../stores/settings.js";
+import { setVersion } from "../mixins/setVersion.js";
 
 import _ from "lodash";
 import colors from "vuetify/lib/util/colors";
@@ -97,12 +99,39 @@ export default {
     CustomResourceDefinition,
     AlertError,
   },
+  mixins: [setVersion],
+  data() {
+    return {
+      showSettingsOverlay: false,
+      showAboutOverlay: false,
+      alert: false,
+      errorMessage: "",
+      treeIsReady: false,
+      resourceIsReady: false,
+      resource: {},
+      selected: {},
+      treeData: {},
+      cachedTreeString: "",
+      treeConfig: { nodeWidth: 180, nodeHeight: 50, levelHeight: 120 },
+      scale: 1,
+      legend: {
+        cluster: "Cluster API",
+        bootstrap: "Bootstrap Provider",
+        controlplane: "Control Plane Provider",
+        infrastructure: "Infrastructure Provider",
+        addons: "Add-ons",
+        virtual: "None",
+      },
+      gitVersion: ""
+    };
+  },
   setup() {
     const store = useSettingsStore();
     return { store };
   },
   async beforeMount() {
     await this.fetchCluster();
+    await this.fetchVersion();
   },
   computed: {
     theme() {
@@ -306,30 +335,6 @@ export default {
       );
     },
   },
-  data() {
-    return {
-      showSettingsOverlay: false,
-      showAboutOverlay: false,
-      alert: false,
-      errorMessage: "",
-      treeIsReady: false,
-      resourceIsReady: false,
-      resource: {},
-      selected: {},
-      treeData: {},
-      cachedTreeString: "",
-      treeConfig: { nodeWidth: 180, nodeHeight: 50, levelHeight: 120 },
-      scale: 1,
-      legend: {
-        cluster: "Cluster API",
-        bootstrap: "Bootstrap Provider",
-        controlplane: "Control Plane Provider",
-        infrastructure: "Infrastructure Provider",
-        addons: "Add-Ons",
-        virtual: "None",
-      },
-    };
-  },
 };
 </script>
 
diff --git a/web/src/views/ManagementCluster.vue b/web/src/views/ManagementCluster.vue
index fb716b1..f3bde11 100644
--- a/web/src/views/ManagementCluster.vue
+++ b/web/src/views/ManagementCluster.vue
@@ -37,7 +37,9 @@ import Vue from "vue";
 import ManagementClusterTree from "../components/ManagementClusterTree.vue";
 import SettingsCard from "../components/SettingsCard.vue";
 import AppBar from "../components/AppBar.vue";
+
 import { useSettingsStore } from "../stores/settings.js";
+import { setVersion } from "../mixins/setVersion.js";
 
 export default {
   name: "ManagementCluster",
@@ -46,6 +48,17 @@ export default {
     SettingsCard,
     AppBar,
   },
+  mixins: [setVersion],
+  data() {
+    return {
+      showSettingsOverlay: false,
+      treeConfig: { nodeWidth: 300, nodeHeight: 140, levelHeight: 275 },
+      treeData: {},
+      cachedTreeString: "",
+      treeIsReady: false,
+      scale: 1,
+    };
+  },
   setup() {
     const store = useSettingsStore();
     return { store };
@@ -66,17 +79,6 @@ export default {
     this.selected = {};
     clearInterval(this.polling);
   },
-  data() {
-    return {
-      showSettingsOverlay: false,
-      treeConfig: { nodeWidth: 300, nodeHeight: 140, levelHeight: 275 },
-      treeData: {},
-      cachedTreeString: "",
-      treeIsReady: false,
-      scale: 1,
-      gitVersion: "",
-    };
-  },
   watch: {
     "store.selectedInterval": function (val) {
       console.log("Overview store.selectedInterval: " + val);