Merged
Conversation
There was a problem hiding this comment.
Pull request overview
This PR reorganizes the README for the Asset Tracker Template's NTN use case to improve clarity and structure. The changes focus on enhancing documentation organization and readability without modifying any functional code.
Key changes:
- Restructured content with clearer section hierarchy and headings
- Enhanced explanations of NTN-specific features and requirements
- Improved formatting and language clarity throughout the document
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
jtguggedal
force-pushed
the
ntn-readme
branch
from
3f05904 to
f8ef547
Compare
| ``` | ||
|
|
||
| If `CONFIG_APP_NTN_SEND_GNSS_DATA` is enabled, the GNSS data will be sent instead of temperature: | ||
| If `CONFIG_APP_NTN_SEND_GNSS_DATA` is enabled, GNSS data is sent instead: |
Collaborator
There was a problem hiding this comment.
I think GNSS data is appended, not sent instead
| - **IDLE Mode**: Lower accuracy may be sufficient. | ||
| - **Transition to CONNECTED**: Higher accuracy is typically required to initiate a connection. | ||
|
|
||
| The modem may also predict future accuracy requirements (`next_requested_accuracy`) based on anticipated state transitions, such as waking from sleep or performing a Tracking Area Update (TAU). |
Collaborator
There was a problem hiding this comment.
not sure this sentence should be included, maybe a bit vague and adds more confusion than clarity
Reorganize and improve the language in the README. Signed-off-by: Jan Tore Guggedal <jantore.guggedal@nordicsemi.no>
jtguggedal
force-pushed
the
ntn-readme
branch
from
f8ef547 to
5aa84be
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Reorganize and improve the language in the README.