Skip to content

Conversation

@pujitm
Copy link
Member

@pujitm pujitm commented Jan 8, 2026

Summary

  • Rename "Visit" buttons to "WebUI" for consistent terminology across Docker container UI
  • Use getWebUiUrl instead of getFirstLanIp to display the correct WebUI link

Test plan

  • Verify Docker containers with a WebUI show the "WebUI" button
  • Clicking "WebUI" opens the correct URL in a new tab
  • "WebUI (Tailscale)" button appears for Tailscale-enabled containers

Summary by CodeRabbit

  • Style
    • Updated button and action labels across Docker container interface. Changed "Visit" to "WebUI" and "Web UI" to "WebUI" for consistent naming in container actions and Tailscale variants.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 8, 2026

Walkthrough

The pull request updates UI labels across Docker container-related components, changing "Visit" to "WebUI" in action buttons and related text. Additionally, a utility function import was changed from getFirstLanIp to getWebUiUrl in the container overview component.

Changes

Cohort / File(s) Summary
Web UI Label Standardization
web/src/components/Docker/ContainerOverviewCard.vue, web/src/components/LayoutViews/Card/CardItem.vue, web/src/composables/useDockerRowActions.ts
Updated action button labels from "Visit" to "WebUI" and "Visit (Tailscale)" to "WebUI (Tailscale)" across container action UI components. Also swapped utility function from getFirstLanIp to getWebUiUrl and renamed corresponding computed property in ContainerOverviewCard.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 A rabbit hops with glee, so bright,
Old labels fade to WebUI's light,
From "Visit" calls to web-bound dreams,
New names flow through the Docker streams!

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes both main changes: renaming "Visit" to "WebUI" and switching to the correct WebUI link using getWebUiUrl instead of getFirstLanIp.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

Comment @coderabbitai help to get the list of available commands and usage tips.

@codecov
Copy link

codecov bot commented Jan 8, 2026

Codecov Report

❌ Patch coverage is 0% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 46.48%. Comparing base (9ef1cf1) to head (ff5b205).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
...eb/src/components/Docker/ContainerOverviewCard.vue 0.00% 4 Missing ⚠️
web/src/composables/useDockerRowActions.ts 0.00% 2 Missing ⚠️
web/src/components/LayoutViews/Card/CardItem.vue 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1863      +/-   ##
==========================================
- Coverage   46.49%   46.48%   -0.01%     
==========================================
  Files         954      954              
  Lines       59788    59788              
  Branches     5552     5548       -4     
==========================================
- Hits        27799    27795       -4     
- Misses      31870    31874       +4     
  Partials      119      119              

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions
Copy link
Contributor

github-actions bot commented Jan 8, 2026

This plugin has been deployed to Cloudflare R2 and is available for testing.
Download it at this URL:

https://preview.dl.unraid.net/unraid-api/tag/PR1863/dynamix.unraid.net.plg

@pujitm pujitm requested a review from elibosley January 8, 2026 18:59
@pujitm pujitm merged commit cab0880 into main Jan 8, 2026
17 of 20 checks passed
@pujitm pujitm deleted the fix/docker-webui branch January 8, 2026 19:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants