-
-
Notifications
You must be signed in to change notification settings - Fork 7.5k
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
Elmax/cover #24308
Elmax/cover #24308
Conversation
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.
There are unrelated commits in this PR. Please git rebase your changes onto the latest upstream next branch in order to clean that up.
Thanks 👍
../Frenck
c374614
to
dca8182
Compare
Should be fine now @frenck! Sorry for having included that commit. Thanks! |
That is not possible, as it depends on an upstream change in the core codebase (the cover platform isn't merged). |
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.
✅ Approved. Can be merged as soon as the parent PR gets merged.
Hi guys. Update for implementation? |
Any updates? |
Proposed change
Declaring alarm_control_panel and cover platforms support.
Type of change
current
branch).current
branch).next
branch).next
branch).Additional information
Checklist
current
branch.next
branch.