From 1025345928221a9e187199f9d9a2b1042df3e53f Mon Sep 17 00:00:00 2001
From: Tom Russell <TR115251@hotmail.co.uk>
Date: Tue, 13 Aug 2024 16:41:11 +0100
Subject: [PATCH 1/6] virus scanner status on releases page

---
 src/views/ReleasesList.vue | 41 +++++++++++++++++++++++++++++++++++---
 1 file changed, 38 insertions(+), 3 deletions(-)

diff --git a/src/views/ReleasesList.vue b/src/views/ReleasesList.vue
index c966fffce..3fcf5aba9 100644
--- a/src/views/ReleasesList.vue
+++ b/src/views/ReleasesList.vue
@@ -1,10 +1,16 @@
 <template>
   <div>
-    <h1
-      class="govuk-heading-xl govuk-!-margin-bottom-6"
-    >
+    <h1 class="govuk-heading-xl govuk-!-margin-bottom-6">
       Latest Releases
     </h1>
+
+    <!-- New section for file upload status -->
+    <div>
+      <h2 class="govuk-heading-m">
+        Virus Scanner functionality: {{ fileUploadStatus }}
+      </h2>
+    </div>
+
     <Table
       data-key="id"
       :data="tableData"
@@ -35,6 +41,8 @@ import permissionMixin from '@/permissionMixin';
 import dayjs from 'dayjs';
 import Table from '@jac-uk/jac-kit/components/Table/Table.vue';
 import TableCell from '@jac-uk/jac-kit/components/Table/TableCell.vue';
