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

Implementation of an option in the settings that allows MQTT users to receive updates of unchanged values. #209

Closed
wants to merge 3 commits into from

Conversation

JoschaMiddendorf
Copy link

Added a checkbox, to optionally enable updates of objects even if they have not changed when MQTT reports them.
Please be so kind not to reject the proposal directly and at least test the adaptation.

Added a checkbox, to enable an option to update objects even if they have not changed when MQTT reports them. And a Disclaimer that this behavior is different from the behavior in CoAP mode.
Added translations for the checkbox description.
Added condition to update the objects, even if they have not changed, if the corresponding option is enabled.
@Strobelix80
Copy link

I have integrated your changes for my own testing. Works great and is THE solution for longpush updates!
Please Implement this option

@schmupu
Copy link
Contributor

schmupu commented Aug 19, 2020

Add this functionality for CoAP, MQTT and http. In the moment it makes only sense for MQTT but hopefully later for CoAP too. I copied some parts of you PR (thanks) and add some stuff for http and CoAP in Shelly Adapter Version 4.0.3

@JoschaMiddendorf , could you please check the version 4.0.3?

@schmupu schmupu closed this Aug 19, 2020
schmupu pushed a commit that referenced this pull request Aug 19, 2020
schmupu pushed a commit that referenced this pull request Aug 20, 2020
@foxriver76
Copy link
Member

IMHO the checkbox + changelog is a little bit confusing, it is about updating states and not objects

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

Successfully merging this pull request may close these issues.

None yet

4 participants