Address copilot review comments

Author: 1Blademaster

gcs/src/missions.jsx

@@ -611,17 +611,17 @@ export default function Missions() {
             </p>
           )}
 
-          {(missionProgressModalData.progress !== null ||
-            missionProgressModalData !== undefined) && (
-            <Progress
-              color="lime"
-              animated
-              size="lg"
-              transitionDuration={300}
-              value={missionProgressModalData.progress * 100}
-              className="w-full mx-auto my-auto"
-            />
-          )}
+          {missionProgressModalData.progress !== null &&
+            missionProgressModalData.progress !== undefined && (
+              <Progress
+                color="lime"
+                animated
+                size="lg"
+                transitionDuration={300}
+                value={missionProgressModalData.progress * 100}
+                className="w-full mx-auto my-auto"
+              />
+            )}
         </div>
       </Modal>
 

radio/app/controllers/missionController.py

@@ -90,14 +90,17 @@ def _getCommandName(self, command: int) -> str:
             return f"Unknown command {command}"
 
     def getCurrentMission(
-        self, mission_type: int, progressUpdateCallback: Optional[Callable] = None
+        self,
+        mission_type: int,
+        progressUpdateCallback: Optional[Callable[[str, float], None]] = None,
     ) -> Response:
         """
         Get the current mission of a specific type from the drone.
 
         Args:
             mission_type (int): The type of mission to get. 0=Mission,1=Fence,2=Rally.
             progressUpdateCallback (Optional[Callable]): A callback function to update the progress of the mission fetch.
+                The callback should accept a string message and a float progress value.
         """
         mission_type_check = self._checkMissionType(mission_type)
         if not mission_type_check.get("success"):
@@ -159,14 +162,17 @@ def getCurrentMissionAll(self) -> Response:
         }
 
     def getMissionItems(
-        self, mission_type: int, progressUpdateCallback: Optional[Callable] = None
+        self,
+        mission_type: int,
+        progressUpdateCallback: Optional[Callable[[str, float], None]] = None,
     ) -> Response:
         """
         Get all mission items of a specific type from the drone.
 
         Args:
             mission_type (int): The type of mission to get. 0=Mission,1=Fence,2=Rally.
             progressUpdateCallback (Optional[Callable]): A callback function to update the progress of the mission fetch.
+                The callback should accept a string message and a float progress value.
         """
         mission_type_check = self._checkMissionType(mission_type)
         if not mission_type_check.get("success"):
@@ -537,7 +543,7 @@ def uploadMission(
         self,
         mission_type: int,
         waypoints: List[dict],
-        progressUpdateCallback: Optional[Callable] = None,
+        progressUpdateCallback: Optional[Callable[[str, float], None]] = None,
     ) -> Response:
         """
         Uploads the current mission to the drone. This method overwrites the current loader if the upload is successful.
@@ -546,6 +552,7 @@ def uploadMission(
             mission_type (int): The type of mission to upload. 0=Mission,1=Fence,2=Rally.
             waypoints (List[dict]): The list of waypoints to upload. Each waypoint should be a dict with the required fields.
             progressUpdateCallback (Optional[Callable]): A callback function to update the progress of the mission writing.
+                The callback should accept a string message and a float progress value.
         """
         mission_type_check = self._checkMissionType(mission_type)
         if not mission_type_check.get("success"):