Improve landing page conversion by directing primary CTA to upload page#494
Improve landing page conversion by directing primary CTA to upload page#494mahek2016 wants to merge 2 commits intoAOSSIE-Org:mainfrom
Conversation
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe Home page component was updated to redirect users to the /input route instead of /question-type when clicking primary action buttons. An inline comment was added to document this navigation change, supporting the landing page conversion optimization effort. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~5 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 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
🧹 Nitpick comments (1)
eduaid_web/src/pages/Home.jsx (1)
88-88: Remove PR-context inline comment from production JSX.Line 88 (
Only change here...) is diff-context metadata and will go stale; prefer removing it or replacing it with a product-intent comment.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@eduaid_web/src/pages/Home.jsx` at line 88, In Home.jsx remove the diff-context comment "/* ✅ Only change here: /question-type → /input */" from the JSX (inside the Home component) — either delete it entirely or replace it with a product-intent comment that explains the UI intent (e.g., why the route changed) so the JSX no longer contains PR-specific metadata.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@eduaid_web/src/pages/Home.jsx`:
- Around line 89-101: Remove the nested <button> elements inside the React
Router <Link> components for the "/input" and "/history" links and move the
button-related attributes/classes to the <Link> itself (e.g., copy classes like
"w-full sm:w-auto items-center text-lg flex justify-center gap-3 text-white px-6
py-3 border-gradient hover:wave-effect rounded-md transition-all duration-300"
and the inner text plus <img> into the Link), so that <Link to="/input"> and
<Link to="/history"> are the interactive elements instead of wrapping a
<button>; ensure semantics and keyboard behavior remain by preserving role/aria
attributes if used elsewhere.
---
Nitpick comments:
In `@eduaid_web/src/pages/Home.jsx`:
- Line 88: In Home.jsx remove the diff-context comment "/* ✅ Only change here:
/question-type → /input */" from the JSX (inside the Home component) — either
delete it entirely or replace it with a product-intent comment that explains the
UI intent (e.g., why the route changed) so the JSX no longer contains
PR-specific metadata.
1 participant
Addressed Issues:
Fixes #483
Screenshots/Recordings:
Additional Notes:
This PR improves the landing page conversion flow by directing the primary CTA button to the upload page (
/input) instead of the intermediate question-type page.This reduces friction and aligns with the issue goal of making the product clearer within 5–10 seconds for first-time users.
No structural, styling, or layout changes were introduced.
Checklist
AI Usage Disclosure
I have used the following AI tools:
ChatGPT (for guidance and review)
Summary by CodeRabbit
Features
Style