-
Notifications
You must be signed in to change notification settings - Fork 669
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
MF-429 -Enabled MQTT subtopic's #554
Conversation
Codecov Report
@@ Coverage Diff @@
## master #554 +/- ##
=======================================
Coverage 87.36% 87.36%
=======================================
Files 62 62
Lines 3301 3301
=======================================
Hits 2884 2884
Misses 286 286
Partials 131 131 Continue to review full report at Codecov.
|
PLease sign the commits. |
Signed-off-by: nmarcetic <n.marcetic86@gmail.com>
Signed-off-by: nmarcetic <n.marcetic86@gmail.com>
2756ee3
to
327330a
Compare
Signed-off-by: nmarcetic <n.marcetic86@gmail.com>
cfd7e12
to
018507d
Compare
Signed-off-by: nmarcetic <n.marcetic86@gmail.com>
It would be nice if we add some tests together with @RubenN and add few sentences in the docs ;) |
Yes, let me merge this and I will try to write a few words of doc. Tests are generally missing for MQTT adapter, and it would be good if @RubenN could help us with this. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
It's great to see how quickly some PRs getting reviewed and merged even on a Friday evening. Kudos to all of @mainflux/contributors ! I'm in a mood for few 🍻 and celebration now :) |
* Enabled MQTT subtopic Signed-off-by: nmarcetic <n.marcetic86@gmail.com> * Removed debug logs Signed-off-by: nmarcetic <n.marcetic86@gmail.com> * Resolved remarks Signed-off-by: nmarcetic <n.marcetic86@gmail.com> * Resolved remarks Signed-off-by: nmarcetic <n.marcetic86@gmail.com>
* Enabled MQTT subtopic Signed-off-by: nmarcetic <n.marcetic86@gmail.com> * Removed debug logs Signed-off-by: nmarcetic <n.marcetic86@gmail.com> * Resolved remarks Signed-off-by: nmarcetic <n.marcetic86@gmail.com> * Resolved remarks Signed-off-by: nmarcetic <n.marcetic86@gmail.com>
* Enabled MQTT subtopic Signed-off-by: nmarcetic <n.marcetic86@gmail.com> * Removed debug logs Signed-off-by: nmarcetic <n.marcetic86@gmail.com> * Resolved remarks Signed-off-by: nmarcetic <n.marcetic86@gmail.com> * Resolved remarks Signed-off-by: nmarcetic <n.marcetic86@gmail.com>
* Enabled MQTT subtopic Signed-off-by: nmarcetic <n.marcetic86@gmail.com> * Removed debug logs Signed-off-by: nmarcetic <n.marcetic86@gmail.com> * Resolved remarks Signed-off-by: nmarcetic <n.marcetic86@gmail.com> * Resolved remarks Signed-off-by: nmarcetic <n.marcetic86@gmail.com>
What does this do?
It enables MQTT subtopic's on multi level regarding to #429 feature request.
Which issue(s) does this PR fix/relate to ?
Resolves #429
List any changes that modify/break current functionality
No breaking changes.
Have you included tests for your changes?
No
Did you document any new/modified functionality?
No
Notes
Subtopics are supported only for MQTT to MQTT communication or if you connect directly to NATS topics. Using other protocols or mixing it with MQTT is limited to general topic only
/messages
without subtopics. Issues to support other protocol adapters will be opened.