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.
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
docs: Reorganise examples #2871
Conversation
Codecov Report
@@ Coverage Diff @@
## main #2871 +/- ##
==========================================
- Coverage 52.32% 52.30% -0.03%
==========================================
Files 58 58
Lines 15959 15959
==========================================
- Hits 8351 8347 -4
- Misses 7330 7332 +2
- Partials 278 280 +2
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
3847171
to
d7bc7c5
Compare
@@ -1,33 +1,31 @@ | |||
--- | |||
title: Configuration | |||
|
|||
description: "This document describes how to configure the NGINX App Protect module." | |||
description: "This document describes how to configure the NGINX App Protect WAF module." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should the folder here also be app-protect-waf
instead of just app-protect
? Also the folder in ingress-resources
?
I've also noticed that in the docs is app-protect
but in the examples is appprotect
.
docs/content/troubleshooting/troubleshooting-with-app-protect.md
Outdated
Show resolved
Hide resolved
docs/content/troubleshooting/troubleshooting-with-app-protect.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Jodie Putrino <j.putrino@f5.com>
7c1e995
to
724fc69
Compare
724fc69
to
0b6a1bf
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice!
* Initial attempt at reorganising examples * Rename waf and dos folders * Change App Protect to App Protect WAF * Apply suggestions from code review Co-authored-by: Jodie Putrino <j.putrino@f5.com> * Rename examples folders; rename docs folder and attempt alias Co-authored-by: Jodie Putrino <j.putrino@f5.com>
Proposed changes
It's difficult to find our Custom Resources examples in the current folder layout structure. Additionally, a lot of our default examples in the documentation point to the Ingress examples instead of the VS/ TS examples. This PR moves the Ingress examples to their own folder, moves the shared examples to a new folder, and adds references to the VS examples where applicable (including adding a couple of missing examples from VirtualServer).
Checklist
Before creating a PR, run through this checklist and mark each as complete.