CIRC 0.5 updates: stale indicators, disable go, and reset all#49
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the dashboard’s operational UX for CIRC 0.5 by adding “stale” status indicators (to surface missing telemetry), preventing arm motion commands when the arm isn’t idle, and adding a one-click “Reset All” action for motor error resets.
Changes:
- Add stale-status countdown/indicator to the node status panel.
- Add “Reset All” and stale-status handling to the motor status panel.
- Disable the Arm “Go” action unless a pose is selected and the arm is in an Idle state.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
src/components/panels/NodeStatusPanel.tsx |
Adds a global stale timer and UI indicator for node status updates. |
src/components/panels/MotorStatusPanel.tsx |
Adds “Reset All” plus per-motor stale countdown and updated LED/error display logic. |
src/components/panels/ArmControlPanel.tsx |
Prevents “Go” command unless the arm is Idle. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
I'm pretty rough at reviewing typescript. I tagged copilot to take a look. Double check the things it came up with. The busy timer is the one that scares me the most. |
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
|
Looks like it found what I couldn't, which was a way to do them all at once, so I feel better about it now |
No description provided.