feat: add rover update #6
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
There was a problem hiding this comment.
Pull Request Overview
This PR adds a Docker image update feature to the web UI, allowing users to pull the latest versions of three system images (rover, web-ui, and container-launcher) directly from the interface. It integrates react-hot-toast for real-time feedback and makes the SetResetPanel component more flexible with customizable button labels.
Key Changes:
- Added a new "Update system" button that triggers parallel Docker image pulls with toast notifications for progress and status
- Made
SetResetPanelbutton labels configurable via new optional propsbutton1Nameandbutton2Name - Integrated
react-hot-toastlibrary (v2.6.0) for user feedback with loading, success, and error states
Reviewed Changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| src/components/SetResetPanel.tsx | Added optional props for customizable button labels with default values |
| src/components/ContainerList.tsx | Replaced reset functionality with Docker image pull workflow, added toast notifications and confirmation dialog |
| src/app/launch/page.tsx | Added Toaster component to display toast notifications |
| package.json | Added react-hot-toast dependency (^2.6.0) |
| package-lock.json | Added react-hot-toast and goober dependency entries with resolved versions |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
ConnorNeed
force-pushed
the
update-buttor
branch
from
04f5450 to
46ae8fb
Compare
ErikCald
reviewed
Nov 14, 2025
| "integrity": "sha512-b/ZCF6amfAUb7dJM/MxRs7AetQEahYzJ8PtgfrmEdtw6uyGOr+ZSGtgjFm6mfsBkxJ4d2W7kg+Nlqzqvn3Bc0w==", | ||
| "license": "MIT" | ||
| }, | ||
| "node_modules/goober": { |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request introduces a user-friendly way to update Docker images from the UI, adds toast notifications for feedback, and improves button labeling for clarity. The most important changes are grouped below.
Feature Addition: Docker Image Update Workflow
SetResetPanelinContainerList.tsxthat triggers pulling the latest versions of three Docker images (cprtsoftware/rover:amd64,cprtsoftware/web-ui:latest, andcprtsoftware/container-launcher:latest) from the backend. The process provides confirmation and feedback to the user. [1] [2]User Feedback Improvements
react-hot-toastto display real-time notifications for each Docker image pull operation, including loading, success, and error states, as well as a final "All pulls finished" message. [1] [2] [3] [4]UI/UX Enhancements
SetResetPanelto support customizable button names via new propsbutton1Nameandbutton2Name, allowing for more descriptive labels such as "Update system" instead of the generic "Reset". [1] [2] [3]