-
Notifications
You must be signed in to change notification settings - Fork 402
Display Docker Version Details in the Update Information Box #2488
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
ui/src/app/core/components/information/information.component.ts
Outdated
Show resolved
Hide resolved
ui/src/app/modules/status/widgets/update-info-widget/update-info-widget.component.ts
Show resolved
Hide resolved
ui/src/app/modules/status/widgets/update-info-widget/update-info-widget.component.ts
Show resolved
Hide resolved
ui/src/app/modules/status/widgets/update-info-widget/update-info-widget.component.ts
Outdated
Show resolved
Hide resolved
ui/src/app/modules/status/widgets/update-info-widget/update-info-widget.component.ts
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds Docker container version checking to the Update Information widget, including fetching release notes and displaying them in a modal.
- Adds new translation keys for Docker status and update messaging
- Integrates DockerDetails in the update-info component with expand/collapse UI
- Implements backend service and WebSocket gateway to fetch Docker release metadata
Reviewed Changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 3 comments.
Show a summary per file
File | Description |
---|---|
ui/src/i18n/en.json | Added keys for “unknown” status and Docker update title/message |
ui/src/app/modules/status/widgets/update-info-widget/...ts | Introduced DockerDetails , getDockerInfo() , and modal logic |
ui/src/app/modules/status/widgets/update-info-widget/...html | Updated template to render Docker status block and expand UI |
ui/src/app/core/components/information/information.component.ts | Imported Markdown support and added markdownMessage2 input |
ui/src/app/core/components/information/information.component.html | Renders release notes as markdown |
ui/src/app/core/components/information/information.component.scss | Styles .plugin-md markdown container |
src/modules/status/status.service.ts | Added getDockerDetails() and getRecentReleases() methods |
src/modules/status/status.gateway.ts | Subscribes to 'docker-version-check' messages |
src/modules/platform-tools/docker/docker.service.ts | Wrapped getStartupScript() in try/catch |
Comments suppressed due to low confidence (1)
ui/src/app/modules/status/widgets/update-info-widget/update-info-widget.component.ts:220
InformationComponent
is referenced here but not imported at the top of the file. Please addimport { InformationComponent } from '@/app/core/components/information/information.component'
.
const ref = this.$modal.open(InformationComponent, {
ui/src/app/modules/status/widgets/update-info-widget/update-info-widget.component.ts
Show resolved
Hide resolved
ui/src/app/modules/status/widgets/update-info-widget/update-info-widget.component.html
Show resolved
Hide resolved
Co-Authored-By: Northern Man <19808920+NorthernMan54@users.noreply.github.com>
Co-Authored-By: Northern Man <19808920+NorthernMan54@users.noreply.github.com>
Co-Authored-By: Northern Man <19808920+NorthernMan54@users.noreply.github.com>
Co-Authored-By: Northern Man <19808920+NorthernMan54@users.noreply.github.com>
To fake the UI into thinking you are running in a Docker container, set these environment variables -