Skip to content

Add support for learning RF commands with Broadlink remotes#14398

Merged
frenck merged 2 commits intohome-assistant:nextfrom
felipediel:rf-support
Nov 30, 2020
Merged

Add support for learning RF commands with Broadlink remotes#14398
frenck merged 2 commits intohome-assistant:nextfrom
felipediel:rf-support

Conversation

@felipediel
Copy link
Copy Markdown
Contributor

Proposed change

Add support for learning RF commands with Broadlink remotes.

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.

@probot-home-assistant probot-home-assistant Bot added has-parent This PR has a parent PR in another repo next This PR goes into the next branch labels Sep 5, 2020
@frenck frenck added the new-feature This PR adds documentation for a new Home Assistant feature to an existing integration label Sep 5, 2020
Copy link
Copy Markdown
Member

@frenck frenck left a comment

Choose a reason for hiding this comment

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

Thanks for the PR @felipediel!

Some small comments, could you take a look? Thanks! 👍

Comment thread source/_integrations/broadlink.markdown Outdated
Comment thread source/_integrations/broadlink.markdown Outdated
Comment thread source/_integrations/broadlink.markdown Outdated
Comment thread source/_integrations/broadlink.markdown Outdated
Comment thread source/_integrations/broadlink.markdown Outdated
Comment thread source/_integrations/broadlink.markdown Outdated
@felipediel felipediel marked this pull request as ready for review November 13, 2020 19:51
@frenck frenck added the parent-merged The parent PR has been merged already label Nov 13, 2020
@frenck frenck merged commit 4d40da6 into home-assistant:next Nov 30, 2020
@probot-home-assistant probot-home-assistant Bot removed in-progress This PR/Issue is currently being worked on parent-merged The parent PR has been merged already labels Nov 30, 2020
@felipediel
Copy link
Copy Markdown
Contributor Author

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants