Fix loading logic of "new project" dialog

hms-dbmi-cellenics · May 25, 2021 · 43a9acf · 43a9acf
1 parent 5d71b02
commit 43a9acf
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/src/pages/data-management/index.jsx b/src/pages/data-management/index.jsx
@@ -20,11 +20,13 @@ const DataManagementPage = ({ route }) => {
   const projectsList = useSelector(((state) => state.projects));
   const {
     saving: projectSaving,
+    loading: projectLoading,
+    error: projectError,
   } = projectsList.meta;
   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 activeProject = projectsList[activeProjectUuid];
@@ -45,8 +47,8 @@ const DataManagementPage = ({ route }) => {
   }, [activeProject]);
 
   useEffect(() => {
-    if (projectsList.ids.length) {
-      setNewProjectModalVisible(false);
+    if (!projectLoading.loading && !projectError.error && !projectsList.ids.length) {
+      setNewProjectModalVisible(true);
     }
   }, [projectsList]);