Skip to content
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

Deprecation warning from home assistant #27

Open
provinzio opened this issue Mar 24, 2021 · 3 comments
Open

Deprecation warning from home assistant #27

provinzio opened this issue Mar 24, 2021 · 3 comments

Comments

@provinzio
Copy link
Contributor

This might be a quick fix in MQTTConnector.js, but I am not sure if we can just change value_template with position_topic.

Warning from Home Assistant:

Logger: homeassistant.components.mqtt.cover
Source: components/mqtt/cover.py:130
Integration: MQTT (documentation, issues)
First occurred: 07:45:23 (2 occurrences)
Last logged: 07:45:23
using 'value_template' for 'position_topic' is deprecated and will be removed from Home Assistant in version 2021.6, please replace it with 'position_template'
@binsentsu
Copy link
Owner

Looks like position_template must be added in the discovery configuration of MqttConnector. Value must be the same as current value in value_template. I assume value_template can be left for backwards compatibility purposes. If you're up to, you can try adding position_template in the coverConfig and give me feedback. My HA environment is a bit behind with updates at the moment...
(https://www.home-assistant.io/integrations/cover.mqtt/)

@provinzio
Copy link
Contributor Author

I am afraid that I won't be of any help here. I was barely able to do the setup. I have no idea how I can setup a dev environment or upgrade the module used by pm2.

@binsentsu
Copy link
Owner

No problem. I will check it this weekend.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants