Skip to content

Add separate on/off ids on manual configured IHC lights#20253

Merged
MartinHjelmare merged 6 commits into
home-assistant:devfrom
msvinth:ihc_onoff
Mar 1, 2019
Merged

Add separate on/off ids on manual configured IHC lights#20253
MartinHjelmare merged 6 commits into
home-assistant:devfrom
msvinth:ihc_onoff

Conversation

@msvinth
Copy link
Copy Markdown
Contributor

@msvinth msvinth commented Jan 19, 2019

Description:

Add support for separate on/off ids on manual configured IHC lights.
This makes it easier to support IHC code units thats relies on being
turned on and off through specific inputs.
Also adds a pulse service (ihc.pulse) that supports sending a short on/off pulse to an IHC input.

Related issue (if applicable): fixes #

Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#8218

Checklist:

  • The code change is tested and works locally.
  • Local tests pass with tox. Your PR cannot be merged unless tests pass
  • There is no commented out code in this PR.

If user exposed functionality or configuration variables are added/changed:

@homeassistant
Copy link
Copy Markdown
Contributor

Hi @msvinth,

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

Comment thread homeassistant/components/ihc/util.py Outdated
Comment thread homeassistant/components/ihc/__init__.py
Comment thread homeassistant/components/ihc/light.py Outdated
Comment thread homeassistant/components/ihc/light.py Outdated
Comment thread homeassistant/components/ihc/light.py Outdated
Comment thread homeassistant/components/ihc/light.py Outdated
@msvinth msvinth requested review from a team and tinloaf as code owners February 26, 2019 19:45
@msvinth msvinth requested a review from a team February 26, 2019 19:45
@msvinth msvinth requested a review from a team as a code owner February 26, 2019 19:45
@msvinth
Copy link
Copy Markdown
Contributor Author

msvinth commented Feb 26, 2019

Sorry everyone I asked for a review. I meshed up my git commands and did a pull instead of rebase. Only @MartinHjelmare is needed for the review.
@MartinHjelmare Sorry, but I ended up doing a force push to fix my git mistake.

Comment thread homeassistant/components/ihc/__init__.py Outdated
Comment thread homeassistant/components/ihc/__init__.py Outdated
Comment thread homeassistant/components/ihc/light.py Outdated
Copy link
Copy Markdown
Member

@MartinHjelmare MartinHjelmare left a comment

Choose a reason for hiding this comment

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

Looks good!

@MartinHjelmare MartinHjelmare changed the title Add support for separate on/off ids on manual configured IHC lights. Add separate on/off ids on manual configured IHC lights Feb 28, 2019
@MartinHjelmare
Copy link
Copy Markdown
Member

Try rebasing to make the build pass.

This makes it easier to support IHC code units thats relies on being
turned on and off through specific inputs.
Also adds a pulse service (ihc.pulse) that supports sending a short on/off pulse to an IHC input.
@msvinth
Copy link
Copy Markdown
Contributor Author

msvinth commented Mar 1, 2019

Did the rebasing and the build is now passing. Thank you for the review.

@MartinHjelmare MartinHjelmare merged commit 0aba49a into home-assistant:dev Mar 1, 2019
@ghost ghost removed the in progress label Mar 1, 2019
@balloob balloob mentioned this pull request Mar 20, 2019
@ghost ghost removed the platform: light.ihc label Mar 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants