Add FAQ navigation CTA button Fixes #42#65
Conversation
🚀 PR Received SuccessfullyHello @bhavya-tyagi20, Thank you for taking the initiative to contribute to this project. Please ensure that your PR follows all project guidelines properly before requesting review.
|
📝 WalkthroughWalkthroughFAQ navigation link added to FAQSection component: ChangesFAQ Navigation CTA
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related issues
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)
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 |
![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
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@frontend/src/components/explore/FAQSection.jsx`:
- Around line 77-82: The Link in FAQSection.jsx currently points to "/faq" (the
Link component instance with to="/faq") but there is no "/faq" route registered
in the router; either register a route handler for "/faq" in your router
configuration (e.g., add a Route for path="/faq" that renders the FAQ page
component) or change the Link's target to an existing route (for example to the
existing help/knowledgebase route) so the CTA doesn't hit NotFound; update
either the router's route table or the Link's to value accordingly.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 6e44b38d-f133-4b37-9128-2b138a63e1ae
📒 Files selected for processing (3)
frontend/src/App.jsxfrontend/src/components/explore/FAQSection.jsxfrontend/src/pages/ExplorePage.jsx
| <Link | ||
| to="/faq" | ||
| className="border-3 border-black px-5 py-2 text-sm sm:text-base font-black uppercase bg-white tracking-widest shadow-[5px_5px_0_0_rgba(0,0,0,1)] text-black transition-all duration-200 hover:bg-black hover:text-white hover:translate-x-1 hover:translate-y-1" | ||
| > | ||
| View All FAQs | ||
| </Link> |
There was a problem hiding this comment.
/faq CTA target is not registered in the current router.
Line 78 navigates to /faq, but the provided route table has no /faq route, so this CTA likely lands on NotFound. Please either add a matching route or point this CTA to an existing destination until /faq is available.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@frontend/src/components/explore/FAQSection.jsx` around lines 77 - 82, The
Link in FAQSection.jsx currently points to "/faq" (the Link component instance
with to="/faq") but there is no "/faq" route registered in the router; either
register a route handler for "/faq" in your router configuration (e.g., add a
Route for path="/faq" that renders the FAQ page component) or change the Link's
target to an existing route (for example to the existing help/knowledgebase
route) so the CTA doesn't hit NotFound; update either the router's route table
or the Link's to value accordingly.
|
@coderabbitai The Current scope of this PR was limited to adding the CTA button and preparing navigation flow for future FAQ expansion. Redirecting to the fallback route temporarily is acceptable for the current stage of the project. |
|
Tip For best results, initiate chat on the files or code changes.
Would you like me to open a follow-up GitHub issue to track the implementation of the ✏️ Learnings added
|
|
@bhavya-tyagi20 Proper implementation. Scope was followed correctly, changes are controlled, and unnecessary modifications were avoided. Continue maintaining this level of contribution quality. |
kunalverma2512
added
enhancement
2 participants
Fixes #42
Added a “View All FAQs” CTA button in the FAQ section to improve navigation and prepare for future scalability.
The button routes to /faq using React Router. No new routes or pages were added, as per issue constraints.
Changes are limited strictly to the FAQ section and maintain UI consistency and responsiveness.
Summary by CodeRabbit