-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
doc: improved note in nRF Cloud multi-service sample #20657
Conversation
CI InformationTo view the history of this post, clich the 'edited' button above Inputs:Sources:more detailsGithub labels
List of changed files detected by CI (0)
Outputs:ToolchainVersion: Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped;
|
dd3a7ec
to
32b206f
Compare
You can find the documentation preview for this PR here. Preview links for modified nRF Connect SDK documents: |
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.
It is fine with this change, though deceiving -- this particular sample does not support REST, only MQTT or CoAP. So it would be better to remove REST. Not sure why it mentioned that before.
Seems that the note is totally useless. |
Clarified the protocol options in the note. Signed-off-by: Pekka Niskanen <pekka.niskanen@nordicsemi.no>
32b206f
to
9076e11
Compare
After discussing with Markku Lehto, decided to remove the note as it does not add any value in this context. |
|
Removed the note as it does not add any value.