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

Fix broken URLs from repository migration #7213

Merged
merged 4 commits into from
Jan 28, 2025
Merged

Conversation

ADubhlaoich
Copy link
Contributor

Proposed changes

This pull request fixes a number of URLs that broke during the repository migration from the nginxinc to nginx organisation: some URLs were erroneously updated, leading to 404 issues.

Besides direct URL fixes, some links were changed to internal references instead of production URLs, which make link issues appear prominently as a Hugo error instead of silently failing. Some metadata fixes are also included in affected files.

Checklist

Before creating a PR, run through this checklist and mark each as complete.

  • I have read the CONTRIBUTING doc
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked that all unit tests pass after adding my changes
  • I have updated necessary documentation
  • I have rebased my branch onto main
  • I will ensure my PR is targeting the main branch and pulling from my branch from my own fork

This commit fixes a number of URLs that broke during the repository
migration from the nginxinc to nginx organisation: some URLs were
erroneously updated, leading to 404 issues.

Besides direct URL fixes, some links were changed to internal references
instead of production URLs, which make link issues appear prominently as
a Hugo error instead of silently failing. Some metadata fixes are also
included in affected files.
@ADubhlaoich ADubhlaoich added the needs cherry pick Cherry pick this PR into a release branch label Jan 24, 2025
@ADubhlaoich ADubhlaoich requested review from a team as code owners January 24, 2025 17:01
@github-actions github-actions bot added the documentation Pull requests/issues for documentation label Jan 24, 2025
Copy link

Deploy Preview will be available once build job completes!

Name Link
😎 Deploy Preview https://frontdoor-test-docs.nginx.com/previews/nginx-ingress-controller/7213/

@ADubhlaoich ADubhlaoich enabled auto-merge (squash) January 27, 2025 16:54
@ADubhlaoich ADubhlaoich merged commit 7015f55 into main Jan 28, 2025
30 checks passed
@ADubhlaoich ADubhlaoich deleted the docs/fix-broken-links branch January 28, 2025 13:48
nginx-bot pushed a commit that referenced this pull request Jan 28, 2025
This commit fixes a number of URLs that broke during the repository
migration from the nginxinc to nginx organisation: some URLs were
erroneously updated, leading to 404 issues.

Besides direct URL fixes, some links were changed to internal references
instead of production URLs, which make link issues appear prominently as
a Hugo error instead of silently failing. Some metadata fixes are also
included in affected files.
ADubhlaoich added a commit that referenced this pull request Jan 28, 2025
Fix broken URLs from repository migration (#7213)

This commit fixes a number of URLs that broke during the repository
migration from the nginxinc to nginx organisation: some URLs were
erroneously updated, leading to 404 issues.

Besides direct URL fixes, some links were changed to internal references
instead of production URLs, which make link issues appear prominently as
a Hugo error instead of silently failing. Some metadata fixes are also
included in affected files.

Co-authored-by: Alan Dooley <a.dooley@f5.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
documentation Pull requests/issues for documentation needs cherry pick Cherry pick this PR into a release branch
Projects
Status: Done 🚀
Development

Successfully merging this pull request may close these issues.

3 participants