Skip to content

fixed the tournament breadcrumb name issue - #557

Open
insanekrishnna wants to merge 1 commit into
AOSSIE-Org:mainfrom
insanekrishnna:fix/tournament-name-issue
Open

insanekrishnna wants to merge 1 commit into
AOSSIE-Org:mainfrom
insanekrishnna:fix/tournament-name-issue

Conversation

@insanekrishnna

@insanekrishnna insanekrishnna commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

Addressed Issues

Fixes #543

Changes

Updated the tournament bracket breadcrumb to display the tournament’s actual name instead of its numeric ID.

  • Reused the tournament object already passed through React Router state when opening or joining a tournament.
  • Added dynamic breadcrumb-label resolution for tournament bracket routes.
  • Replaced the tournament ID segment with the corresponding tournament name.
  • Centralized the existing breadcrumb label formatting to avoid duplicate formatting logic.
  • Preserved existing special labels such as Support DebateAI and Bot Selection.
  • Added a safe fallback to the tournament ID when route data is unavailable.
  • Left tournament routing, bracket behavior, and other breadcrumbs unchanged.

The breadcrumb now displays paths such as:

  • Home → Tournament → Spring Showdown → Bracket
  • Home → Tournament → SummerSlam → Bracket
  • Home → Tournament → Rapid Fire Blitz → Bracket

Testing

  • Verified a bracket breadcrumb using Spring Showdown.
  • Verified a bracket breadcrumb using SummerSlam.
  • Confirmed the tournament name replaces the numeric route ID.
  • Confirmed the Tournament and Bracket breadcrumb segments remain unchanged.
  • Confirmed direct bracket URLs without tournament route data safely fall back to the ID.
  • Confirmed existing non-tournament breadcrumb formatting remains unchanged.
  • Ran focused ESLint checks with no errors.
  • Ran git diff --check successfully.
  • Confirmed only frontend/src/components/Header.tsx was modified.
  • Confirmed no tournament or bracket functionality was changed.

Screenshots/Recordings

Before

The tournament bracket breadcrumb displayed the numeric tournament ID:

Home → Tournament → 1789587026644 → Bracket

tournament_name_issue.mp4

After

The breadcrumb displays the tournament name obtained from the existing tournament route data:

Home → Tournament → Spring Showdown → Bracket

sol_tournament_breadcrumb.mp4

@coderabbitai

coderabbitai Bot commented Sep 17, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 14 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: fa8d9c23-6ea9-4bee-8434-bdd9bdd159fb

📥 Commits

Reviewing files that changed from the base of the PR and between 9f90f9b and 51c32b3.

📒 Files selected for processing (1)
  • frontend/src/components/Header.tsx

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.

@insanekrishnna
insanekrishnna force-pushed the fix/tournament-name-issue branch from d68b77e to 51c32b3 Compare September 18, 2026 11:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG]: Tournament breadcrumb displays tournament ID instead of tournament name

1 participant