Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Make the game list searchable by user #6

Merged
merged 14 commits into from
Feb 3, 2025
Merged

Make the game list searchable by user #6

merged 14 commits into from
Feb 3, 2025

Conversation

julienjme
Copy link
Collaborator

@julienjme julienjme commented Jan 31, 2025 •
edited
Loading

https://trello.com/c/K6f3PC8N/57-1-as-john-i-want-to-follow-the-activity-of-a-given-user

image

  • Deploy Postrgres and SPA
  • Create the SQL View
  • SearchInput on one player
  • SearchInput on both players
  • AutocompleteInput to filter by winner
  • SelectInput filter to filter by game status

slax57
slax57 reviewed Jan 31, 2025
View reviewed changes
src/games/GameList.tsx Outdated Show resolved Hide resolved
supabase/migrations/20240130152000_create_model.sql Outdated Show resolved Hide resolved
@julienjme
Copy link
Collaborator Author

image Added autocomplete on winner and select on game_status

slax57
slax57 reviewed Feb 3, 2025
View reviewed changes
Copy link
Contributor

@slax57 slax57 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Otherwise it's going in the right direction, good! 🙂

supabase/migrations/20240130152000_create_model.sql Outdated Show resolved Hide resolved
@julienjme julienjme requested a review from slax57 February 3, 2025 13:27
@slax57 slax57 merged commit 86b007d into main Feb 3, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@slax57 slax57 slax57 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@julienjme @slax57