Skip to content

docs: add port control section to UniFi integration documentation #40349

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: current
Choose a base branch
from
Open

docs: add port control section to UniFi integration documentation #40349

wants to merge 1 commit into from

Conversation

Tomeroeni
Copy link

Proposed change

Added documentation for the new port control feature in the UniFi Network integration. This feature allows users to enable or disable individual switch ports on UniFi switches, which is separate from the existing PoE port control functionality. The documentation includes a new "Port control" section under the Switch category that follows the same format and style as other switch features in the integration.

Type of change

  • Spelling, grammar or other readability improvements (current branch).
  • Adjusted missing or incorrect information in the current documentation (current branch).
  • Added documentation for a new integration I'm adding to Home Assistant (next branch).
  • I've opened up a PR to add logos and icons in Brands repository.
  • Added documentation for a new feature I'm adding to Home Assistant (next branch).
  • Removed stale or deprecated documentation.

Additional information

  • Link to parent pull request in the codebase: [Add the link to your Home Assistant core PR that implements the port control feature]
  • Link to parent pull request in the Brands repository: N/A
  • This PR fixes or closes issue: fixes #[issue number if applicable]

Checklist

  • This PR uses the correct branch, based on one of the following:
    • I made a change to the existing documentation and used the current branch.
    • I made a change that is related to an upcoming version of Home Assistant and used the next branch.
  • The documentation follows the Home Assistant documentation standards.

@Copilot Copilot AI review requested due to automatic review settings August 9, 2025 08:32
@Tomeroeni Tomeroeni requested a review from Kane610 as a code owner August 9, 2025 08:32
@home-assistant home-assistant bot added the current This PR goes into the current branch label Aug 9, 2025
@netlify Netlify
Copy link

netlify bot commented Aug 9, 2025 •
edited
Loading

✅ Deploy Preview for home-assistant-docs ready!

Name Link
🔨 Latest commit d08403b
🔍 Latest deploy log https://app.netlify.com/projects/home-assistant-docs/deploys/68970788680cc7000889ed36
😎 Deploy Preview https://deploy-preview-40349--home-assistant-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copilot
Copilot AI reviewed Aug 9, 2025
View reviewed changes
Copy link
Contributor

@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 documentation for a new port control feature in the UniFi Network integration. The feature allows users to enable or disable individual switch ports on UniFi switches, which is distinct from the existing PoE port control functionality.

  • Adds a new "Port control" section under the Switch category
  • Documents the feature's default entity state and admin privilege requirements
  • Follows the established documentation format used by other UniFi switch features

source/_integrations/unifi.markdown
@@ -141,6 +141,10 @@ Allow control of network access to clients configured in the {% term integration

Provides per-port PoE control. Entities are disabled by default. This feature requires admin privileges.

### Port control

Provides individual control to enable or disable switch ports. Entities are enabled by default. This feature requires admin privileges.
Copy link
Preview

Copilot AI Aug 9, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The sentence structure could be improved for better readability. Consider revising to: 'Allows you to individually enable or disable switch ports. Entities are enabled by default. This feature requires admin privileges.'

Copilot generated this review using guidance from copilot-instructions.md.

@Tomeroeni Tomeroeni mentioned this pull request Aug 9, 2025
Draft
19 tasks
@silamon silamon added next This PR goes into the next branch and removed current This PR goes into the current branch labels Aug 9, 2025
@silamon silamon closed this Aug 9, 2025
@silamon silamon reopened this Aug 9, 2025
@silamon
Copy link
Contributor

silamon commented Aug 9, 2025

New features should go to the next branch, not current.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

Copilot code review Copilot Copilot left review comments

@Kane610 Kane610 Awaiting requested review from Kane610 Kane610 is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
next This PR goes into the next branch
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Tomeroeni @silamon