+import { collection, doc, getDoc } from 'firebase/firestore';
+import { firestore } from '@/firebase';
 
 export default {
   name: 'ReleasesList',
@@ -51,15 +59,20 @@ export default {
         { title: 'Author' },
         { title: 'Published At' },
       ],
+      fileUploadEnabled: false, // To store the file upload status
     };
   },
   computed: {
     tableData() {
       return this.$store.state.releases.records;
     },
+    fileUploadStatus() {
+      return this.fileUploadEnabled ? 'Online' : 'Offline';
+    },
   },
   created() {
     this.$store.dispatch('releases/getLatestReleases');
+    this.fetchFileUploadStatus();
   },
   methods: {
     formatDate(dateStr) {
@@ -68,6 +81,28 @@ export default {
     getTableData(params) {
       this.$store.dispatch('releases/getLatestReleases', params);
     },
+    async fetchFileUploadStatus() {
+      try {
+        // Reference to the 'settings' collection
+        const settingsRef = collection(firestore, 'settings');
+
+        // Reference to the 'candidateSettings' subcollection within 'settings'
+        const candidateSettingsRef = collection(settingsRef, 'candidateSettings');
+
+        // Reference to the 'bypasses' subcollection within 'candidateSettings'
+        const bypassesRef = collection(candidateSettingsRef, 'bypasses');
+
+        // Reference to the 'fileUpload' document within 'bypasses'
+        const fileUploadDoc = doc(bypassesRef, 'fileUpload');
+
+        // Fetch the feature flag
+        const docSnapshot = await getDoc(fileUploadDoc);
+        this.fileUploadEnabled = docSnapshot.exists() ? docSnapshot.data().enabled : false;
+      } catch (error) {
+        console.error('Failed to fetch file upload status:', error);
+        this.fileUploadEnabled = false; // Default to false if there's an error
+      }
+    },
   },
 };
 </script>

From 2ced6ccff1beb7c0c9eed0cb1c5bd20b2a3ad4af Mon Sep 17 00:00:00 2001
From: Tom Russell <TR115251@hotmail.co.uk>
Date: Thu, 15 Aug 2024 15:31:21 +0100
Subject: [PATCH 2/6] working

---
 src/views/ReleasesList.vue | 15 +++------------
 1 file changed, 3 insertions(+), 12 deletions(-)

diff --git a/src/views/ReleasesList.vue b/src/views/ReleasesList.vue
index 3fcf5aba9..56ab416c9 100644
--- a/src/views/ReleasesList.vue
+++ b/src/views/ReleasesList.vue
@@ -84,20 +84,11 @@ export default {
     async fetchFileUploadStatus() {
       try {
         // Reference to the 'settings' collection
-        const settingsRef = collection(firestore, 'settings');
-
-        // Reference to the 'candidateSettings' subcollection within 'settings'
-        const candidateSettingsRef = collection(settingsRef, 'candidateSettings');
-
-        // Reference to the 'bypasses' subcollection within 'candidateSettings'
-        const bypassesRef = collection(candidateSettingsRef, 'bypasses');
-
-        // Reference to the 'fileUpload' document within 'bypasses'
-        const fileUploadDoc = doc(bypassesRef, 'fileUpload');
+        const settingsRef = doc(firestore, 'settings/candidateSettings');
 
         // Fetch the feature flag
-        const docSnapshot = await getDoc(fileUploadDoc);
-        this.fileUploadEnabled = docSnapshot.exists() ? docSnapshot.data().enabled : false;
+        const docSnapshot = await getDoc(settingsRef);
+        this.fileUploadEnabled = docSnapshot.exists() ? docSnapshot.data().fileUpload.enabled : false;
       } catch (error) {
         console.error('Failed to fetch file upload status:', error);
         this.fileUploadEnabled = false; // Default to false if there's an error

From cfc2374e682534c184a0fbee90b14166f27eb26b Mon Sep 17 00:00:00 2001
From: Tom Russell <TR115251@hotmail.co.uk>
Date: Thu, 15 Aug 2024 15:41:13 +0100
Subject: [PATCH 3/6] working

---
 src/views/ReleasesList.vue | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/views/ReleasesList.vue b/src/views/ReleasesList.vue
index 56ab416c9..fb6a2b36d 100644
--- a/src/views/ReleasesList.vue
+++ b/src/views/ReleasesList.vue
@@ -41,7 +41,7 @@ import permissionMixin from '@/permissionMixin';
 import dayjs from 'dayjs';
 import Table from '@jac-uk/jac-kit/components/Table/Table.vue';
 import TableCell from '@jac-uk/jac-kit/components/Table/TableCell.vue';
-import { collection, doc, getDoc } from 'firebase/firestore';
+import { doc, getDoc } from 'firebase/firestore';
 import { firestore } from '@/firebase';
 
 export default {

From 4fd5c45e39dd9f1a63efc7aaa55399dec910155d Mon Sep 17 00:00:00 2001
From: Tom Russell <TR115251@hotmail.co.uk>
Date: Wed, 21 Aug 2024 17:39:40 +0100
Subject: [PATCH 4/6] text size

---
 src/views/ReleasesList.vue | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/views/ReleasesList.vue b/src/views/ReleasesList.vue
index fb6a2b36d..fdcde7dd0 100644
--- a/src/views/ReleasesList.vue
+++ b/src/views/ReleasesList.vue
@@ -1,16 +1,16 @@
 <template>
   <div>
-    <h1 class="govuk-heading-xl govuk-!-margin-bottom-6">
-      Latest Releases
-    </h1>
-
     <!-- New section for file upload status -->
     <div>
-      <h2 class="govuk-heading-m">
+      <h2 class="govuk-heading-m govuk-!-margin-bottom-2">
         Virus Scanner functionality: {{ fileUploadStatus }}
       </h2>
     </div>
 
+    <h2 class="govuk-heading-m govuk-!-margin-bottom-2">
+      Latest Releases
+    </h2>
+
     <Table
       data-key="id"
       :data="tableData"

From c150ec87d5ca9ebd7098eb20c7e99b7fb0f021a9 Mon Sep 17 00:00:00 2001
From: drieJac <jac.drie@digitalteam.uk>
Date: Tue, 27 Aug 2024 13:18:16 +0100
Subject: [PATCH 5/6] Move file upload status calls into the store.

---
 src/store.js                   |  2 ++
 src/store/candidateSettings.js | 30 ++++++++++++++++++++++++++++++
 src/views/ReleasesList.vue     | 24 +++++++-----------------
 3 files changed, 39 insertions(+), 17 deletions(-)
 create mode 100644 src/store/candidateSettings.js

diff --git a/src/store.js b/src/store.js
index 4f3e97dbb..5b881c63e 100644
--- a/src/store.js
+++ b/src/store.js
@@ -48,6 +48,7 @@ import bugReports from '@/store/bugReports/collection';
 import bugReport from '@/store/bugReports/document';
 
 import releases from '@/store/releases';
+import candidateSettings from '@/store/candidateSettings';
 
 //const store = new Vuex.Store({
 const store = createStore({
@@ -94,6 +95,7 @@ const store = createStore({
     vacancy,
     bugReport,
     bugReports,
+    candidateSettings,
   },
   state: {
     packageVersion: import.meta.env.PACKAGE_VERSION || '0',
diff --git a/src/store/candidateSettings.js b/src/store/candidateSettings.js
new file mode 100644
index 000000000..19091761a
--- /dev/null
+++ b/src/store/candidateSettings.js
@@ -0,0 +1,30 @@
+import { doc } from '@firebase/firestore';
+import { firestore } from '@/firebase';
+import { firestoreAction } from '@/helpers/vuexfireJAC';
+import vuexfireSerialize from '@jac-uk/jac-kit/helpers/vuexfireSerialize';
+
+export default {
+  namespaced: true,
+  actions: {
+    bind: firestoreAction(({ bindFirestoreRef }) => {
+      const firestoreRef = doc(firestore, 'settings/candidateSettings');
+      return bindFirestoreRef('record', firestoreRef, { serialize: vuexfireSerialize });
+    }),
+    unbind: firestoreAction(({ unbindFirestoreRef }) => {
+      return unbindFirestoreRef('record');
+    }),
+  },
+  mutations: {
+    set(state, { name, value }) {
+      state[name] = value;
+    },
+  },
+  state: {
+    record: null,
+  },
+  getters: {
+    getUploadStatus: (state) => {
+      return state.record && state.record.fileUpload.enabled;
+    },
+  },
+};
diff --git a/src/views/ReleasesList.vue b/src/views/ReleasesList.vue
index fdcde7dd0..ef1e63cf5 100644
--- a/src/views/ReleasesList.vue
+++ b/src/views/ReleasesList.vue
@@ -41,8 +41,7 @@ import permissionMixin from '@/permissionMixin';
 import dayjs from 'dayjs';
 import Table from '@jac-uk/jac-kit/components/Table/Table.vue';
 import TableCell from '@jac-uk/jac-kit/components/Table/TableCell.vue';
-import { doc, getDoc } from 'firebase/firestore';
-import { firestore } from '@/firebase';
+import { mapGetters } from 'vuex';
 
 export default {
   name: 'ReleasesList',
@@ -59,10 +58,13 @@ export default {
         { title: 'Author' },
         { title: 'Published At' },
       ],
-      fileUploadEnabled: false, // To store the file upload status
     };
   },
   computed: {
+    ...mapGetters({
+      fileUploadEnabled: 'candidateSettings/getUploadStatus',
+    }),
+
     tableData() {
       return this.$store.state.releases.records;
     },
@@ -71,8 +73,9 @@ export default {
     },
   },
   created() {
+    // Can make the calls synchronously below
     this.$store.dispatch('releases/getLatestReleases');
-    this.fetchFileUploadStatus();
+    this.$store.dispatch('candidateSettings/bind');
   },
   methods: {
     formatDate(dateStr) {
@@ -81,19 +84,6 @@ export default {
     getTableData(params) {
       this.$store.dispatch('releases/getLatestReleases', params);
     },
-    async fetchFileUploadStatus() {
-      try {
-        // Reference to the 'settings' collection
-        const settingsRef = doc(firestore, 'settings/candidateSettings');
-
-        // Fetch the feature flag
-        const docSnapshot = await getDoc(settingsRef);
-        this.fileUploadEnabled = docSnapshot.exists() ? docSnapshot.data().fileUpload.enabled : false;
-      } catch (error) {
-        console.error('Failed to fetch file upload status:', error);
-        this.fileUploadEnabled = false; // Default to false if there's an error
-      }
-    },
   },
 };
 </script>

From 80d02aa1bc64f0ceebe4fa2beb30205601d195df Mon Sep 17 00:00:00 2001
From: drieJac <jac.drie@digitalteam.uk>
Date: Tue, 27 Aug 2024 13:39:33 +0100
Subject: [PATCH 6/6] Fixed typo after merge

---
 src/views/ReleasesList.vue | 1 -
 1 file changed, 1 deletion(-)

diff --git a/src/views/ReleasesList.vue b/src/views/ReleasesList.vue
index f676b89c4..44c845696 100644
--- a/src/views/ReleasesList.vue
+++ b/src/views/ReleasesList.vue
@@ -70,7 +70,6 @@ export default {
   computed: {
     ...mapGetters({
       fileUploadEnabled: 'candidateSettings/getUploadStatus',
-    }),
       lastFetchedDT: 'releases/getLastFetchedDT',
     }),
     ...mapState('releases', [