wpp/weirsoe-party-protocol
8
0
Fork 0
Code Issues 19 Pull Requests Actions Packages Projects Releases Wiki Activity

F3: Beregn point efter guessfase og skift til reveal #10

Merged
email-manager merged 1 commits from feature/f3-score-calculate into main 2026-02-27 17:07:48 +01:00
Conversation 2 Commits 1 Files Changed 4
+220 -1
email-manager commented 2026-02-27 17:01:55 +01:00
Owner
Copy Link
Copy Source

Closes #9

Implementerer pointudregning i Fase 3:

  • Host endpoint til score-beregning
  • points_correct for korrekte gæt
  • points_bluff for spiller hvis løgn vælges
  • ScoreEvent audit entries
  • Transition til reveal-fase
  • Idempotens mod dobbeltkørsel

Validering lokalt:

  • ruff check lobby
  • python manage.py test lobby -v 1 (25/25 grøn)
Closes #9 Implementerer pointudregning i Fase 3: - Host endpoint til score-beregning - points_correct for korrekte gæt - points_bluff for spiller hvis løgn vælges - ScoreEvent audit entries - Transition til reveal-fase - Idempotens mod dobbeltkørsel Validering lokalt: - ruff check lobby - python manage.py test lobby -v 1 (25/25 grøn)
email-manager added 1 commit 2026-02-27 17:01:55 +01:00
feat(f3): calculate round scores and move to reveal phase
CI / test-and-quality (pull_request) Successful in 59s
Details
CI / test-and-quality (push) Successful in 57s
Details
1017ed0c4c
email-manager commented 2026-02-27 17:03:31 +01:00
Author
Owner
Copy Link
Copy Source

Integrator-check: PR er grøn på CI, men kan ikke merges endnu, fordi review-leddet i flowet mangler (krav: Udv -> branch -> PR -> review -> merge).

Blokering (need-to-have):

  • Mangler review/approval på PR #10

Næste skridt:

  1. Review-runner laver review på PR #10
  2. Ved godkendt review + fortsat grøn CI merger integrator straks til main med source-branch sletning.
Integrator-check: PR er grøn på CI, men **kan ikke merges endnu**, fordi review-leddet i flowet mangler (krav: `Udv -> branch -> PR -> review -> merge`). Blokering (need-to-have): - Mangler review/approval på PR #10 Næste skridt: 1) Review-runner laver review på PR #10 2) Ved godkendt review + fortsat grøn CI merger integrator straks til `main` med source-branch sletning.
email-manager added the need-to-have label 2026-02-27 17:03:38 +01:00
agw approved these changes 2026-02-27 17:04:00 +01:00
email-manager merged commit ae25403e18 into main 2026-02-27 17:07:48 +01:00
email-manager deleted branch feature/f3-score-calculate 2026-02-27 17:07:48 +01:00
Sign in to join this conversation.
Reviewers
No Reviewers
agw
Labels
Clear labels
PO vil have det her prioriteret
PO-prioriteret
 architect
Architect-owned coordination thread
 backend
Backend
 bot-task
Task managed by scheduler bots
 devops
DevOps/deploy related
 epic
Long-lived scope/phase tracking
 frontend
Frontend
 i18n
Internationalization
 mvp
auto by architect runner
 need-to-have
Required before merge/release
 needs-approval
Requires PO approval before execution
 nice-to-have
Improvement not blocking MVP
 release-approved
Architect approved release scope
 resolved
Løst og merged
 scope-guardrail
Scope and acceptance criteria source-of-truth
 smoke-fail
Smoke test failure
 spa
auto by architect runner
 staging
Staging infrastructure/deploy
 stale
Ingen aktivitet / forældet ift. main
 ui
Frontend/UI related
No labels need-to-have
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
architecture-bot dev-bot email-manager integrator-bot manager-bot reviewer-bot scheduler-bot tester-bot
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: wpp/weirsoe-party-protocol#10