Skip to content
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

Ignore Sonos players with unknown hostnames #11013

Merged
merged 1 commit into from
Dec 8, 2017

Conversation

amelchio
Copy link
Contributor

@amelchio amelchio commented Dec 7, 2017

Description:

Related issue (if applicable): fixes #10906

Example entry for configuration.yaml (if applicable):

media_player:
  - platform: sonos
    hosts:
      - unknown

Checklist:

If the code communicates with devices, web services, or third-party tools:

  • Local tests with tox run successfully.
  • New dependencies have been added to the REQUIREMENTS variable (example).
  • New dependencies are only imported inside functions that use them (example).
  • New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
  • New files were added to .coveragerc.

@pvizeli pvizeli merged commit 0a7e6ac into home-assistant:dev Dec 8, 2017
akatrevorjay added a commit to akatrevorjay/home-assistant that referenced this pull request Dec 11, 2017
…into dev

* 'dev' of https://github.com/home-assistant/home-assistant:
  In dev mode expose only relevant sources (home-assistant#11026)
  Serialize mochad requests (home-assistant#11029)
  Change default js version to auto (home-assistant#10999)
  Update snips to listen on new mqtt topic and utilize rawValue (home-assistant#11020)
  Support LIFX Mini products (home-assistant#10996)
  Ignore Sonos players with unknown hostnames (home-assistant#11013)
  Add support for Canary component and platforms (home-assistant#10306)
@fabaff fabaff mentioned this pull request Dec 16, 2017
amelchio added a commit to amelchio/home-assistant that referenced this pull request Jan 11, 2018
@home-assistant home-assistant locked and limited conversation to collaborators Mar 30, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missing speaker prevents sonos component from loading
3 participants