Skip to content

Add option to hide the group card switch #1516

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 2 commits into from
Dec 4, 2016
Merged

Conversation

teodoc
Copy link
Contributor

@teodoc teodoc commented Nov 29, 2016

Description:
Add option to hide the group card switch

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

@mention-bot
Copy link

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

@Landrash Landrash added the new-feature This PR adds documentation for a new Home Assistant feature to an existing integration label Nov 30, 2016
Copy link
Contributor

@Landrash Landrash left a comment

Choose a reason for hiding this comment

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

Can be merged when parent pr is merged if no changes are made to configuration variables.

@@ -45,6 +45,7 @@ Configuration variables:
- **view** (*Optional*): If yes then the entry will be shown as a view (tab).
- **name** (*Optional*): Name of the group.
- **icon** (*Optional*): If the group is a view, this icon will show at the top in the frontend instead of the name. If it's not a view, then the icon shows when this group is used in another group.
- **hide_switch** (*Optional*): If yes then the group switch will be hidden.
Copy link
Contributor

Choose a reason for hiding this comment

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

Since this is a boolean value it should generally be True or False.

@Landrash Landrash merged commit 9ed887e into home-assistant:next Dec 4, 2016
trisk pushed a commit to trisk/home-assistant.github.io that referenced this pull request Dec 7, 2016
* Add option to hide the group card switch
* Changed hide_switch to control
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-feature This PR adds documentation for a new Home Assistant feature to an existing integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants