Skip to content

updated docs to add SSL option#4238

Merged
emlove merged 2 commits into
home-assistant:nextfrom
ryanm101:plex_sensor_https_support
Dec 21, 2017
Merged

updated docs to add SSL option#4238
emlove merged 2 commits into
home-assistant:nextfrom
ryanm101:plex_sensor_https_support

Conversation

@ryanm101
Copy link
Copy Markdown
Contributor

Doc update to support: home-assistant/core#11072

@frenck frenck added the new-feature This PR adds documentation for a new Home Assistant feature to an existing integration label Dec 18, 2017
Copy link
Copy Markdown
Member

@frenck frenck left a comment

Choose a reason for hiding this comment

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

@ryanm101 Thank you for this PR, could you please take a look at the feedback?

Comment thread source/_components/sensor.plex.markdown Outdated
- **password** (*Optional*): The password for your given account on the remote Plex server.
- **server** (*Optional*): The name of your remote Plex server.
- **token** (*Optional*): X-Plex-Token of your remote Plex server.
- **ssl** (*Optional*): Use HTTPS to connecto to plex server, *NOTE* host *must not* be an IP when this option is enabled. Defaults to "False"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Spelling:

  • connecto -> connect
  • plex -> Plex

Copy link
Copy Markdown
Member

@frenck frenck 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 as soon as the parent PR gets merged.

@emlove emlove merged commit 29b4c67 into home-assistant:next Dec 21, 2017
@ryanm101 ryanm101 deleted the plex_sensor_https_support branch December 21, 2017 15:28
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