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

added documentation for monoprice platform #3495

Merged
merged 8 commits into from
Oct 18, 2017

Conversation

etsinko
Copy link
Contributor

@etsinko etsinko commented Oct 2, 2017

Description:

This is a markdown for monoprice platform that enables control of Monoprice 6-zone amplifier

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

footer: true
logo: mp-logo.svg
ha_category: Media Player
ha_release: 0.25
Copy link
Member

Choose a reason for hiding this comment

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

Please update.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Which release version should I put there? 0.55?

comments: false
sharing: true
footer: true
logo: mp-logo.svg
Copy link
Member

Choose a reason for hiding this comment

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

Please change the filename to monoprice.svg. Easier to identify if the name is there.

@fabaff fabaff added Hacktoberfest An PR on this issue (or the PR itself) is eligible towards Hacktoberfest! new-integration This PR adds documentation for a new Home Assistant integration labels Oct 2, 2017
updated ha_release to 0.55
@etsinko
Copy link
Contributor Author

etsinko commented Oct 8, 2017

@fabaff I noticed that Monoprice logo is pure white and it is barely visible on white background. I'm not sure what you usually do in such cases but I'd suggest changing it to solid black or maybe black outline
screenshot from 2017-10-08 13 17 31
screenshot from 2017-10-08 13 17 34

.

@arsaboo
Copy link
Contributor

arsaboo commented Oct 8, 2017

How about something like
image
or
image

@etsinko
Copy link
Contributor Author

etsinko commented Oct 8, 2017

@arsaboo I'm not sure about red because it is not one of Monoprice's colors. But I think black will work great.
screenshot from 2017-10-08 13 33 31
screenshot from 2017-10-08 13 33 26

@frenck
Copy link
Member

frenck commented Oct 17, 2017

@fabaff, could you please revisit your approval? Thx 👍

@fabaff fabaff merged commit e42a0fb into home-assistant:next Oct 18, 2017
@etsinko etsinko deleted the monoprice branch November 2, 2017 20:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Hacktoberfest An PR on this issue (or the PR itself) is eligible towards Hacktoberfest! 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