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

Use single topic for MQTT client status (LWT, birth, etc.) #435

Merged
merged 6 commits into from
Apr 15, 2019
Merged

Conversation

sidoh
Copy link
Owner

@sidoh sidoh commented Apr 15, 2019

As suggested/discussed in this comment thread: #418 (comment)

Thanks, @rejoe2!

@sidoh sidoh merged commit 6659728 into 1.9.0 Apr 15, 2019
@sidoh sidoh deleted the mqtt_status branch April 15, 2019 17:39
@odis68
Copy link

odis68 commented Apr 16, 2019

Tell me how to useLWT. Because messages must be 2. The first is entered into the broker when connected with an indication of the topic, and the second is periodically sent to the broker for a status condition. Is it possible to use a separate topic (milight/states/LWT) for LWT or should it be combined with any of the other topics?

@odis68
Copy link

odis68 commented Apr 16, 2019

image
The module is connected and active!

@rejoe2
Copy link

rejoe2 commented Apr 16, 2019

Are you using latest version (1.9.0-rc3)?

@odis68
Copy link

odis68 commented Apr 16, 2019

Sorry for an example from another project. This is not disrespectful but for example only.
image

Yes, I have installed latest version (1.9.0-rc3)

@sidoh
Copy link
Owner Author

sidoh commented Apr 16, 2019

@odis68, the name of the setting changed. It's now mqtt_client_status_topic. You'll need to reconfigure it.

@sidoh
Copy link
Owner Author

sidoh commented Apr 16, 2019

Guessing the data on your status/LWT topic is a left over retained message.

@odis68
Copy link

odis68 commented Apr 16, 2019

image

@sidoh
Copy link
Owner Author

sidoh commented Apr 16, 2019

That is not RC3. Looks like RC2 or earlier. RC3 here:

https://github.com/sidoh/esp8266_milight_hub/releases/tag/1.9.0-rc3

@odis68
Copy link

odis68 commented Apr 16, 2019

r. RC3
I cleared the topic (milight/states/LWT) by entering an empty value but after resetting the same value.

@sidoh
Copy link
Owner Author

sidoh commented Apr 16, 2019

The UI is definitely from RC2.

Can you maybe try force-refreshing? Perhaps the UI got cached by the browser or something.

@odis68
Copy link

odis68 commented Apr 16, 2019

You were right!!! The problem in the browser helped clear the cache !! Thank you very much you have fulfilled several trials at once.

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.

3 participants