Fix: classroom chat button redirects to 404 navigation page.#694
Fix: classroom chat button redirects to 404 navigation page.#694ArrushiTripathi2429 wants to merge 2 commits into
Conversation
- Add missing NEXT_PUBLIC_CHAT_SYSTEM_URL to .env.example - Remove duplicate grid div in features section causing JSX syntax error - Add missing scrollProgress state and scroll event listener
|
@ArrushiTripathi2429 is attempting to deploy a commit to the Karan Mani Tripathi 's projects Team on Vercel. A member of the Team first needs to authorize it. |
Thanks for using CodeAnt! 🎉We're free for open-source projects. if you're enjoying it, help us grow by sharing. Share on X · |
github-actions
Bot
added
gssoc'26
WalkthroughA new required environment variable section is added to ChangesChat System Environment Variable
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes 🚥 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 |
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
⭐ Star Required
Hi @ArrushiTripathi2429! Thank you for your contribution to DoubtDesk.
Before we can complete the review and merge this pull request, please star the DoubtDesk repository.
Once you have starred the repository, please drop a comment here saying "done" and we will proceed with reviewing your PR. Thank you!
|
@coderabbitai review |
codeant-ai
Bot
added
the
size:XS
github-actions
Bot
removed
the
size:XS
![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 @.env.example:
- Around line 55-62: The .env.example file has formatting issues that need to be
corrected. Remove the extra blank lines that appear before the Chat System
configuration section comment. Additionally, ensure the file ends with a single
trailing newline after the NEXT_PUBLIC_CHAT_SYSTEM_URL variable assignment. The
Chat System section should have only one blank line separating it from any
preceding content, and the file should properly terminate with a newline
character.
🪄 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: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 5423c30b-3523-4dce-b51f-7b7afe23a6b9
📒 Files selected for processing (1)
.env.example
|
|
||
|
|
||
|
|
||
| # Chat System (Required for Classroom Chat & Peer-to-Peer features) | ||
| # Run the chat-system app separately and set its URL here. | ||
| # Local development: http://localhost:3001 | ||
| # Production: https://your-chat-system.vercel.app | ||
| NEXT_PUBLIC_CHAT_SYSTEM_URL=http://localhost:3001 No newline at end of file |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
Remove extra blank lines and add trailing newline.
Static analysis flags formatting issues: lines 56–57 have extra blank lines, and the file is missing a trailing blank line.
🔧 Proposed fix for formatting
---- unchanged lines 1-54 not shown ----
-
-
-
# Chat System (Required for Classroom Chat & Peer-to-Peer features)
# Run the chat-system app separately and set its URL here.
# Local development: http://localhost:3001
# Production: https://your-chat-system.vercel.app
NEXT_PUBLIC_CHAT_SYSTEM_URL=http://localhost:3001
+📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| # Chat System (Required for Classroom Chat & Peer-to-Peer features) | |
| # Run the chat-system app separately and set its URL here. | |
| # Local development: http://localhost:3001 | |
| # Production: https://your-chat-system.vercel.app | |
| NEXT_PUBLIC_CHAT_SYSTEM_URL=http://localhost:3001 | |
| # Chat System (Required for Classroom Chat & Peer-to-Peer features) | |
| # Run the chat-system app separately and set its URL here. | |
| # Local development: http://localhost:3001 | |
| # Production: https://your-chat-system.vercel.app | |
| NEXT_PUBLIC_CHAT_SYSTEM_URL=http://localhost:3001 | |
🧰 Tools
🪛 dotenv-linter (4.0.0)
[warning] 56-56: [ExtraBlankLine] Extra blank line detected
(ExtraBlankLine)
[warning] 57-57: [ExtraBlankLine] Extra blank line detected
(ExtraBlankLine)
[warning] 62-62: [EndingBlankLine] No blank line at the end of the file
(EndingBlankLine)
🤖 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 @.env.example around lines 55 - 62, The .env.example file has formatting
issues that need to be corrected. Remove the extra blank lines that appear
before the Chat System configuration section comment. Additionally, ensure the
file ends with a single trailing newline after the NEXT_PUBLIC_CHAT_SYSTEM_URL
variable assignment. The Chat System section should have only one blank line
separating it from any preceding content, and the file should properly terminate
with a newline character.
Source: Linters/SAST tools
ArrushiTripathi2429
changed the title
knoxiboy
added
level:beginner
2 participants
User description
Description
Classroom Chat sidebar button was redirecting to
/chaton the main app, causing a 404 error. The chat-system runs as a separate Next.js app on a different port, butNEXT_PUBLIC_CHAT_SYSTEM_URLwas missing from.env.example, so contributors didn't know to set it.Related Issue
Closes #659
Changes
NEXT_PUBLIC_CHAT_SYSTEM_URLto.env.examplewith documentation<div>in features section causing JSX syntax errorscrollProgressuseState and scroll event listener inpage.tsxTesting
Locally tested — chat system opens correctly on
localhost:3001/chatwhenenv variable is set to
http://localhost:3001Screenshot
Type of Change
How Has This Been Tested?
npm run devlocalhost:3001NEXT_PUBLIC_CHAT_SYSTEM_URL=http://localhost:3001is set in.env.localChecklist
CodeAnt-AI Description
Add the Classroom Chat app URL to the environment setup
What Changed
Impact
✅ Fewer setup errors for Classroom Chat✅ Clearer local development setup💡 Usage Guide
Checking Your Pull Request
Every time you make a pull request, our system automatically looks through it. We check for security issues, mistakes in how you're setting up your infrastructure, and common code problems. We do this to make sure your changes are solid and won't cause any trouble later.
Talking to CodeAnt AI
Got a question or need a hand with something in your pull request? You can easily get in touch with CodeAnt AI right here. Just type the following in a comment on your pull request, and replace "Your question here" with whatever you want to ask:
This lets you have a chat with CodeAnt AI about your pull request, making it easier to understand and improve your code.
Example
Preserve Org Learnings with CodeAnt
You can record team preferences so CodeAnt AI applies them in future reviews. Reply directly to the specific CodeAnt AI suggestion (in the same thread) and replace "Your feedback here" with your input:
This helps CodeAnt AI learn and adapt to your team's coding style and standards.
Example
Retrigger review
Ask CodeAnt AI to review the PR again, by typing:
Check Your Repository Health
To analyze the health of your code repository, visit our dashboard at https://app.codeant.ai. This tool helps you identify potential issues and areas for improvement in your codebase, ensuring your repository maintains high standards of code health.
Summary by CodeRabbit