Specify game points #9

Merged

BothimTV merged 3 commits from specify-game-points into main

2026-02-04 16:08:28 +00:00

BothimTV commented

2026-02-04 16:06:11 +00:00

(Migrated from github.com)

Summary by CodeRabbit

New Features
- Multiplier buttons now enable/disable dynamically per selected game.
- Games support configurable multiplier factors and update point calculations.
- Point display updated to show total and added points for a single selected student.
Tests / Chores
- Test-mode student fixture now includes a mail count field.
- CI workflow renamed for clarity.

## Summary by CodeRabbit * **New Features** * Multiplier buttons now enable/disable dynamically per selected game. * Games support configurable multiplier factors and update point calculations. * Point display updated to show total and added points for a single selected student. * **Tests / Chores** * Test-mode student fixture now includes a mail count field. * CI workflow renamed for clarity.

coderabbitai[bot] commented

2026-02-04 16:06:29 +00:00

(Migrated from github.com)

📝 Walkthrough

Walkthrough

Backend: in test mode the injected dummy student now includes a mailcount: 0 field. Frontend: the game page replaces static multiplier buttons with per-game factor-driven dynamic buttons, expands the games data model, and updates point calculation/display. CI workflow name changed.

Changes

Cohort / File(s)	Summary
Backend Test Data `backend/src/routes/mail/POST.ts`	When injecting the test student (`TEST-DUMMY-001`) add `mailcount: 0` to the test student object; no other control flow changed.
Frontend Game Multiplier Refactor `frontend/src/pages/spiel.vue`	Replace static multiplier buttons with dynamic disabled states based on per-game `factors`; expand `games` model to include `title`, `defaultPoints`, and `factors` mapping; update point calculation/display for single-student selection; minor import reorder.
CI Workflow Rename `.github/workflows/build.yml`	Rename workflow from "Deploy to Server" to "Build containers" without altering triggers or steps.
Manifest `package.json`	Included in manifest listing for this diff (no code edits inferred).

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 I hopped into code both bright and neat,
Buttons now dance with factors on their feet,
A test student counts zero mail with cheer,
Builds renamed, the rabbits clap — hip-hip, hooray! 🎉

Note

🎁 Summarized by CodeRabbit Free

Your organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login.

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

<details> <summary>📝 Walkthrough</summary> ## Walkthrough Backend: in test mode the injected dummy student now includes a `mailcount: 0` field. Frontend: the game page replaces static multiplier buttons with per-game factor-driven dynamic buttons, expands the games data model, and updates point calculation/display. CI workflow name changed. ## Changes |Cohort / File(s)|Summary| |---|---| |**Backend Test Data** `backend/src/routes/mail/POST.ts`|When injecting the test student (`TEST-DUMMY-001`) add `mailcount: 0` to the test student object; no other control flow changed.| |**Frontend Game Multiplier Refactor** `frontend/src/pages/spiel.vue`|Replace static multiplier buttons with dynamic disabled states based on per-game `factors`; expand `games` model to include `title`, `defaultPoints`, and `factors` mapping; update point calculation/display for single-student selection; minor import reorder.| |**CI Workflow Rename** `.github/workflows/build.yml`|Rename workflow from "Deploy to Server" to "Build containers" without altering triggers or steps.| |**Manifest** `package.json`|Included in manifest listing for this diff (no code edits inferred).| ## Estimated code review effort 🎯 3 (Moderate) | ⏱️ ~20 minutes ## Poem > 🐰 I hopped into code both bright and neat, > Buttons now dance with factors on their feet, > A test student counts zero mail with cheer, > Builds renamed, the rabbits clap — hip-hip, hooray! 🎉 </details>   --- > [!NOTE] > <details> > <summary>🎁 Summarized by CodeRabbit Free</summary> > > Your organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting <https://app.coderabbit.ai/login>. > > </details> Comment `@coderabbitai help` to get the list of available commands and usage tips.

Commenting is not possible because the repository is archived.

No reviewers