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.

Sign up for GitHub

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

Jump to bottom

doc: add 2.1.0 release doc updates #8510

Merged
merged 5 commits into from
Sep 15, 2022
Merged

doc: add 2.1.0 release doc updates #8510

merged 5 commits into from
Sep 15, 2022

Conversation

greg-fer
Copy link
Contributor

@greg-fer greg-fer commented Aug 24, 2022 •
edited
Loading

Updated files related to doc version bump for 2.1.0 NCS release.

Signed-off-by: Grzegorz Ferenc Grzegorz.Ferenc@nordicsemi.no

@greg-fer greg-fer added the doc-required PR must not be merged without tech writer approval. label Aug 24, 2022
@greg-fer greg-fer added this to the 2.1.0 milestone Aug 24, 2022
@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 Aug 24, 2022
greg-fer
greg-fer commented Aug 24, 2022
View reviewed changes
doc/versions.json Outdated Show resolved Hide resolved
@greg-fer greg-fer force-pushed the doc_ncs_release_210_doc_update branch from 6564d58 to b2d5ea4 Compare August 24, 2022 13:02
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Aug 24, 2022 •
edited
Loading

The following west manifest projects have been modified in this Pull Request:

Name Old Revision New Revision Diff
nrfxlib nrfconnect/sdk-nrfxlib@11763cf (main) nrfconnect/sdk-nrfxlib#809 nrfconnect/sdk-nrfxlib#809/files

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@NordicBuilder
Copy link
Contributor

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 Publishing GitHub Action.

umapraseeda
umapraseeda requested changes Aug 24, 2022
View reviewed changes
Copy link
Contributor

@umapraseeda umapraseeda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't we update nrfxlib/conf.py?

doc/nrf/release_notes.rst Outdated Show resolved Hide resolved
doc/versions.json Show resolved Hide resolved
@greg-fer greg-fer force-pushed the doc_ncs_release_210_doc_update branch from b2d5ea4 to b8e4ad3 Compare August 31, 2022 13:42
@greg-fer
Copy link
Contributor Author

greg-fer commented Sep 2, 2022

Edits from the doc review meeting for the documentation section:
- Add Wi-Fi samples
- Link to SML
- Mention of output files
- Debugging >> App development
- Memory opt sections >> App development
- Check about Gazell in the changelog doc section
- New library pages >> library sections
- BLE >> note in Usage in Samples, not elsewhere
- Add links to OS section across docs where reqs are mentioned
- SPM last entry in Docs to be removed
- MCUboot: minor doc updates

@greg-fer greg-fer added the DNM label Sep 2, 2022
@NordicBuilder NordicBuilder removed the DNM label Sep 5, 2022
@greg-fer greg-fer force-pushed the doc_ncs_release_210_doc_update branch from 49b1d64 to 560e8f1 Compare September 5, 2022 12:18
@github-actions github-actions bot removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Sep 5, 2022
@FrancescoSer FrancescoSer self-requested a review September 5, 2022 13:07
@greg-fer greg-fer force-pushed the doc_ncs_release_210_doc_update branch from a9d21e2 to e5ece06 Compare September 5, 2022 14:31
@greg-fer greg-fer mentioned this pull request Sep 6, 2022
Merged
5 tasks
@greg-fer greg-fer force-pushed the doc_ncs_release_210_doc_update branch 2 times, most recently from de92c7c to 2fd3838 Compare September 6, 2022 08:36
FrancescoSer
FrancescoSer approved these changes Sep 6, 2022
View reviewed changes
Copy link
Contributor

@FrancescoSer FrancescoSer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2 small suggestions, but approving

doc/nrf/releases/release-notes-2.1.0.rst Outdated Show resolved Hide resolved
doc/nrf/releases/release-notes-2.1.0.rst Outdated Show resolved Hide resolved
grochu
grochu requested changes Sep 6, 2022
View reviewed changes
doc/nrf/releases/release-notes-2.1.0.rst Outdated Show resolved Hide resolved
@greg-fer greg-fer force-pushed the doc_ncs_release_210_doc_update branch from 870581f to 3649553 Compare September 6, 2022 11:57
@greg-fer
Copy link
Contributor Author

@doki-nordic , @maje-emb , @kapi-no , @alwa-nordic , @kkasperczyk-no , @simensrostad , @jtguggedal , @jhn-nordic , @coderbyheart , you've been added to this PR because of the last commit that updates the pages that mention "experimental support" with a link to the Software maturity levels page that explains the meaning of this phrase. Please check and approve.

@maje-emb
Copy link
Contributor

It seems that the documentation of Distance Measurement and the DDF Service does not contain information about experimental support.

@greg-fer greg-fer force-pushed the doc_ncs_release_210_doc_update branch from 423342b to d89472b Compare September 13, 2022 14:09
@greg-fer
Copy link
Contributor Author

It seems that the documentation of Distance Measurement and the DDF Service does not contain information about experimental support.

@maje-emb , I've updated the files based on maje-emb@606f178

@maje-emb maje-emb mentioned this pull request Sep 14, 2022
Closed
maje-emb
maje-emb approved these changes Sep 14, 2022
View reviewed changes
Copy link
Contributor

@maje-emb maje-emb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM (DM)

