Hammer/generic alerts

[aaronchongth]

What's new

This works with open-rmf/rmf_internal_msgs#71

Replaces the existing task alert system to be much more flexible and generic, with the key purpose to support the smart cart API server interaction.

Test with regular task

• task complete
• failed dispatch

Test with fleet alerts

ros2 topic pub /fleet_alert rmf_fleet_msgs/msg/FleetAlert '{id: test2, title: title, subtitle: subtitle, message: message, display: true, tier: 0, responses_available: [move_off, timeout], alert_parameters: [{name: waiting_at, value: test_location}], task_id: test_task}' --once

What's next:

• Migrate delivery_alerts to use this as well
• Frontend for new task types

Self-checks

• I have prototyped this new feature (if necessary) on Figma
• I'm familiar with and follow this Typescript guideline
• I added unit-tests for new components
• I tried testing edge cases
• I tested the behavior of the components that interact with the backend, with an e2e test

[codecov]

Codecov Report

Attention: Patch coverage is 1.32450% with 149 lines in your changes missing coverage. Please review.

Please upload report for BASE (deploy/hammer-humble@3649b5e). Learn more about missing BASE report.

Files	Patch %	Lines
...ackages/dashboard/src/components/alert-manager.tsx	0.00%	134 Missing ⚠️
packages/dashboard/src/components/appbar.tsx	20.00%	8 Missing ⚠️
...shboard/src/components/tasks/task-cancellation.tsx	0.00%	7 Missing ⚠️

Additional details and impacted files

@@                   Coverage Diff                   @@
##             deploy/hammer-humble     #948   +/-   ##
=======================================================
  Coverage                        ?   13.55%           
=======================================================
  Files                           ?       66           
  Lines                           ?     3099           
  Branches                        ?      911           
=======================================================
  Hits                            ?      420           
  Misses                          ?     2673           
  Partials                        ?        6           

Flag	Coverage Δ
dashboard	13.55% <1.32%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[koonpeng]

Haven't look through the whole thing yet.

[koonpeng]

not sure if this image is still updated, there are multiple issues with the design

1. The number of available responses is unbounded, better to list them in a dropdown.
2. The buttons are different size.

[koonpeng]

A suggestion for the alert responses is to use split buttons https://mui.com/material-ui/react-button-group/#split-button