Skip to content

Update tuya.markdown#6250

Closed
DonnyBahama wants to merge 1 commit intohome-assistant:currentfrom
DonnyBahama:patch-1
Closed

Update tuya.markdown#6250
DonnyBahama wants to merge 1 commit intohome-assistant:currentfrom
DonnyBahama:patch-1

Conversation

@DonnyBahama
Copy link
Copy Markdown

Values should be enclosed in single quotes so I updated the example accordingly. Platform required was set to false; it is required so I changed it to true and added it to the example. Only two platform options were given- tuya and smart_life. My devices required the platform to be set to 'jinvoo_smart' so I updated the platforms to include that option. If there are other known options, I suggest they be listed as well.

Description:

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

Checklist:

  • Branch: next is for changes and new documentation that will go public with the next home-assistant release. Fixes, changes and adjustments for the current release should be created against current.
  • The documentation follow the standards.

Values should be enclosed in single quotes so I updated the example accordingly. Platform required was set to false; it is required so I changed it to true and added it to the example. Only two platform options were given- tuya and smart_life. My devices required the platform to be set to 'jinvoo_smart' so I updated the platforms to include that option. If there are other known options, I suggest they be listed as well.
@ghost ghost added the to-do label Sep 10, 2018
@frenck frenck added enhancement ready-for-review This PR needs to be reviewed current This PR goes into the current branch and removed to-do labels Sep 10, 2018
@fabaff
Copy link
Copy Markdown
Member

fabaff commented Sep 11, 2018

The docs update was already merged: #6048

@fabaff fabaff closed this Sep 11, 2018
@ghost ghost removed the ready-for-review This PR needs to be reviewed label Sep 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

current This PR goes into the current branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants