Skip to content
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

net: lib: nrf_cloud: Update A-GPS cddl #12566

Merged
merged 1 commit into from
Oct 10, 2023

Conversation

plskeggs
Copy link
Contributor

@plskeggs plskeggs commented Oct 6, 2023

The number of types of A-GPS elements has increased to support QZSS, so the nrf_cloud/coap/cddl/nrf_cloud_coap_agps.cddl has been changed to match, and the encoder has been regenerated.

@plskeggs plskeggs added this to the 2.5.0 milestone Oct 6, 2023
@plskeggs plskeggs requested a review from tokangas October 6, 2023 18:27
@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Oct 6, 2023
The number of types of A-GPS has increased to support
QZSS, so the nrf_cloud/coap/cddl/nrf_cloud_coap_agps.cddl
has been changed to match, and the encoder has been
regenerated.

Signed-off-by: Pete Skeggs <peter.skeggs@nordicsemi.no>
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Oct 6, 2023

Test specification

CI/Jenkins/NRF

  • Integration Platforms

CI/Jenkins/integration

Test Module File based changes Manually selected West overwrite
test-fw-nrfconnect-nrf-iot_cloud X
test-fw-nrfconnect-nrf-iot_mosh X
test-fw-nrfconnect-nrf-iot_positioning X
test-fw-nrfconnect-nrf-iot_serial_lte_modem X
test-fw-nrfconnect-nrf-iot_thingy91 X

Detailed information of selected test modules

Note: This message is automatically posted and updated by the CI

@plskeggs plskeggs removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Oct 6, 2023
@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Oct 6, 2023
@tokangas
Copy link
Contributor

tokangas commented Oct 9, 2023

@plskeggs I've also updated (and renamed) the A-GPS cddl in #12568, so either of the PRs needs to be rebased.

@nordicjm nordicjm merged commit b606f1d into nrfconnect:main Oct 10, 2023
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants