fix: build warning in useDataset.ts

- always use the global dataset object rather than local state.
- remove a debugging log.

Author: eatyourgreens

frontend-v2/src/features/trial/Protocols.tsx

@@ -49,7 +49,6 @@ const Protocols: FC = () => {
   );
 
   const { dataset } = useDataset(selectedProject);
-  console.log('trials', dataset)
 
   const loading = [isProjectLoading, isProtocolsLoading, unitsLoading].some(
     (x) => x,

frontend-v2/src/hooks/useDataset.ts

@@ -19,6 +19,10 @@ export default function useDataset(selectedProject: number | null) {
     createDataset
   ] = useDatasetCreateMutation();
 
+  if (dataset !== appDataset) {
+    setDataset(appDataset);
+  }
+
   useEffect(function onDataLoad() {
     async function addDataset() {
       let [newDataset] = datasets;
@@ -34,7 +38,7 @@ export default function useDataset(selectedProject: number | null) {
         }
       }
       appDataset = newDataset;
-      setDataset(newDataset);
+      setDataset(appDataset);
     }
     if (selectedProjectOrZero && !isDatasetLoading) {
       addDataset();
@@ -43,11 +47,11 @@ export default function useDataset(selectedProject: number | null) {
 
   const updateDataset = useCallback((newDataset: DatasetRead) => {
     appDataset = newDataset;
-    setDataset(newDataset);
+    setDataset(appDataset);
   }, []);
 
   return {
-    dataset: appDataset,
+    dataset,
     updateDataset
   };
 }