Skip to content

fix: remove duplicate "Learning Requests" label in mobile navigation …#1015

Open
TarekMGamal wants to merge 1 commit intoalphaonelabs:mainfrom
TarekMGamal:fix/mobile-nav-duplicate-label
Open

fix: remove duplicate "Learning Requests" label in mobile navigation …#1015
TarekMGamal wants to merge 1 commit intoalphaonelabs:mainfrom
TarekMGamal:fix/mobile-nav-duplicate-label

Conversation

@TarekMGamal
Copy link

@TarekMGamal TarekMGamal commented Mar 9, 2026

Related issues

Fixes #1001

Screenshot 2026-03-10 013905

Checklist

  • Did you run the pre-commit? (If not, your PR will most likely not pass — please ensure it passes pre-commit)
  • Did you test the change? (Ensure you didn’t just prompt the AI and blindly commit — test the code and confirm it works)
  • Added screenshots to the PR description (if applicable)

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.html

  • Changed the display label for the surveys menu item from "Learning Requests" to "Surveys" (line 631)
  • The link URL was already correct (href="{% url 'surveys' %}") — only the displayed text needed correction
  • No structural or functionality changes

Impact

  • User Experience: Mobile users will now see distinct labels ("Learning Requests" and "Surveys") in the COMMUNITY navigation section, matching the intended menu structure
  • Scope: Minimal — purely cosmetic fix to display text
  • Risk: Very low — single-line label change with no functional implications

@github-actions github-actions bot added the files-changed: 1 PR changes 1 file label Mar 9, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Mar 9, 2026

👀 Peer Review Required

Hi @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:

  • Go to the PR page and click "Reviewers" on the right sidebar.
  • Select a team member or contributor to review your changes.
  • Once they approve, this reminder will be automatically removed.

Thank you for contributing! 🎉

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 9, 2026

Note

.coderabbit.yaml has unrecognized properties

CodeRabbit is using all valid settings from your configuration. Unrecognized properties (listed below) have been ignored and may indicate typos or deprecated fields that can be removed.

⚠️ Parsing warnings (1)
Validation error: Unrecognized key(s) in object: 'tools'
⚙️ Configuration instructions
  • Please see the configuration documentation for more information.
  • You can also validate your configuration using the online YAML validator.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository: alphaonelabs/coderabbit/.coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 9e53a70e-548c-4087-8bb2-1da5c3ab9de2

📥 Commits

Reviewing files that changed from the base of the PR and between c94caf8 and 259220c.

📒 Files selected for processing (1)
  • web/templates/base.html

Walkthrough

A single text label correction in the mobile navigation template. The COMMUNITY > Surveys menu item label was changed from "Learning Requests" to "Surveys" to resolve a duplicate label ambiguity in the navigation menu.

Changes

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

files-changed: 1 PR changes 1 file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: Duplicate "Learning Requests" label in mobile navigation menu

1 participant