Skip to content

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

Merged
merged 10 commits into from
Jul 10, 2025

Conversation

NorthernMan54
Copy link
Contributor

To fake the UI into thinking you are running in a Docker container, set these environment variables -

declare -x DOCKER_HOMEBRIDGE_VERSION="2025-06-04"
declare -x HOMEBRIDGE_CONFIG_UI="1"

@github-actions github-actions bot added the beta Related to Beta Branch label Jul 10, 2025
@NorthernMan54 NorthernMan54 marked this pull request as draft July 10, 2025 14:08
@NorthernMan54 NorthernMan54 requested a review from Copilot July 10, 2025 14:08
Copilot

This comment was marked as outdated.

@NorthernMan54 NorthernMan54 requested a review from Copilot July 10, 2025 15:01
Copilot

This comment was marked as outdated.

@NorthernMan54 NorthernMan54 marked this pull request as ready for review July 10, 2025 15:04
Copilot

This comment was marked as outdated.

@NorthernMan54 NorthernMan54 requested a review from Copilot July 10, 2025 21:56
Copilot

This comment was marked as outdated.

@NorthernMan54 NorthernMan54 requested a review from Copilot July 10, 2025 22:04
Copy link

@Copilot Copilot AI left a 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 add import { InformationComponent } from '@/app/core/components/information/information.component'.
    const ref = this.$modal.open(InformationComponent, {

@NorthernMan54 NorthernMan54 merged commit a4a9ae4 into beta-5.0.0 Jul 10, 2025
3 checks passed
@NorthernMan54 NorthernMan54 deleted the Docker-Version branch July 10, 2025 22:09
bwp91 added a commit that referenced this pull request Jul 11, 2025
Co-Authored-By: Northern Man <19808920+NorthernMan54@users.noreply.github.com>
bwp91 added a commit that referenced this pull request Jul 12, 2025
Co-Authored-By: Northern Man <19808920+NorthernMan54@users.noreply.github.com>
bwp91 added a commit that referenced this pull request Jul 12, 2025
Co-Authored-By: Northern Man <19808920+NorthernMan54@users.noreply.github.com>
bwp91 added a commit that referenced this pull request Jul 12, 2025
Co-Authored-By: Northern Man <19808920+NorthernMan54@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
beta Related to Beta Branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants