Fixed homepage links#391
Conversation
palisadoes
added
the
ignore-sensitive-files-pr
Our Pull Request Approval ProcessWe have these basic policies to make the approval process smoother for our volunteer team. Testing Your CodePlease make sure your code passes all tests. Our test code coverage system will fail if either of these two conditions occur:
The process helps maintain the overall reliability of the code base and is a prerequisite for getting your PR approved. Assigned reviewers regularly review the PR queue and tend to focus on PRs that are passing. ReviewersWhen your PR has been assigned reviewers contact them to get your code reviewed and approved via:
Reviewing Your CodeYour reviewer(s) will have the following roles:
🎯 Contacting the person who assigned the mentors is not advised unless they make a request. Do not @ the person who did the assignment otherwise. |
WalkthroughDocumentation configuration and homepage navigation updates: quote normalization (single to double quotes) and spacing adjustments in Docusaurus config, plus updates to CTA button links and text on the homepage. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes
Poem
Pre-merge checks and finishing touches❌ Failed checks (2 warnings)
✅ Passed checks (1 passed)
✨ Finishing touches
🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
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 |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #391 +/- ##
===========================================
+ Coverage 82.53% 85.94% +3.40%
===========================================
Files 105 128 +23
Lines 6961 10797 +3836
Branches 0 749 +749
===========================================
+ Hits 5745 9279 +3534
- Misses 1216 1503 +287
- Partials 0 15 +15 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
docs/docusaurus.config.ts (1)
89-93: Remove duplicatecolorModeconfiguration.The
colorModeobject is defined twice inthemeConfig(lines 89-93 and 150-154), with conflicting values forrespectPrefersColorScheme(false vs. true). JavaScript objects only retain the last definition, making this confusing and error-prone.Apply this diff to remove the first definition and keep only the second:
- colorMode: { - defaultMode: "light", - disableSwitch: false, - respectPrefersColorScheme: false, - }, footer: { style: "dark",Also applies to: 150-154
🧹 Nitpick comments (1)
docs/docusaurus.config.ts (1)
1-3: Quote normalization improves consistency.The formatting changes standardize string literals to use double quotes throughout the configuration file, improving code consistency and likely aligning with project linting rules.
Also applies to: 10-10, 13-13, 19-19, 22-22, 30-31, 36-36, 39-39, 43-44, 47-47
📜 Review details
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (3)
docs/docusaurus.config.ts(4 hunks)docs/package.json(1 hunks)docs/src/pages/index.tsx(1 hunks)
🧰 Additional context used
🧠 Learnings (2)
📚 Learning: 2025-09-04T20:04:37.212Z
Learnt from: Abhi-MS
Repo: PalisadoesFoundation/switchmap-ng PR: 337
File: docs/docs/frontend/types/graphql/GetZoneDevices.md:59-64
Timestamp: 2025-09-04T20:04:37.212Z
Learning: Documentation files in docs/docs/frontend/ are autogenerated by TypeDoc from TypeScript source code and should not be manually edited. Changes should be made to the source TypeScript files instead.
Applied to files:
docs/src/pages/index.tsxdocs/docusaurus.config.ts
📚 Learning: 2025-07-24T19:36:00.577Z
Learnt from: Abhi-MS
Repo: PalisadoesFoundation/switchmap-ng PR: 330
File: docs/src/components/HeroImage/HeroSection.css:12-15
Timestamp: 2025-07-24T19:36:00.577Z
Learning: In Docusaurus projects, using !important declarations may be necessary to override framework-injected styles that have high specificity from CSS modules or inline styles, especially when increasing selector specificity alone is insufficient.
Applied to files:
docs/docusaurus.config.ts
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: Test Application
🔇 Additional comments (3)
docs/src/pages/index.tsx (1)
16-22: Simplified entry point looks good.Changing the "Get Started" link from the specific installation page to the general docs landing page (
/docs) provides a better initial user experience, allowing users to explore the documentation structure before diving into installation.docs/docusaurus.config.ts (1)
100-104: The community forums URL is correct.The link update from Slack to Forums pointing to
https://community.talawa.io/is valid. This is the official Talawa Community forum (XenForo-based) and includes dedicated sections for Switchmap-NG discussions alongside other community resources.docs/package.json (1)
48-48: ThepackageManagerfield correctly enforces pnpm@10.18.3 with an integrity hash. pnpm version 10.18.3 is a confirmed release, and using the packageManager field with Corepack is the recommended approach for reproducible builds and version enforcement.
1 participant
Fixed homepage links
Summary by CodeRabbit
Style
Chores
✏️ Tip: You can customize this high-level summary in your review settings.