cleanup after review

Author: PhilBastian

src/Frontend/src/stores/ConnectionsAndStatsStore.ts

@@ -18,10 +18,7 @@ export const useConnectionsAndStatsStore = defineStore("ConnectionsAndStatsStore
 
   const { count: requiresFullFailureDetailsSubscriberCount, inc, dec } = useCounter(0);
   function requiresFullFailureDetails() {
-    onMounted(() => {
-      inc();
-      refresh();
-    });
+    onMounted(() => inc()); //NOTE: not forcing a refresh here since we expect the view utilising this store to also setup a refresh on mount
     onUnmounted(() => dec());
   }
 

src/Frontend/src/stores/MessagesStore.ts

@@ -176,7 +176,7 @@ export const useMessagesStore = defineStore("MessagesStore", () => {
 
   async function setSort(sort: string, direction?: SortDirection) {
     if (controller) {
-      // need to cancel any existing fetch which otherwise will set messages of the incorrect status
+      // need to cancel any existing fetch which otherwise will set messages of the incorrect sort
       controller.abort();
     }
     sortBy.value = sort;
@@ -187,7 +187,7 @@ export const useMessagesStore = defineStore("MessagesStore", () => {
 
   async function setPeriod(period: DeletedPeriodOption | RetryPeriodOption) {
     if (controller) {
-      // need to cancel any existing fetch which otherwise will set messages of the incorrect status
+      // need to cancel any existing fetch which otherwise will set messages of the incorrect period
       controller.abort();
     }
     selectedPeriod.value = period;