fix: remove duplicate "Learning Requests" label in mobile navigation …#1015
fix: remove duplicate "Learning Requests" label in mobile navigation …#1015TarekMGamal wants to merge 1 commit intoalphaonelabs:mainfrom
Conversation
👀 Peer Review RequiredHi @TarekMGamal! This pull request does not yet have a peer review. Before this PR can be merged, please request a review from one of your peers:
Thank you for contributing! 🎉 |
|
Note
|
| Cohort / File(s) | Summary |
|---|---|
Mobile Navigation Template web/templates/base.html |
Updated COMMUNITY > Surveys menu item label from "Learning Requests" to "Surveys" to eliminate duplicate label in mobile navigation dropdown. |
Estimated code review effort
🎯 1 (Trivial) | ⏱️ ~2 minutes
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
| Check name | Status | Explanation |
|---|---|---|
| Description Check | ✅ Passed | Check skipped - CodeRabbit’s high-level summary is enabled. |
| Title check | ✅ Passed | The title accurately describes the main change: fixing a duplicate 'Learning Requests' label in mobile navigation by updating it to 'Surveys'. |
| Linked Issues check | ✅ Passed | The PR successfully addresses issue #1001 by changing the duplicate 'Learning Requests' label to 'Surveys' in the COMMUNITY navigation section. |
| Out of Scope Changes check | ✅ Passed | The change is narrowly scoped to fixing the duplicate label issue; only one line modified in base.html with no extraneous changes. |
| Docstring Coverage | ✅ Passed | No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. |
✏️ Tip: You can configure your own custom pre-merge checks in the settings.
✨ Finishing Touches
🧪 Generate unit tests (beta)
- Create PR with unit tests
- Post copyable unit tests in a comment
Tip
Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.
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 help to get the list of available commands and usage tips.
Related issues
Fixes #1001
Checklist
Overview
This PR fixes a duplicate "Learning Requests" label in the mobile navigation menu's COMMUNITY section by correcting the mislabeled "Surveys" menu item.
Changes Made
File:
web/templates/base.htmlhref="{% url 'surveys' %}") — only the displayed text needed correctionImpact