feat: implement dynamic copyright years and interactive AOSSIE footer#357
feat: implement dynamic copyright years and interactive AOSSIE footer#357SoorejS wants to merge 2 commits intoAOSSIE-Org:mainfrom
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughFrontend pages About.tsx and CoachPage.tsx now use a dynamic current year via Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@frontend/src/Pages/CoachPage.tsx`:
- Around line 136-137: Replace the single-year copyright text in the JSX
paragraph inside CoachPage (the <p className="text-xs text-muted-foreground
mt-1"> element) with a dynamic range pattern starting at 2016 and ending at the
current year and change the owner to "AOSSIE" (e.g., "© 2016-{new
Date().getFullYear()} AOSSIE. All rights reserved.") so it matches the About
page and the requirement.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 364119d7-d9d9-4bc7-a702-0a14c06c360f
📒 Files selected for processing (2)
frontend/src/Pages/About.tsxfrontend/src/Pages/CoachPage.tsx
Addressed Issues:
Fixes #354
Screenshots/Recordings:
Before:

After:

Additional Notes:
This PR removes hardcoded copyright years and replaces them with a dynamic solution using:
new Date().getFullYear()
This ensures the year automatically updates without requiring manual changes in future.
Additionally:
target="_blank"andrel="noopener noreferrer"for safe external navigation.Changes made:
All changes were tested locally and verified to work correctly without introducing any UI or functional regressions.
Benefits:
All changes were tested locally and do not introduce any regressions or console errors.
AI Usage Disclosure:
We encourage contributors to use AI tools responsibly when creating Pull Requests. While AI can be a valuable aid, it is essential to ensure that your contributions meet the task requirements, build successfully, include relevant tests, and pass all linters. Submissions that do not meet these standards may be closed without warning to maintain the quality and integrity of the project. Please take the time to understand the changes you are proposing and their impact. AI slop is strongly discouraged and may lead to banning and blocking. Do not spam our repos with AI slop.
Check one of the checkboxes below:
I have used the following AI models and tools: NA
Checklist
Summary by CodeRabbit
Bug Fixes
New Features
Style