Chore: Add footer component to add the license in all the pages#458
Chore: Add footer component to add the license in all the pages#458
Conversation
Deploying frameworks with
|
| Latest commit: |
7d37b0b
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://870d3a9e.frameworks-573.pages.dev |
| Branch Preview URL: | https://chore-license-footer.frameworks-573.pages.dev |
frameworks-volunteer
left a comment
There was a problem hiding this comment.
Model: `z-ai/glm-5.1` Reasoning: `high` Provider: `openrouter`
Reviewing the two new files: docs/footer.tsx and docs/styles.css additions.
Security: No issues. The link uses rel="noopener noreferrer" correctly. No secrets, no injection vectors, no unsafe patterns. Clean.
QA findings (2 issues, 1 suggestion):
ISSUE 1 -- Footer is not wired up and will not render.
The Footer component is created in docs/footer.tsx but is never imported or referenced in vocs.config.tsx or any layout file. Vocs does not auto-discover components from docs/. As-is, this PR adds dead code -- the footer will not appear on any page.
Two options to fix:
(a) Use vocs built-in outlineFooter config (renders in the sidebar outline area):
In vocs.config.tsx, add: outlineFooter:
Then import Footer from "./docs/footer"
(b) Use a custom layout override if you need the footer at the bottom of the content area (requires a docs/_layout.tsx or similar approach per vocs docs).
ISSUE 2 -- Missing trailing newline in docs/styles.css.
The appended .footer block ends without a newline character. This triggers git/editor warnings and is inconsistent with the rest of the file (which ends with a newline). Add a blank line after the closing brace.
SUGGESTION -- Copyright year may be outdated.
The footer says "© 2023". Consider using "© 2023-present" or "© 2023-2026" to keep it current without annual updates. Match whatever convention the LICENSE file uses.
Add vocs footer component in the docs folder so the license is visible in all the pages on the site
Frameworks PR Checklist
Thank you for contributing to the Security Frameworks! Before you open a PR, make sure to read information for contributors and take a look at the following checklist:
vocs.config.tsadding thedev: trueparameter