Move set top header visibility logic to task action page

Moves the actions dispatches to the `TaskAction` component, removing it from both the `Editor` and the `RapidEditor` components.

frontend/src/components/editor.js

@@ -31,16 +31,8 @@ export default function Editor({ setDisable, comment, presets, imagery, gpxUrl }
     }
   }, [customImageryIsSet, imagery, iDContext, customSource]);
 
-  useEffect(() => {
-    return () => {
-      dispatch({ type: 'SET_VISIBILITY', isVisible: true });
-    };
-    // eslint-disable-next-line
-  }, []);
-
   useEffect(() => {
     if (windowInit) {
-      dispatch({ type: 'SET_VISIBILITY', isVisible: false });
       if (iDContext === null) {
         // we need to keep iD context on redux store because iD works better if
         // the context is not restarted while running in the same browser session

frontend/src/components/rapidEditor.js

@@ -38,16 +38,8 @@ export default function RapidEditor({
     }
   }, [customImageryIsSet, imagery, RapiDContext, customSource]);
 
-  useEffect(() => {
-    return () => {
-      dispatch({ type: 'SET_VISIBILITY', isVisible: true });
-    };
-    // eslint-disable-next-line
-  }, []);
-
   useEffect(() => {
     if (windowInit) {
-      dispatch({ type: 'SET_VISIBILITY', isVisible: false });
       if (RapiDContext === null) {
         // we need to keep iD context on redux store because iD works better if
         // the context is not restarted while running in the same browser session

frontend/src/views/taskAction.js

@@ -1,5 +1,5 @@
 import React, { useState, useEffect } from 'react';
-import { useSelector } from 'react-redux';
+import { useDispatch, useSelector } from 'react-redux';
 import { useNavigate, useParams } from 'react-router-dom';
 import { useQueryParam, StringParam } from 'use-query-params';
 import { FormattedMessage } from 'react-intl';
@@ -25,6 +25,7 @@ export function ValidateTask() {
 
 export function TaskAction({ project, action }: Object) {
   const navigate = useNavigate();
+  const dispatch = useDispatch();
   const userDetails = useSelector((state) => state.auth.userDetails);
   const token = useSelector((state) => state.auth.token);
   const locale = useSelector((state) => state.preferences.locale);
@@ -33,14 +34,12 @@ export function TaskAction({ project, action }: Object) {
   const [tasks, setTasks] = useState([]);
   const [loading, setLoading] = useState(true);
 
-  const getTasks = () => {
-    fetchLocalJSONAPI(`users/queries/tasks/locked/details/`, token, 'GET', locale)
-      .then((res) => {
-        setTasks(res.tasks);
-        setLoading(false);
-      })
-      .catch((e) => navigate(`/projects/${project}/tasks/`));
-  };
+  useEffect(() => {
+    dispatch({ type: 'SET_VISIBILITY', isVisible: false });
+    return () => {
+      dispatch({ type: 'SET_VISIBILITY', isVisible: true });
+    };
+  }, [dispatch]);
 
   useEffect(() => {
     if (userDetails.id && token && action && project) {