Skip to content

Revert "feat: Refactor admin view toggle into a dedicated component and enhan…"#521

Merged
PaulicStudios merged 1 commit intodevfrom
revert-516-group-phase-bracket-design-improvements
Feb 6, 2026
Merged

Revert "feat: Refactor admin view toggle into a dedicated component and enhan…"#521
PaulicStudios merged 1 commit intodevfrom
revert-516-group-phase-bracket-design-improvements

Conversation

@PaulicStudios
Copy link
Member

@PaulicStudios PaulicStudios commented Feb 6, 2026

Reverts #516

Summary by CodeRabbit

  • New Features

    • Added admin toggle UI for reveal controls on bracket and group views.
  • UI/UX Updates

    • Simplified page layouts for bracket and group tournament views.
    • Removed clickable navigation from match nodes for cleaner interaction.
    • Updated graph styling, spacing, and dimensions for improved visual presentation.
  • Chores

    • Removed seed user command from available scripts.

@PaulicStudios PaulicStudios merged commit 9834a47 into dev Feb 6, 2026
2 checks passed
@coderabbitai
Copy link

coderabbitai bot commented Feb 6, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

This PR removes backend seed data generation functionality and simplifies the frontend by disabling admin view toggles, eliminating match node navigation, and refactoring graph rendering with simplified coordinate computation and edge removal across bracket and groups features.

Changes

Cohort / File(s) Summary
Backend Seed Data Removal
api/README.md, api/package.json, api/src/scripts/seed-users-teams.ts
Deleted seed-users-teams.ts script and removed seed:users npm script; formatting changes to README converted bullet markers from hyphens to asterisks.
Bracket Feature Simplification
frontend/app/events/[id]/bracket/actions.tsx, frontend/app/events/[id]/bracket/page.tsx, frontend/app/events/[id]/bracket/graphView.tsx
Disabled admin view toggle in actions (now renders empty), simplified page layout (removed header, container styling), and refactored graphView with new createTreeCoordinate helper, removed edges logic, and streamlined node rendering.
Groups Feature Simplification
frontend/app/events/[id]/groups/actions.tsx, frontend/app/events/[id]/groups/page.tsx, frontend/app/events/[id]/groups/graphView.tsx
Disabled admin toggle UI in actions, simplified page layout, removed RoundNode component, reduced layout constants, adjusted styling, and changed effect dependencies to only rely on matches.
Match Node Component
frontend/components/match/MatchNode.tsx
Removed routing imports and Handle elements, eliminated clickable team names, removed optional showTargetHandle/showSourceHandle properties from public interface, and added internal formatTeamName helper.
Team Creation Form
frontend/components/team/TeamCreationSection.tsx
Changed submission trigger from form-based onSubmit to button onClick handler while preserving validation and loading states.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

Suggested reviewers

  • Peu77

Poem

🐰 Toggles dimmed and edges fade,
Admin views are now unmade,
Nodes navigate no more in flight,
Simpler graphs, a cleaner sight!
Hop along—the cleanup's bright!

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch revert-516-group-phase-bracket-design-improvements

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.

@PaulicStudios PaulicStudios deleted the revert-516-group-phase-bracket-design-improvements branch February 6, 2026 14:59
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.

1 participant