-
-
Notifications
You must be signed in to change notification settings - Fork 7.6k
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
Adding note to assist troubleshooting entities unavailable due to BAF fan losing connection to remote #37075
base: current
Are you sure you want to change the base?
Conversation
A helpful troubleshooting tip to resolve issues with BAF climate entity and some additional sensors. See home-assistant/core#130402
✅ Deploy Preview for home-assistant-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
📝 WalkthroughWalkthroughA new note has been added to the documentation for the Big Ass Fans (BAF) integration, addressing the availability of climate and climate sensors for models featuring Auto Comfort. It clarifies that these sensors are supplied by the remote control rather than the fan itself. The note also indicates that if the fan loses connection to the remote, these entities may become unavailable while the fan continues to operate. Users are advised to consult vendor documentation for reconnecting the fan to the remote and to reload the integration to restore entity availability. Changes
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (1)
source/_integrations/baf.markdown (1)
49-51
: Fix formatting and grammar in the note block.The content is helpful and aligns well with the troubleshooting objectives. However, there are a few minor issues to address:
Apply these changes to improve the formatting and readability:
-{% note %} -**Climate and Climate Sensors become unavailable:**: Climate and some sensors like temperature are provided by the remote not the fan itself on certain models. When the fan loses connection to the remote, these entities may become unavailable while the fan remains available. Consult vendor documentation on how to re-establish connectivity between the fan and remote and reload the integration to restore entities availability. -{% endnote %} +{% note %} +**Climate and Climate Sensors Become Unavailable:** Climate and some sensors like temperature are provided by the remote, not the fan itself on certain models. When the fan loses connection to the remote, these entities may become unavailable while the fan remains available. Consult vendor documentation on how to re-establish connectivity between the fan and remote and reload the integration to restore entities availability. +{% endnote %}Changes made:
- Removed double colon in heading
- Fixed capitalization in heading
- Added missing comma after "remote"
🧰 Tools
🪛 LanguageTool
[uncategorized] ~50-~50: A comma might be missing here.
Context: ...rs like temperature are provided by the remote not the fan itself on certain models. W...(AI_EN_LECTOR_MISSING_PUNCTUATION_COMMA)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
source/_integrations/baf.markdown
(1 hunks)
🧰 Additional context used
🪛 LanguageTool
source/_integrations/baf.markdown
[uncategorized] ~50-~50: A comma might be missing here.
Context: ...rs like temperature are provided by the remote not the fan itself on certain models. W...
(AI_EN_LECTOR_MISSING_PUNCTUATION_COMMA)
⏰ Context from checks skipped due to timeout of 90000ms (3)
- GitHub Check: Redirect rules - home-assistant-docs
- GitHub Check: Header rules - home-assistant-docs
- GitHub Check: Pages changed - home-assistant-docs
@coderabbitai review |
@@ -46,6 +46,10 @@ For devices that support Auto Comfort and are running firmware 3.1 or later, an | |||
|
|||
For devices that support Auto Comfort, a climate {% term entity %} allows adjusting the target temperature. | |||
|
|||
{% note %} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe it's worth adding a Troubleshooting section to this page. It's too far out for me to make a suggestion at the end of the page, but maybe something like this:
Troubleshooting
Climate
Symptom: Climate and climate sensors are unavailable
The climate sensors have been successfully integrated into Home Assistant, but they have become unavailable after a while.
Description
On certain models, climate and some sensors like temperature are provided by the remote, not the fan itself. When the fan loses connection to the remote, these entities may become unavailable while the fan remains available.
Resolution
- Consult vendor documentation on how to re-establish connectivity between the fan and remote.
- Reload the integration to restore entity availability.
Proposed change
Type of change
current
branch).current
branch).next
branch).next
branch).Additional information
Checklist
current
branch.next
branch.Summary by CodeRabbit