Skip to content

Conversation

@zahava-brown
Copy link
Contributor

Proposed changes

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.

@zahava-brown zahava-brown requested a review from a team as a code owner October 26, 2025 12:20
@github-actions github-actions bot added documentation Improvements or additions to documentation product/waf Issues related to F5 WAF for NGINX labels Oct 26, 2025
@zahava-brown zahava-brown changed the title Add nap5 prerequisites DRAFT: Add nap5 prerequisites Oct 26, 2025
@zahava-brown zahava-brown marked this pull request as draft October 27, 2025 11:23
@zahava-brown zahava-brown changed the title DRAFT: Add nap5 prerequisites Add nap5 prerequisites Oct 27, 2025
Copy link
Member

@ADubhlaoich ADubhlaoich 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 absolutely no context around what this new document has been created for, or what problem it is attempting to solve.

Every single deployment document already has a "Before you begin" section which includes all of this detail, and any logical links between this page and others are at the very bottom.

image

This entire page is redundant with the detail on the existing pages, meaning that a user will read through all possible requirements for any possible deployment even if they are only interested in one, only to come across the exact same set of requirements for their desired deployment at the top of the actual deployment page.

This is a pattern that does not exist in any product documentation for the rest of the NGINX portfolio.

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/waf Issues related to F5 WAF for NGINX

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants