Skip to content

PLM Overview and Installation #1060

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 4 commits into from
Sep 8, 2025
Merged

PLM Overview and Installation #1060

merged 4 commits into from
Sep 8, 2025

Conversation

zahava-brown
Copy link
Contributor

@zahava-brown zahava-brown commented Sep 4, 2025 •
edited
Loading

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

@github-actions github-actions bot added documentation Improvements or additions to documentation product/waf Issues related to F5 WAF for NGINX labels Sep 4, 2025
@github-actions GitHub Actions
Copy link

github-actions bot commented Sep 4, 2025 •
edited
Loading

✅ All required contributors have signed the F5 CLA for this PR. Thank you!
Posted by the CLA Assistant Lite bot.

@zahava-brown
Copy link
Contributor Author

recheck

@zahava-brown
Copy link
Contributor Author

I have hereby read the F5 CLA and agree to its terms

@zahava-brown
Copy link
Contributor Author

recheck

@zahava-brown zahava-brown changed the base branch from main to nap-release-plm September 7, 2025 06:13
@zahava-brown zahava-brown marked this pull request as ready for review September 7, 2025 06:13
@zahava-brown zahava-brown requested a review from a team as a code owner September 7, 2025 06:13
liadlevif5
liadlevif5 reviewed Sep 7, 2025
View reviewed changes
ADubhlaoich
ADubhlaoich approved these changes Sep 8, 2025
View reviewed changes
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.

LGTM! There's some small things about the text I think can be improved, but I will take responsibility for that.

@zahava-brown
Copy link
Contributor Author

LGTM! There's some small things about the text I think can be improved, but I will take responsibility for that.

@ADubhlaoich Do I need to do anything else before it can be merged?

@ADubhlaoich ADubhlaoich merged commit fd48e4b into nginx:nap-release-plm Sep 8, 2025
7 checks passed
ADubhlaoich added a commit that referenced this pull request Sep 22, 2025
@ADubhlaoich @zahava-brown
@ViktorFefilovF5 @mouraddmeiri @Y-Klughaupt
feat: WAF PLM Overview and Installation (#1060) (#1077)
b1beb03 
Add "working" state of documentation related to the PLM feature. This content has been re-worked/re-written in another branch, whose contents will deprecate this one.

---------

Co-authored-by: zahava-brown <[email protected]>
Co-authored-by: ViktorFefilovF5 <[email protected]>
Co-authored-by: mouraddmeiri <[email protected]>
Co-authored-by: Y-Klughaupt <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ADubhlaoich ADubhlaoich ADubhlaoich approved these changes

+1 more reviewer

@liadlevif5 liadlevif5 liadlevif5 left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

documentation Improvements or additions to documentation product/waf Issues related to F5 WAF for NGINX

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@zahava-brown @ADubhlaoich @liadlevif5