-
Notifications
You must be signed in to change notification settings - Fork 1
Add bracket predictions #25
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
sergical
wants to merge
10
commits into
syntaxfm:main
Choose a base branch
from
sergical:sergical/bracket-picking
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
8547fc6 to
d9bd981
Compare
- Add /test route with bracket prediction functionality - Add predictions API endpoints and usePredictions hook - Redesign login card with 1991 retro paper-cut aesthetic - Fix bracket finalist node centering Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Restore view-only Bracket.tsx from main for index page - Create InteractiveBracket.tsx with picking functionality - Update test.tsx to use InteractiveBracket - Use single yellow ring color instead of left/right colors - Make placeholders gray with grayscale filter - Players only grayscale when eliminated Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- API now deletes existing predictions before inserting new ones - Reset + save now actually clears predictions from database - Match FIT_VIEW_PADDING between Bracket and InteractiveBracket Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Merge InteractiveBracket.tsx into Bracket.tsx with isInteractive prop - Non-interactive mode (homepage): all TBD slots, yellow/red rings, 50px sizing, no grayscale - Interactive mode (/test): predictions-based, gray rings for TBD, larger finalist/championship sizing - Delete InteractiveBracket.tsx (logic merged into Bracket.tsx) - Update test.tsx to use Bracket with isInteractive prop Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Redirect to /test after GitHub OAuth login - Auto-scroll to #bracket section after login (once per session) - Clear scroll flag on logout - Add trustedOrigins for CSRF protection - Add GitHub OAuth setup docs to CLAUDE.md - Update players: add Cassidy Williams, Josh Comeau, Kyle Cook Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Style React Flow controls with dark theme (black bg, yellow accents) - Stack finalists and championship vertically with centered text - Add progressive photo sizing: finalists 65px, championship 80px - Center championship node between finalists - Add mobile responsive styles Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Add intermediate breakpoint with 200px cards to ensure 2 columns fit in mid-range viewports where 300px cards were too wide. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
React Flow handles zoom/pan on mobile, so node size overrides aren't needed. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
e7d12f4 to
cb4ee69
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
/testroute with bracket prediction functionalitySetup
Database migrations
Environment variables
Make sure
.dev.varshas the GitHub OAuth credentials:Testing
/). After signing in, you'll be redirected to/- manually navigate to/testto see the prediction functionality.pnpm devhttp://localhost:3000/test/after auth)/testto make predictions🤖 Generated with Claude Code