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

Include a new library : MQTT #2561

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Conversation

TopNiz
Copy link

@TopNiz TopNiz commented Mar 20, 2020

This library fully functional helps deal with MQTT messaging protocol through Snap!.
It is also a good OOP example

@TopNiz TopNiz marked this pull request as ready for review March 20, 2020 20:01
@TopNiz
Copy link
Author

TopNiz commented Mar 20, 2020

Additional demo projects are also added
The issue #2380 (comment) can be closed

@TopNiz TopNiz mentioned this pull request Mar 20, 2020
@jmoenig
Copy link
Owner

jmoenig commented Apr 23, 2020

Wow, Thank you!

@TopNiz
Copy link
Author

TopNiz commented Apr 23, 2020

It's a pleasure.
The demo project may lead to some confusions. So I have enhanced it to be completely functional and intuitive.
When this pull-request is merged, I'll proceed with and additional pull-request with the new demo project.

* Made more intuitive
* corrected odd behavior (red shadow for non initialized client variable when project is loaded)
* transferred the connection at the click  event instead of at the start event
 *and use of .toString() method to capture the received message (otherwise, it is a ByteBuffer)
Take into account the new version of the elaborated MQTT  demo project
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.

2 participants