Skip to content

Adds documentation for airpatrol integration#40160

Merged
c0ffeeca7 merged 2 commits intohome-assistant:nextfrom
antondalgren:airpatrol-next
Dec 15, 2025
Merged

Adds documentation for airpatrol integration#40160
c0ffeeca7 merged 2 commits intohome-assistant:nextfrom
antondalgren:airpatrol-next

Conversation

@antondalgren
Copy link
Copy Markdown
Contributor

@antondalgren antondalgren commented Jul 27, 2025

Proposed change

Adds documentation for airpatrol integration home-assistant/core#149247

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

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 July 27, 2025 13:56
@home-assistant home-assistant Bot added has-parent This PR has a parent PR in another repo next This PR goes into the next branch labels Jul 27, 2025
@netlify
Copy link
Copy Markdown

netlify Bot commented Jul 27, 2025

Deploy Preview for home-assistant-docs ready!

Name Link
🔨 Latest commit 6e266ab
🔍 Latest deploy log https://app.netlify.com/projects/home-assistant-docs/deploys/693acb69b1cbd70008d46e60
😎 Deploy Preview https://deploy-preview-40160--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.

Copy link
Copy Markdown
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 adds comprehensive documentation for the new Airpatrol integration, which enables Home Assistant users to control air conditioning units through Airpatrol WiFi controllers. The documentation covers setup prerequisites, available entities, and configuration requirements.

Key changes:

  • Creates complete integration documentation with setup instructions and entity descriptions
  • Documents climate and sensor platform capabilities
  • Includes important prerequisites and configuration details

Comment thread source/_integrations/airpatrol.markdown Outdated
Comment thread source/_integrations/airpatrol.markdown Outdated
Comment thread source/_integrations/airpatrol.markdown Outdated
@c0ffeeca7 c0ffeeca7 changed the title feat: Adds documentation for airpatrol integration Adds documentation for airpatrol integration Jul 28, 2025
@c0ffeeca7 c0ffeeca7 added the new-integration This PR adds documentation for a new Home Assistant integration label Jul 28, 2025
Comment thread source/_integrations/airpatrol.markdown Outdated
Comment thread source/_integrations/airpatrol.markdown Outdated
Comment thread source/_integrations/airpatrol.markdown
Comment thread source/_integrations/airpatrol.markdown Outdated
Comment thread source/_integrations/airpatrol.markdown Outdated
Comment thread source/_integrations/airpatrol.markdown
Comment thread source/_integrations/airpatrol.markdown Outdated
Comment thread source/_integrations/airpatrol.markdown Outdated
Comment thread source/_integrations/airpatrol.markdown Outdated
Comment thread source/_integrations/airpatrol.markdown Outdated
Comment thread source/_integrations/airpatrol.markdown Outdated
Comment thread source/_integrations/airpatrol.markdown Outdated
@c0ffeeca7 c0ffeeca7 marked this pull request as draft July 28, 2025 12:14
@c0ffeeca7
Copy link
Copy Markdown
Contributor

I'm drafting this PR, as changes are needed. Please undraft it, once the changes have been implemented and the PR is ready for review again.

@antondalgren antondalgren marked this pull request as ready for review July 28, 2025 12:19
Comment thread source/_integrations/airpatrol.markdown
Comment thread source/_integrations/airpatrol.markdown
Comment thread source/_integrations/airpatrol.markdown Outdated
c0ffeeca7
c0ffeeca7 previously approved these changes Jul 28, 2025
Copy link
Copy Markdown
Contributor

@c0ffeeca7 c0ffeeca7 left a comment

Choose a reason for hiding this comment

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

Thank you, @antondalgren 👍

@c0ffeeca7 c0ffeeca7 added the awaits-parent Awaits the merge of an parent PR label Jul 28, 2025
@c0ffeeca7
Copy link
Copy Markdown
Contributor

Nice first docs PR

@antondalgren
Copy link
Copy Markdown
Contributor Author

Nice first docs PR

Thanks @c0ffeeca7!

Any clue on when I can expect a review on the parent PR?

Comment on lines +39 to +45
### Climate

The integration will create a climate entity for each air conditioning system found. The climate entity allows you to control:

- **HVAC mode**: Set the operation mode, such as off, heat, and cool.
- **Target temperature**: Set the desired temperature for heating or cooling.
- **Fan mode**: Control the fan speed (if supported by your system).
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Does not align with parent PR

@home-assistant home-assistant Bot marked this pull request as draft August 17, 2025 19:14
@home-assistant
Copy link
Copy Markdown

Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍

Learn more about our pull request process.

@antondalgren
Copy link
Copy Markdown
Contributor Author

Looks like I managed to happen in some weird state. Sorry! Will try to sort it out 🙈

@c0ffeeca7
Copy link
Copy Markdown
Contributor

c0ffeeca7 commented Oct 8, 2025

For future PRs: when you are done editing and ready for review, please mark the PR as Ready for review. Reviewers don't usually look at PRs that are in draft mode.

See also:
https://developers.home-assistant.io/docs/review-process#prs-are-being-drafted-when-changes-are-needed

image

Ooh, I see. Maybe you were still working on that PR. Don't mind my comment, then.

@c0ffeeca7 c0ffeeca7 marked this pull request as ready for review October 8, 2025 05:08
@home-assistant home-assistant Bot requested a review from gjohansson-ST October 8, 2025 05:08
@c0ffeeca7
Copy link
Copy Markdown
Contributor

Please implement the review comments mentioned here: #40160 (comment)

@c0ffeeca7 c0ffeeca7 marked this pull request as draft October 8, 2025 05:14
@antondalgren
Copy link
Copy Markdown
Contributor Author

Please implement the review comments mentioned here: #40160 (comment)

That comment is outdated as the parent PR is a climate entity and the documentation in this PR is also climate.

@antondalgren antondalgren marked this pull request as ready for review December 9, 2025 19:18
@home-assistant home-assistant Bot added the parent-merged The parent PR has been merged already label Dec 9, 2025
Comment thread source/_integrations/airpatrol.markdown Outdated
Comment thread source/_integrations/airpatrol.markdown Outdated
@home-assistant home-assistant Bot marked this pull request as draft December 9, 2025 20:27
@antondalgren antondalgren marked this pull request as ready for review December 11, 2025 13:47
@home-assistant home-assistant Bot requested a review from joostlek December 11, 2025 13:47
@antondalgren
Copy link
Copy Markdown
Contributor Author

Can we align home-assistant/core#158726 and antondalgren#1 with this PR? @joostlek

Copy link
Copy Markdown
Contributor

@c0ffeeca7 c0ffeeca7 left a comment

Choose a reason for hiding this comment

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

Thank you, @antondalgren 👍

@c0ffeeca7 c0ffeeca7 dismissed stale reviews from joostlek and gjohansson-ST December 15, 2025 11:44

requested changes have been implemented

@c0ffeeca7 c0ffeeca7 merged commit ea8c1a9 into home-assistant:next Dec 15, 2025
7 checks passed
@home-assistant home-assistant Bot removed awaits-parent Awaits the merge of an parent PR parent-merged The parent PR has been merged already labels Dec 15, 2025
@github-actions github-actions Bot locked and limited conversation to collaborators Dec 16, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

has-parent This PR has a parent PR in another repo new-integration This PR adds documentation for a new Home Assistant integration next This PR goes into the next branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants