-
Notifications
You must be signed in to change notification settings - Fork 14.5k
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
Broken links #16637
Broken links #16637
Conversation
This fix resolve - broken links in documentation (Indonesian and Korean) - misplaced markdown braces (that was generated web errors) - fix a few links in blog (english)
Welcome @butuzov! |
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
It will be great to have "hacktoberfest" labels =) |
Deploy preview for kubernetes-io-master-staging ready! Built with commit e457f2a https://deploy-preview-16637--kubernetes-io-master-staging.netlify.com |
Hi @butuzov Thanks for submitting this. This pull request touches several different parts of the website. Would you be willing to resubmit this as different pull requests? |
/lgtm |
Hi @sftim No problem. Can I combine blog related changes in one PR? |
|
It can be closed, in favor of that bunch of PRs. On other points - there are a lot of 404 at the website.. like 267 only for local links. Where can I share this info? I rather kill myself than create an issue for each URL (which I am investigating manually)? Or should I create PR in auto mode (it shouldn't be hard using scripting). |
/do-not-merge |
I think this is related to #16642 |
yes! also related to
And I have no idea what to do with the rest of the broken links I have found. |
Since you already selarate it into new PRs may be you can close this PR in favor of that other PRs? Cheers! |
oh. sure. |
@butuzov there are two ways you can help further (if you're interested):
|
This fix resolve