Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Broken link #47940

Closed
PrashanthKumar0 opened this issue May 9, 2023 · 2 comments · Fixed by #47942
Closed

Broken link #47940

PrashanthKumar0 opened this issue May 9, 2023 · 2 comments · Fixed by #47942
Labels
doc Issues and PRs related to the documentations.

Comments

@PrashanthKumar0
Copy link

Affected URL(s)

https://nodejs.org/dist/latest-v18.x/docs/api/buffer.html

Description of the problem

visit https://nodejs.org/dist/latest-v18.x/docs/api/buffer.html

and search for "binary string"
so the link pointing to
this mdn article is broken
@PrashanthKumar0 PrashanthKumar0 added the doc Issues and PRs related to the documentations. label May 9, 2023
@mertcanaltin
Copy link
Member

mertcanaltin commented May 9, 2023 •
edited
Loading

@PrashanthKumar0 hello I know we need to open this issue in the nodejs.org repo

Trott added a commit to Trott/io.js that referenced this issue May 9, 2023
@Trott
doc: remove broken link
028099b 
Closes: nodejs#47940
@Trott Trott mentioned this issue May 9, 2023
Merged
nodejs-github-bot pushed a commit that referenced this issue May 11, 2023
@Trott
doc: remove broken link
6191dcf 
Closes: #47940
PR-URL: #47942
Fixes: #47940
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
targos pushed a commit that referenced this issue May 12, 2023
@Trott @targos
doc: remove broken link
d97ff24 
Closes: #47940
PR-URL: #47942
Fixes: #47940
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@Trott
Copy link
Member

Trott commented May 12, 2023 •
edited
Loading

@PrashanthKumar0 hello I know we need to open this issue in the nodejs.org repo

I don't believe that is correct. This repo is the right one for broken links within the API documentation. (I know it's confusing, though.)

targos pushed a commit that referenced this issue May 15, 2023
@Trott @targos
doc: remove broken link
6f5ba92 
Closes: #47940
PR-URL: #47942
Fixes: #47940
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
danielleadams pushed a commit that referenced this issue Jul 6, 2023
@Trott @danielleadams
doc: remove broken link
eb433dd 
Closes: #47940
PR-URL: #47942
Fixes: #47940
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
MoLow pushed a commit to MoLow/node that referenced this issue Jul 6, 2023
@Trott @MoLow
doc: remove broken link
244db96 
Closes: nodejs#47940
PR-URL: nodejs#47942
Fixes: nodejs#47940
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
doc Issues and PRs related to the documentations.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

doc: remove broken link Trott/io.js
3 participants
@Trott @PrashanthKumar0 @mertcanaltin