⚠ This page is served via a proxy. Original site: https://github.com
This service does not collect credentials or authentication data.
Skip to content

Conversation

@xynicole
Copy link
Contributor

@xynicole xynicole commented Jan 14, 2026

remove nginx configuration required content

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.

@xynicole xynicole requested a review from a team as a code owner January 14, 2026 00:05
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Jan 14, 2026
@github-actions
Copy link

Deploy Preview will be available once build job completes!

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

@tstraley
Copy link
Contributor

Please remove references to this section:

  • - Deployments require specific NGINX configuration snippets to ensure a seamless experience. See [NGINX configuration required content]({{< ref "nginxaas-google/getting-started/nginx-configuration/overview.md#nginx-configuration-required-content" >}}) for specific details.

  • {{< call-out "warning" >}} If you are not using the "F5 NGINXaaS Default", or if making modifications to the config, please be aware of the [NGINX configuration required content]({{< ref "nginxaas-google/getting-started/nginx-configuration/overview.md#nginx-configuration-required-content" >}}) which must be included. {{< /call-out >}}

  • {{< call-out "warning" >}} When updating a NGINX config, please be aware of the [NGINX configuration required content]({{< ref "nginxaas-google/getting-started/nginx-configuration/overview.md#nginx-configuration-required-content" >}}) which must be included. {{< /call-out >}}

@xynicole xynicole requested a review from a team as a code owner January 14, 2026 00:44
@xynicole
Copy link
Contributor Author

@tstraley Thanks for catching it! I've updated the docs to remove these references

@xynicole xynicole merged commit 661dafa into main Jan 14, 2026
7 checks passed
@xynicole xynicole deleted the 913-config branch January 14, 2026 22:57
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants