new project modal state set as closed to avoid fluttering in & out (#286

)
hms-dbmi-cellenics · May 28, 2021 · e52d7ed · e52d7ed
1 parent aa4ca95
commit e52d7ed
Showing 1 changed file with 9 additions and 4 deletions.
diff --git a/src/pages/data-management/index.jsx b/src/pages/data-management/index.jsx
@@ -25,9 +25,9 @@ const DataManagementPage = ({ route }) => {
   const {
     saving: sampleSaving,
   } = useSelector((state) => state.samples.meta);
-  const [newProjectModalVisible, setNewProjectModalVisible] = useState(true);
+  const [newProjectModalVisible, setNewProjectModalVisible] = useState(false);
   const experiments = useSelector((state) => state.experiments);
-  const activeProjectUuid = useSelector((state) => state.projects.meta.activeProjectUuid);
+  const { activeProjectUuid, loading: projectsLoading } = useSelector((state) => state.projects.meta);
   const activeProject = projectsList[activeProjectUuid];
 
   const existingExperiments = activeProject?.experiments
@@ -54,10 +54,15 @@ const DataManagementPage = ({ route }) => {
   }, [activeProject]);
 
   useEffect(() => {
-    if (projectsList.ids.length) {
+    if (projectsLoading === true) {
+      return;
+    }
+    if (projectsList.ids.length === 0) {
+      setNewProjectModalVisible(true);
+    } else {
       setNewProjectModalVisible(false);
     }
-  }, [projectsList]);
+  }, [projectsList, projectsLoading]);
 
   const unnamedExperimentName = 'Unnamed Analysis';