search

custom-components / sensor.radarr_upcoming_media

🎬 Radarr component to feed Upcoming Media Card.
Apache License 2.0
59 stars 24 forks source link

Corrected YAML #13

Closed agneevX closed 4 years ago

agneevX commented 4 years ago

Corrected the YAML code.

maykar commented 4 years ago

May I ask why? It's just an example and even has a warning:

Do not just copy examples, please use config options above to build your own!

Also, your example removes the mention of sensor which for someone who's never used yaml config might be confused by.

agneevX commented 4 years ago

It's an example, but the indenting is not correct.

Also, your example removes the mention of sensor which for someone who's never used yaml config might be confused by.

Are you sure? I changed the indentation and the data on some of the examples.

maykar commented 4 years ago

Yes, sorry, sensor was kept. The indentation of the current example is valid YAML, just not doubled. While I'm fine changing that I'd rather not have default values represented in the example as they wouldn't be needed.

agneevX commented 4 years ago

Is SSL really used in the local subnet, or on the same host?

maykar commented 4 years ago

It's just an example to show all options and should never be copy and pasted, especially after being explicitly told not to. If you feel strongly about it, I'll happily change it