Merge pull request #29 from Visual-Regression-Tracker/75-navigate-by-url

url navigation fixed

Author: pashidlos

src/components/BuildList.tsx

@@ -102,7 +102,7 @@ const BuildList: FunctionComponent = () => {
                 onClick={() => {
                   deleteBuild(buildDispatch, build.id)
                     .then((b) =>
-                      enqueueSnackbar(`${b.id} removed`, {
+                      enqueueSnackbar(`${b.id} deleted`, {
                         variant: "success",
                       })
                     )

src/pages/ProjectPage.tsx

@@ -73,7 +73,6 @@ const ProjectPage = () => {
   const buildDispatch = useBuildDispatch();
   const {
     testRuns,
-    selectedTestRunId,
     selectedTestRunIndex,
   } = useTestRunState();
   const testRunDispatch = useTestRunDispatch();
@@ -109,21 +108,19 @@ const ProjectPage = () => {
 
   useEffect(() => {
     const queryParams = getQueryParams(location.search);
-    if (!selectedBuildId) {
-      if (queryParams.buildId) {
-        selectBuild(buildDispatch, queryParams.buildId);
-      } else if (buildList.length > 0) {
-        selectBuild(buildDispatch, buildList[0].id);
-      }
+    if (queryParams.buildId) {
+      selectBuild(buildDispatch, queryParams.buildId);
+    } else if (buildList.length > 0) {
+      selectBuild(buildDispatch, buildList[0].id);
     }
-  }, [buildDispatch, buildList, location.search, selectedBuildId]);
+  }, [buildDispatch, buildList, location.search]);
 
   useEffect(() => {
     const queryParams = getQueryParams(location.search);
-    if (!selectedTestRunId && queryParams.testId) {
+    if (queryParams.testId) {
       selectTestRun(testRunDispatch, queryParams.testId);
     }
-  }, [location.search, testRuns, selectedTestRunId, testRunDispatch]);
+  }, [location.search, testRuns, testRunDispatch]);
 
   useEffect(() => {
     setFilteredTestRuns(