Skip to content

Add sony songpal docs #4570

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

Merged
merged 9 commits into from
Mar 9, 2018
Merged

Add sony songpal docs #4570

merged 9 commits into from
Mar 9, 2018

Conversation

rytilahti
Copy link
Member

Description:
Add documentation for Sony Songpal platform.

Pull request in home-assistant (if applicable):
home-assistant/core#12143

Checklist:

  • Branch: Fixes, changes and adjustments should be created against current. New documentation for platforms/components and features should go to next.
  • The documentation follow the standards.

@fabaff fabaff added the new-integration This PR adds documentation for a new Home Assistant integration label Feb 15, 2018
@rytilahti
Copy link
Member Author

The code part of this got just merged, so it would be nice if someone would review this, too :-)

c727
c727 previously approved these changes Mar 2, 2018
c727
c727 previously approved these changes Mar 9, 2018
@c727
Copy link
Contributor

c727 commented Mar 9, 2018

I don't understand why travis fails here...

@balloob balloob added this to the 0.65 milestone Mar 9, 2018
@balloob
Copy link
Member

balloob commented Mar 9, 2018

@c727 invalid syntax was used for the linkable title
9a84d72

@balloob balloob merged commit 086811d into home-assistant:next Mar 9, 2018
@rytilahti rytilahti deleted the add_songpal branch March 9, 2018 18:30
balloob pushed a commit that referenced this pull request Mar 9, 2018
* add sony songpal docs

* use configuration, add a title for configuration section

* remove oxford comma, add link to media_player's services

* add entity_id desc for service

* be more descriptive which devices are supported

* Update songpal supported devices + bump version

* discoverY

* Update media_player.songpal.markdown

* Fix invalid syntax
@rytilahti
Copy link
Member Author

rytilahti commented Mar 9, 2018

Sorry for not noticing that before, thanks balloob for the fix and c727 for the review!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-integration This PR adds documentation for a new Home Assistant integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants