Skip to content

Remove duplicate Removing the integration section from Brother documentation #41619

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
Merged
merged 3 commits into from
Nov 5, 2025
Merged

Remove duplicate Removing the integration section from Brother documentation #41619

merged 3 commits into from
Nov 5, 2025

Conversation

@bieniu
Copy link
Member

@bieniu bieniu commented Nov 5, 2025

Proposed change

Removes duplicate Removing the integration section.

obraz

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).
  • 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:
  • Link to parent pull request in the Brands repository:
  • This PR fixes or closes issue: fixes #

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 AI review requested due to automatic review settings November 5, 2025 09:20
@home-assistant home-assistant bot added the current This PR goes into the current branch label Nov 5, 2025
@netlify
Copy link

netlify bot commented Nov 5, 2025 •
edited
Loading

✅ Deploy Preview for home-assistant-docs ready!

Name Link
🔨 Latest commit 687293a
🔍 Latest deploy log https://app.netlify.com/projects/home-assistant-docs/deploys/690b45c49ccf550008efd9f6
😎 Deploy Preview https://deploy-preview-41619--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 AI reviewed Nov 5, 2025
View reviewed changes
Copy link
Contributor

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 removes a duplicate "Removing the integration" section from the Brother integration documentation. The Brother integration documentation had two identical "Removing the integration" sections - one appearing early in the document (after configuration options) and another at the end (after "Known limitations"). This change eliminates the duplicate section at the end of the document, keeping only the earlier section to maintain proper documentation structure.

@bieniu bieniu mentioned this pull request Nov 5, 2025
Draft
75 tasks
@klaasnicolaas
Copy link
Member

klaasnicolaas commented Nov 5, 2025

If you leave the bottom state and remove the top one, it will be more in line with how we describe the removal process (in terms of order) in all integrations.

./Klaas

@thecode thecode merged commit 08e0917 into home-assistant:current Nov 5, 2025
7 checks passed
@bieniu bieniu deleted the brother-remove-duplicated-section branch November 5, 2025 13:53
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

@thecode thecode thecode approved these changes

+1 more reviewer

@mik-laj mik-laj mik-laj approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

current This PR goes into the current branch

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@bieniu @klaasnicolaas @thecode @mik-laj