Open
Conversation
Signed-off-by: Felipe Zipitria <felipe.zipitria@owasp.org>
Signed-off-by: Felipe Zipitria <felipe.zipitria@owasp.org>
Deploying crs-documentation with
|
| Latest commit: |
b4cfaf8
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://3a0328e4.documentation-km5.pages.dev |
| Branch Preview URL: | https://docs-update-logo-and-notices.documentation-km5.pages.dev |
Signed-off-by: Felipe Zipitria <felipe.zipitria@owasp.org>
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates documentation styling by migrating to a newer version of the Hugo Relearn theme and standardizing notice shortcodes. The changes fix logo display issues in the new theme and add a comprehensive security policy for the CRS Sandbox.
Changes:
- Updated Hugo Relearn theme submodule and replaced custom logo partial with theme configuration
- Migrated all notice shortcodes to new syntax with icon support
- Added comprehensive CRS Sandbox Security Policy document
- Updated CRS Sandbox documentation with privacy notices and revised version information
Reviewed changes
Copilot reviewed 23 out of 24 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| themes/hugo-relearn | Updated theme submodule to newer version |
| layouts/partials/logo.html | Removed custom logo partial (now handled by theme) |
| layouts/partials/custom-header.html | Added CSS to configure logo width |
| content/_index.md | Updated notice syntax with icon |
| content/8-additional-resources/_index.md | Updated notice syntax with icon |
| content/7-known-issues/_index.md | Restructured content and updated notice syntax |
| content/6-development/6-7-crs-sandbox-security-policy.md | Added new security policy document |
| content/6-development/6-4-using-the-crs-sandbox.md | Added privacy notice and updated version references |
| content/6-development/6-2-crs-toolchain.md | Updated notice syntax with icons |
| content/5-advanced-topics/5-3-kubernetes-ingress-controller.md | Updated notice syntax with icon |
| content/5-advanced-topics/5-1-self-contained-mode.md | Updated notice syntax with icon |
| content/4-about-plugins/4-1-plugins.md | Updated notice syntax with icons |
| content/3-about-rules/ruleid.md | Updated notice syntax with icon |
| content/3-about-rules/metadata.md | Updated notice syntax with icon |
| content/3-about-rules/creating.md | Updated notice syntax with icon |
| content/2-how-crs-works/2-4-sampling_mode.md | Updated notice syntax with icon |
| content/2-how-crs-works/2-3-false-positives-and-tuning.md | Updated notice syntax with icons |
| content/2-how-crs-works/2-2-paranoia_levels.md | Updated notice syntax with icon |
| content/2-how-crs-works/2-1-anomaly_scoring/index.md | Updated notice syntax with icons |
| content/1-getting-started/1-4-engine_integration_options.md | Updated notice syntax with icons |
| content/1-getting-started/1-2-extended_install.md | Updated notice syntax with icons |
| content/1-getting-started/1-1-crs-installation.md | Updated notice syntax with icons |
| config/_default/config.toml | Added logo configuration parameters |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Felipe Zipitria <felipe.zipitria@owasp.org>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
what
why