fix: mobile navbar overflow + README port/setup corrections#45
fix: mobile navbar overflow + README port/setup corrections#45Priyanshu-byte-coder wants to merge 1 commit into
Conversation
… docs - Add max-h and overflow-y-auto to mobile menu so auth buttons stay accessible when Tools section expands - Fix port references in README (5000 -> 8000) to match server/.env.example - Create frontend/.env.example with correct VITE_API_BASE_URL - Add MongoDB Atlas, SMTP setup guides and troubleshooting table to README
📝 WalkthroughWalkthroughThis PR addresses setup onboarding issues and mobile navigation usability by synchronizing documented port configurations across frontend and backend to 8000, adding MongoDB Atlas and SMTP setup guides with troubleshooting reference, and improving mobile menu scrolling behavior in the Navbar component. ChangesDevelopment Setup and Configuration Synchronization
Mobile Navigation Usability
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~4 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)
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 |
🚀 PR Received SuccessfullyHello @Priyanshu-byte-coder, Thank you for taking the initiative to contribute to this project. Please ensure that your PR follows all project guidelines properly before requesting review.
|
![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)
README.md (1)
867-916: ⚡ Quick winMake the new setup sections discoverable from the Table of Contents.
MongoDB Atlas Setup,SMTP Setup, andTroubleshootingwere added but are not linked in the ToC, which hurts onboarding navigation. Consider adding ToC anchors (and optionally placing these sections beforeLicense).🤖 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 `@README.md` around lines 867 - 916, The README added three new sections ("MongoDB Atlas Setup", "SMTP Setup", "Troubleshooting") but they are not referenced from the Table of Contents; update the ToC to include anchor links to these headers and ensure the headers use consistent markdown heading levels (e.g., "### MongoDB Atlas Setup", "### SMTP Setup", "### Troubleshooting") so anchors work, or move these sections before the "License" section if preferred; add entries in the TOC like "MongoDB Atlas Setup", "SMTP Setup", and "Troubleshooting" pointing to their markdown anchors so readers can navigate to those sections easily.
🤖 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 `@README.md`:
- Line 874: Replace the README guidance that suggests adding "0.0.0.0/0" under
Network Access with a least-privilege recommendation: instruct users to add only
necessary IP ranges or specific CIDRs for their environment and to avoid using
0.0.0.0/0 except as a temporary, short-lived fallback; if you keep mention of
0.0.0.0/0 include an explicit, prominent warning about the security risk and
require steps to remove it before production.
---
Nitpick comments:
In `@README.md`:
- Around line 867-916: The README added three new sections ("MongoDB Atlas
Setup", "SMTP Setup", "Troubleshooting") but they are not referenced from the
Table of Contents; update the ToC to include anchor links to these headers and
ensure the headers use consistent markdown heading levels (e.g., "### MongoDB
Atlas Setup", "### SMTP Setup", "### Troubleshooting") so anchors work, or move
these sections before the "License" section if preferred; add entries in the TOC
like "MongoDB Atlas Setup", "SMTP Setup", and "Troubleshooting" pointing to
their markdown anchors so readers can navigate to those sections easily.
🪄 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: 85af6b01-009d-4155-9f7d-e7fe9d6c9c70
📒 Files selected for processing (3)
README.mdfrontend/.env.examplefrontend/src/components/shared/Navbar.jsx
| 1. Go to [cloud.mongodb.com](https://cloud.mongodb.com) and create a free account | ||
| 2. Create a new **Cluster** (free M0 tier works fine) | ||
| 3. Under **Database Access**, create a user with read/write permissions | ||
| 4. Under **Network Access**, add your IP (or `0.0.0.0/0` for development) |
There was a problem hiding this comment.
Avoid recommending 0.0.0.0/0 as a default Atlas network rule.
This guidance weakens security posture and is easy to copy into non-dev environments. Prefer recommending least-privilege IP allowlists, with 0.0.0.0/0 only as a short-lived fallback plus explicit warning.
🤖 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 `@README.md` at line 874, Replace the README guidance that suggests adding
"0.0.0.0/0" under Network Access with a least-privilege recommendation: instruct
users to add only necessary IP ranges or specific CIDRs for their environment
and to avoid using 0.0.0.0/0 except as a temporary, short-lived fallback; if you
keep mention of 0.0.0.0/0 include an explicit, prominent warning about the
security risk and require steps to remove it before production.
1 participant
Summary
max-h-[calc(100vh-4rem)] overflow-y-autoto the mobile menu container so the Tools accordion expanding no longer pushes Login/Signup/Logout buttons off-screenlocalhost:5000references tolocalhost:8000to matchserver/.env.example; createdfrontend/.env.example; added MongoDB Atlas and SMTP setup guides; added a Troubleshooting tableChanges
frontend/src/components/shared/Navbar.jsx— mobile menu div gets scrollable max-heightREADME.md— port fix + MongoDB Atlas setup + SMTP setup + Troubleshooting sectionfrontend/.env.example— new file withVITE_API_BASE_URL=http://localhost:8000/apiTest plan
Summary by CodeRabbit
Documentation
Bug Fixes