-
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
lib: location: Update documentation #19727
Conversation
CI InformationTo view the history of this post, clich the 'edited' button above Inputs:Sources:sdk-nrf: PR head: f2be28aeb7e308b1d448e3d5aee6ec7205c5384f more detailssdk-nrf:
Github labels
List of changed files detected by CI (6)
Outputs:ToolchainVersion: 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. |
a22e9f8
to
8192f46
Compare
8192f46
to
8e54d7f
Compare
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.
Approved with a nit.
Added a limitation in the library documentation. GNSS API should not be used directly by the application when Location library is used. Doing so may lead to unexpected behavior, because GNSS may be in a different state than expected by the library. Signed-off-by: Tommi Kangas <tommi.kangas@nordicsemi.no>
The title of the GNSS interface documentation has been changed. Added custom link text to all references. Signed-off-by: Tommi Kangas <tommi.kangas@nordicsemi.no>
8e54d7f
to
f2be28a
Compare
Added a limitation in the library documentation. GNSS API should not be used directly by the application when Location library is used. Doing so may lead to unexpected behavior, because GNSS may be in a different state than expected by the library.