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

feat(samples): add basic discovery sample with mqtt5 client #423

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sperka
Copy link

@sperka sperka commented Aug 31, 2023

Issue #, if available:

Description of changes:
Added basic discovery example for MQTT5. MQTT5 client implementation has major changes compared to v311, hence, an example would benefit customers to see how to connect with the MQTT5 client using discovery.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@bretambrose
Copy link
Contributor

This is really nice. I'll see about getting it merged; the only possible hangup is there may be a "well the other SDKs should get a corresponding sample too" which might slow things down a bit.

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