Centralize and synchronize organization statistics#576
Centralize and synchronize organization statistics#576prashantkumarniraj wants to merge 5 commits intoAOSSIE-Org:mainfrom
Conversation
📝 WalkthroughWalkthroughA new centralized configuration file containing organization statistics is introduced, and multiple pages and components are updated to import and use these values instead of hardcoded literals. Additionally, Twitter icon imports are updated from Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (2 warnings)
✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧪 Generate unit tests (beta)
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.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (2)
src/pages/index.jsx (1)
91-97:⚠️ Potential issue | 🟡 MinorAria label still says "Follow on Twitter" while the icon is now X (faXTwitter).
Now that the icon has been updated to the X brand, the
aria-labelshould be updated to match (e.g.,"Follow on X"). Same applies inFooter.jsx(line 65). This affects screen-reader users who will hear "Twitter" while the visual branding says "X".Proposed fix
<Link - aria-label="Follow on Twitter" + aria-label="Follow on X" className="text-zinc-500 transition hover:text-[`#00843D`] dark:text-zinc-400 dark:hover:text-yellow-400" href="https://twitter.com/aossie_org" >src/components/Footer.jsx (1)
32-34:⚠️ Potential issue | 🟡 MinorCopyright year is
2016-2025but it's now 2026.This is outside the scope of the orgStats change, but worth updating while you're editing this file.
Proposed fix
- © 2016-2025 AOSSIE. All rights reserved. + © 2016-2026 AOSSIE. All rights reserved.
This PR centralizes the organization statistics (Active Projects, Total Contributors, Community Count)
into a shared configuration file to establish a single source of truth.
Previously, these values were hard-coded separately across pages, which caused inconsistencies
(e.g., differing Community Count values on the home and about pages).
Changes:
src/config/orgStats.jsas a shared configuration.This ensures future updates require changes in only one place.
Fixes #575
Summary by CodeRabbit
Release Notes
New Features
Refactor