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

[nrf fromlist] boards: arm: nrf9151dk: fix broken link #1540

Merged

Conversation

maxd-nordic
Copy link
Contributor

This patch fixes a broken link in the docs for the nrf9151dk.
It points to the SiP's product page for now.

Upstream PR: zephyrproject-rtos/zephyr#69516

Signed-off-by: Maximilian Deubel [email protected]
(cherry picked from commit be17e6e7133a99156191e64ca3ca6233dc0fad0e)

Copy link

@coderbyheart coderbyheart left a comment

Choose a reason for hiding this comment

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

There is no information in Infocenter about the 9151.

@divipillai
Copy link

There is no information in Infocenter about the 9151.

The PS, hardware guides would be added only during the launch time and that will be to Zoomin - https://docs.nordicsemi.com/category/nrf-91-series. We could removed this link for now, and once the guides are added we could add this zoomin link to the page?

@coderbyheart
Copy link

Yes, that would work.

@maxd-nordic maxd-nordic force-pushed the nrf9151dk_broken_link_downstream branch 2 times, most recently from 3d857b7 to 65b2728 Compare February 28, 2024 11:10
@maxd-nordic maxd-nordic force-pushed the nrf9151dk_broken_link_downstream branch from b8b3f62 to cbe7751 Compare February 28, 2024 12:15
This patch fixes a broken link in the docs for the
nrf9151dk.
It points to the SiP's product page for now.

Upstream PR: zephyrproject-rtos/zephyr#69516

Signed-off-by: Maximilian Deubel <[email protected]>
(cherry picked from commit 1b88b6f4d1384d69fde89f2ff67612288ccca998)
@maxd-nordic maxd-nordic force-pushed the nrf9151dk_broken_link_downstream branch from cbe7751 to 447be29 Compare February 29, 2024 08:41
@jfischer-no jfischer-no merged commit 9af8910 into nrfconnect:main Feb 29, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants