-
-
Notifications
You must be signed in to change notification settings - Fork 7.6k
Add DD7006A Smart Home bridge as supported by Motion Blinds #35012
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
Conversation
📝 WalkthroughWalkthroughThe changes introduce the Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant Motionblinds
participant Bridge
User->>Motionblinds: Request to control blinds
Motionblinds->>Bridge: Send control command
Bridge-->>Motionblinds: Acknowledge command
Motionblinds-->>User: Confirm action
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 (
|
✅ Deploy Preview for home-assistant-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
It seems that this PR is targeted against an incorrect branch since it has a parent PR on one of our codebases. Documentation that needs to be updated for an upcoming release should target the |
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
🧹 Outside diff range and nitpick comments (1)
source/_integrations/motion_blinds.markdown (1)
67-67
: LGTM! Consider adding a reference to the manufacturer.The addition of the DD7006A Smart Home bridge to the list of compatible bridges is accurate and aligns with the PR objectives. This change enhances the documentation by providing users with up-to-date information about compatible hardware.
To maintain consistency with some of the other entries in the list, consider adding the manufacturer's name if it's different from "Motionblinds". If it is indeed a Motionblinds product, you can leave it as is.
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.
Thank you, @starkillerOG 👍
Proposed change
Add DD7006A Smart Home bridge as supported by Motion Blinds
Reported to work by a user here: #34893
Type of change
current
branch).current
branch).next
branch).next
branch).Additional information
Checklist
current
branch.next
branch.Summary by CodeRabbit
New Features
DD7006A Smart Home bridge
in the Motionblinds integration, expanding hardware compatibility.Documentation