Skip to content

Conversation

@DavidLP
Copy link
Contributor

@DavidLP DavidLP commented Jan 28, 2017

Description:
The last PR #5223 stalled, thus this is a minimum change PR fixing regression #5119.

Related issue (if applicable): fixes #5119

Example entry for configuration.yaml (if applicable):

pilight:
  host: 127.0.0.1
  port: 5000
switch:
- platform: pilight
  switches:
    light:
      on_code:
        protocol: intertechno_old
        unit: 3
        id: 'A1'
        'on': 1
      off_code:
        protocol: intertechno_old
        unit: 3
        id: 'A2'
        'off': 1
      on_code_receive:
        protocol: daycom
        state: 'off'
      off_code_receive:
        protocol: daycom
        state: 'on'

Checklist:

If the code does not interact with devices:

  • Local tests with tox run successfully. Your PR cannot be merged unless tests pass
  • Tests have been added to verify that the new code works.

@mention-bot
Copy link

@DavidLP, thanks for your PR! By analyzing the history of the files in this pull request, we identified @fabaff and @janLo to be potential reviewers.

@DavidLP
Copy link
Contributor Author

DavidLP commented Jan 28, 2017

Travis py34 env fails in unrelated components.

@fabaff fabaff changed the title Fixes: Pilight Switch rejects alphanumeric IDs #5119 Pilight Switch rejects alphanumeric IDs Jan 28, 2017
@balloob balloob added this to the 0.37.1 milestone Jan 29, 2017
@balloob balloob merged commit a9325ea into home-assistant:dev Jan 29, 2017
@home-assistant home-assistant locked and limited conversation to collaborators Apr 30, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Pilight Switch rejects alphanumeric IDs

5 participants