-
-
Notifications
You must be signed in to change notification settings - Fork 7.7k
Add Dobiss documentation #39529
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
base: next
Are you sure you want to change the base?
Add Dobiss documentation #39529
Conversation
It seems that this PR is targeted against an incorrect branch. Documentation updates which apply to our current stable release should target the |
✅ Deploy Preview for home-assistant-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
📝 WalkthroughWalkthroughA new documentation file for the Dobiss smart home integration has been introduced. This file provides metadata and a brief description of the Dobiss integration, specifying its configuration, supported platform, and related documentation links. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant HomeAssistant
participant DobissIntegration
User->>HomeAssistant: Initiate configuration flow
HomeAssistant->>DobissIntegration: Load integration metadata
DobissIntegration-->>HomeAssistant: Provide supported platform and details
HomeAssistant-->>User: Display configuration options and documentation
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. 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 (2)
source/_integrations/dobiss.markdown (2)
19-20
: Consider wording for clarity.
You might replace “into Home Assistant” with “with Home Assistant” to improve the phrasing of the integration description.
21-22
: Add a Platforms section for usage details.
To help users understand how to work with thelight
entities exposed by Dobiss, consider adding a “## Platforms” header after the config flow snippet and briefly documenting available services, entity names, and example usages.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
Cache: Disabled due to data retention organization setting
Knowledge Base: Disabled due to data retention organization setting
📒 Files selected for processing (1)
source/_integrations/dobiss.markdown
(1 hunks)
⏰ 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
🔇 Additional comments (1)
source/_integrations/dobiss.markdown (1)
1-17
: Front matter metadata looks good.
YAML configuration (title, description, code owners, config flow support, category, IoT class, release, domain, platforms, integration type, and related link) aligns with Home Assistant documentation standards.
- docs: /docs/configuration/ | ||
title: 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.
- docs: /docs/configuration/ | |
title: Configuration file |
title: Configuration file | ||
--- | ||
|
||
The `dobiss` {% term integration %} allows you to integrate your [Dobiss NXT](https://www.dobiss.com/) smart home system into Home Assistant. |
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.
The `dobiss` {% term integration %} allows you to integrate your [Dobiss NXT](https://www.dobiss.com/) smart home system into Home Assistant. | |
The **Dobiss** {% term integration %} allows you to integrate your [Dobiss NXT](https://www.dobiss.com/) smart home system into Home Assistant. |
--- | ||
|
||
The `dobiss` {% term integration %} allows you to integrate your [Dobiss NXT](https://www.dobiss.com/) smart home system into Home Assistant. | ||
|
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.
are there any prerequisites? do I need something before I can install the integration? a dociss account, an app, log in credentials?
- docs: /docs/configuration/ | ||
title: 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.
is there anything else you can say about this integration? what is the supported functionality? what sensors, devices are supported?
Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍 |
Proposed change
This PR add the documentation for the Dobiss integration that will be committed as well.
Type of change
current
branch).current
branch).next
branch).next
branch).Additional information
Checklist
current
branch.next
branch.Summary by CodeRabbit