@greg-fer greg-fer force-pushed the doc_ncs_release_210_doc_update branch from d89472b to 3a7a449 Compare September 14, 2022 10:03
@github-actions github-actions bot removed the manifest label Sep 14, 2022
@greg-fer greg-fer force-pushed the doc_ncs_release_210_doc_update branch from 3a7a449 to 0f95b0f Compare September 14, 2022 10:10
tejlmand
tejlmand approved these changes Sep 14, 2022
View reviewed changes
Copy link
Contributor

@tejlmand tejlmand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@greg-fer greg-fer force-pushed the doc_ncs_release_210_doc_update branch from 0f95b0f to 5f8acef Compare September 14, 2022 13:48
@greg-fer
doc: add 2.1.0 release doc updates
7497d61 
Updated files related to doc version bump for 2.1.0 NCS release.

Signed-off-by: Grzegorz Ferenc <Grzegorz.Ferenc@nordicsemi.no>
@greg-fer
doc: add release notes for 2.1.0
4de1f92 
Added release notes and release notes highlights for NCS 2.1.0.

Signed-off-by: Grzegorz Ferenc <Grzegorz.Ferenc@nordicsemi.no>
@greg-fer
doc: clean changelog for 2.1.0
24f20ca 
Removed all entries from changelog and orphaned page.

Signed-off-by: Grzegorz Ferenc <Grzegorz.Ferenc@nordicsemi.no>
@greg-fer
doc: last-minute pre-release doc fixes
936c26b 
This commit gathers last-minute pre-release fixes for NCS 2.1.0.

Signed-off-by: Grzegorz Ferenc <Grzegorz.Ferenc@nordicsemi.no>
@greg-fer
doc: replace notes on experimental with links
2b18bab 
Replaced lengthy mentions of experimental support with links.
These now point to the Software Maturity Levels page.

Signed-off-by: Grzegorz Ferenc <Grzegorz.Ferenc@nordicsemi.no>
@greg-fer greg-fer force-pushed the doc_ncs_release_210_doc_update branch from 5f8acef to 2b18bab Compare September 14, 2022 14:16
guwa
guwa reviewed Sep 14, 2022
View reviewed changes
doc/nrf/releases/release-notes-2.1.0.rst
* Support for changing the default TX power using Kconfig options that start with ``CONFIG_BT_CTLR_TX_PWR_*``.

* Updated LTE dual-chip Coex support on the nRF52 Series.
It is now ready for production.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we also get this into the highlight list, since it gets production supported?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @guwa , it is too late for this change.

@rlubos rlubos merged commit 311cd35 into nrfconnect:main Sep 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@laxiLang laxiLang laxiLang left review comments

@guwa guwa guwa left review comments

@nordic-hani nordic-hani nordic-hani left review comments

@mia-ko mia-ko mia-ko approved these changes

@lemrey lemrey lemrey left review comments

@nvlsianpu nvlsianpu nvlsianpu left review comments

@maje-emb maje-emb maje-emb approved these changes

@LuDuda LuDuda LuDuda approved these changes

@koffes koffes koffes approved these changes

@tejlmand tejlmand tejlmand approved these changes

@peknis peknis peknis approved these changes

@pirun pirun pirun approved these changes

@oyvindronningstad oyvindronningstad oyvindronningstad approved these changes

@umapraseeda umapraseeda umapraseeda approved these changes

@joerchan joerchan joerchan approved these changes

@grochu grochu grochu approved these changes

@wiba-nordic wiba-nordic wiba-nordic approved these changes

@richabp richabp richabp approved these changes

@Damian-Nordic Damian-Nordic Damian-Nordic approved these changes

@edmont edmont edmont approved these changes

@b-gent b-gent b-gent approved these changes

@FrancescoSer FrancescoSer FrancescoSer approved these changes

@MarekPieta MarekPieta MarekPieta approved these changes

@coderbyheart coderbyheart coderbyheart approved these changes

@mariuszpos mariuszpos mariuszpos approved these changes

@divipillai divipillai divipillai approved these changes

@osaether osaether osaether approved these changes

@shantha-14 shantha-14 Awaiting requested review from shantha-14

@carlescufi carlescufi Awaiting requested review from carlescufi

@gmarull gmarull Awaiting requested review from gmarull

@frkv frkv Awaiting requested review from frkv

@pdunaj pdunaj Awaiting requested review from pdunaj

@gkafel gkafel Awaiting requested review from gkafel

@KAGA164 KAGA164 Awaiting requested review from KAGA164

@doki-nordic doki-nordic Awaiting requested review from doki-nordic

@kapi-no kapi-no Awaiting requested review from kapi-no

@alwa-nordic alwa-nordic Awaiting requested review from alwa-nordic

@kkasperczyk-no kkasperczyk-no Awaiting requested review from kkasperczyk-no

@simensrostad simensrostad Awaiting requested review from simensrostad

@jtguggedal jtguggedal Awaiting requested review from jtguggedal

@jhn-nordic jhn-nordic Awaiting requested review from jhn-nordic

Assignees

@shantha-14 shantha-14

@rlubos rlubos

Labels
doc-required PR must not be merged without tech writer approval.
Projects
None yet
Milestone
2.1.0
Development

Successfully merging this pull request may close these issues.