Skip to content

Conversation

@ADubhlaoich
Copy link
Member

@ADubhlaoich ADubhlaoich commented Nov 4, 2025

Proposed changes

This commit removes the old, redundant files from the previous iteration of the F5 WAF for NGINX documentation. As part of removing these files, links have been updated across the product documentation website.

These files are still accessible through git history, but the critically important parts of documentation were migrated and rewritten as part of the new documentation set.

Included in the changes are metadata and Markdown fixes for affected files, some of which may require follow-up work with the relevant product teams to ensure automation does not re-introduce the issues.

Checklist

Before sharing this pull request, I completed the following checklist:

Footnotes

  1. Potentially sensitive information includes personally identify information (PII), authentication credentials, and live URLs. Refer to the style guide for guidance about placeholder content.

This commit removes the old, redundant files from the previous iteration
of the F5 WAF for NGINX documentation. As part of removing these files,
links have been updated across the product documentation website.

These files are still accessible through git history, but the critically
important parts of documentation were migrated and rewritten as part of
the new documentation set.

Included in the changes are metadata and Markdown fixes for affected
files, some of which may require follow-up work with the relevant
product teams to ensure automation does not re-introduce the issues.
@ADubhlaoich ADubhlaoich requested review from a team as code owners November 4, 2025 17:28
@github-actions github-actions bot added documentation Improvements or additions to documentation product/nim Issues related to NGINX Instance Manager product/nginx-plus Issues related to NGINX Plus (and NGINX Open Source in docs.nginx.com) product/nginx-one Issues related to NGINX One Console product/waf Issues related to F5 WAF for NGINX product/nic Issues related to NGINX Ingress Controller labels Nov 4, 2025
@travisamartin
Copy link
Contributor

Since the preview link isn't generating, I built the docs locally and didn't have any issues.

LGTM. Thank you!

@github-actions
Copy link

github-actions bot commented Nov 5, 2025

Deploy Preview will be available once build job completes!

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

@ADubhlaoich ADubhlaoich merged commit 88ca6bd into main Nov 5, 2025
9 checks passed
@ADubhlaoich ADubhlaoich deleted the nap/remove-old-files branch November 5, 2025 10:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation product/nginx-one Issues related to NGINX One Console product/nginx-plus Issues related to NGINX Plus (and NGINX Open Source in docs.nginx.com) product/nic Issues related to NGINX Ingress Controller product/nim Issues related to NGINX Instance Manager product/waf Issues related to F5 WAF for NGINX

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove legacy /content/nap-waf directory to reduce maintenance overhead and improve AI accuracy

4 participants