-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
lib: lte_link_control: Misc doc updates #19768
base: main
Are you sure you want to change the base?
Conversation
CI InformationTo view the history of this post, clich the 'edited' button above Inputs:Sources:sdk-nrf: PR head: 118fc33c9ab49c4d375e13efa8b0862715ef8eb2 more detailssdk-nrf:
Github labels
List of changed files detected by CI (2)
Outputs:ToolchainVersion: b77d8c1312 Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped;
|
You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds. Note: This comment is automatically posted by the Documentation Publish GitHub Action. |
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 when comment from @peknis is addressed.
2592a08
to
dd3f263
Compare
dd3f263
to
1a1f055
Compare
Adding a note that LTE LC and nrf_modem_at APIs should be used together with caution. A comment in lte_lc_connect* that reconnection happens automatically. Signed-off-by: Tommi Rantanen <tommi.rantanen@nordicsemi.no>
1a1f055
to
118fc33
Compare
Adding a note that LTE LC and
nrf_modem_at
APIs should be used together with caution.A comment in
lte_lc_connect*
that reconnection happens automatically.