Skip to content

Add leaderboardId field to Achievement table (with backfill) - #953

Open
tahminator wants to merge 4 commits into
mainfrom
add-leaderboard-id-field-to-achievement-table
Open

Add leaderboardId field to Achievement table (with backfill)#953
tahminator wants to merge 4 commits into
mainfrom
add-leaderboard-id-field-to-achievement-table

Conversation

@tahminator

@tahminator tahminator commented Aug 2, 2026

Copy link
Copy Markdown
Member

Description of changes

Add leaderboardId field to Achievement table. This includes backfilling this data in production.

Checklist before review

  • I have done a thorough self-review of the PR
  • Copilot has reviewed my latest changes, and all comments have been fixed and/or closed.
  • If I have made database changes, I have made sure I followed all the db repo rules listed in the wiki here. (check if no db changes)
  • All tests have passed
  • I have successfully deployed this PR to staging
  • I have done manual QA in both dev (and staging if possible) and attached screenshots below.

Screenshots

Dev

image

Stack created with GitHub Stacks CLIGive Feedback 💬

@tahminator
tahminator requested a review from a team August 2, 2026 03:13
@github-actions

github-actions Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Available PR Commands

  • /ai - Triggers all AI review commands at once
  • /review - AI review of the PR changes
  • /describe - AI-powered description of the PR
  • /improve - AI-powered suggestions
  • /deploy - Deploy to staging

See: https://github.com/tahminator/codebloom/wiki/CI-Commands

@tahminator
tahminator force-pushed the add-leaderboard-id-field-to-achievement-table branch from 63881b0 to 5956d5a Compare August 2, 2026 04:26
@tahminator
tahminator force-pushed the add-leaderboard-id-field-to-achievement-table branch from 5956d5a to f97db71 Compare August 2, 2026 04:42
@sonarqubecloud

sonarqubecloud Bot commented Aug 2, 2026

Copy link
Copy Markdown

@sonarqubecloud

sonarqubecloud Bot commented Aug 2, 2026

Copy link
Copy Markdown

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