Skip to content

Conversation

mjang
Copy link
Contributor

@mjang mjang commented Sep 15, 2025

Proposed changes

Link to KB article.

Editing note: I'm unsure of the correct way to present this article

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.

@mjang mjang requested a review from a team as a code owner September 15, 2025 13:35
@github-actions github-actions bot added documentation Improvements or additions to documentation product/nginx-one Issues related to NGINX One Console labels Sep 15, 2025
@mjang mjang self-assigned this Sep 15, 2025
Copy link

Deploy Preview will be available once build job completes!

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

Copy link
Contributor

@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.

LGTM: I assume MyF5 is permissible about gerunds.

@mjang
Copy link
Contributor Author

mjang commented Sep 15, 2025

LGTM: I assume MyF5 is permissible about gerunds.

Actually, no. @travisamartin shared their style guide internally. But based on my reading and SME reactions, the content is still helpful and accurate. So it's valid to link to the article

So I'm going to follow up with suggested changes to the actual article.

@mjang mjang merged commit 244fa42 into main Sep 15, 2025
9 of 10 checks passed
@mjang mjang deleted the mjang-patch-5 branch September 15, 2025 21:18
mjang added a commit that referenced this pull request Sep 15, 2025
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants