search

yp87 / leaf2mqtt

Pull in data from the Nissan Connect platform for the LEAF and publish over MQTT
GNU General Public License v3.0
42 stars 39 forks source link

Correct add on to use `options` instead of `config. #56

Closed apbarratt closed 1 year ago

apbarratt commented 1 year ago

Corrected the object name config to options and added a suggested default value for MQTT host.

After discussion with Frenck on Discord, I've found that the object config doesn't exist, it should be options. This wasn't obvious because we weren't actually providing any default values anyway. I've also stuck in a default MQTT host of homeassistant.local for good measure, but obviously it's just a suggested value. Do feel free to drop it if you prefer.

https://developers.home-assistant.io/docs/add-ons/configuration/#options--schema

I mostly only spotted this as I'm using your work as inspiration on another project and noticed I couldn't find any mention of the config object in the docs so asked on the HA discord about it and was assured it has never existed.

apbarratt commented 1 year ago

Apologies in advance for creating work, I have not tested this change.