Skip to content

Show game ratings on cards - #13

Merged
alng20 merged 1 commit into
mainfrom
alng20-show-game-ratings
Sep 25, 2026
Merged

alng20 merged 1 commit into
mainfrom
alng20-show-game-ratings

Conversation

@alng20

@alng20 alng20 commented Sep 25, 2026

Copy link
Copy Markdown
Owner

Description

Game cards currently omit ratings, making it harder for visitors to compare games while browsing. This change displays each game's star rating and shows "No rating yet" when the rating is null.

Related Issue

Closes #12

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Test update
  • Refactor (no functional changes)

Changes Made

  • Render the existing star-rating component in each game card and show "No rating yet" for unrated games.
  • Add an end-to-end assertion that a game card displays its rating.

Testing

Data Layer Changes

  • Ran npm run test:unit - all tests pass
  • Added/updated Vitest tests for data-layer changes
  • Generated a migration (npm run db:generate) for any schema change

Frontend Changes

  • Ran npm run test:e2e - all tests pass
  • Added data-testid attributes to interactive elements
  • Verified build succeeds (npm run build)

Checklist

  • My code follows the project's coding standards
  • I have used explicit TypeScript types for function parameters and return values
  • I have built the UI with Astro components and Tailwind CSS utility classes (dark theme)
  • I have updated documentation (README, instruction files) if needed
  • My changes are focused on a single concern
  • I have written clear commit messages explaining what and why

Additional Notes

Lint and type checks passed. The E2E test run built the site before running the browser tests.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@alng20
alng20 merged commit e602c65 into main Sep 25, 2026
4 checks passed
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.

Show game ratings on game cards

1 participant