Use warning for dialog, and added todo to use authz

Signed-off-by: Aaron Chong <[email protected]>
open-rmf · May 6, 2024 · 253ab34 · 253ab34
1 parent 6eb5777
commit 253ab34
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/packages/api-server/api_server/routes/building_map.py b/packages/api-server/api_server/routes/building_map.py
@@ -46,6 +46,7 @@ async def get_previous_fire_alarm_trigger():
 async def reset_fire_alarm_trigger(
     logger: LoggerAdapter = Depends(get_logger),
 ):
+    # TODO: enforce with authz
     logger.info("Fire alarm trigger reset requested")
     rmf_gateway().reset_fire_alarm_trigger()
     fire_alarm_trigger_state = FireAlarmTriggerState(

diff --git a/packages/dashboard/src/components/appbar.tsx b/packages/dashboard/src/components/appbar.tsx
@@ -709,9 +709,9 @@ export const AppBar = React.memo(({ extraToolbarItems }: AppBarProps): React.Rea
           onClose={() => setOpenFireAlarmTriggerResetDialog(false)}
           onSubmit={handleResetFireAlarmTrigger}
         >
-          <Typography>
-            Please ensure that all other systems are back online and that it is safe to resume robot
-            operations.
+          <Typography color="warning.main">
+            Warning: Please ensure that all other systems are back online and that it is safe to
+            resume robot operations.
           </Typography>
         </ConfirmationDialog>
       )}