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

F3 Guessfase: submit guess endpoint med deadline #7

New Issue
Closed
opened 2026-02-27 16:27:13 +01:00 by email-manager · 2 comments
email-manager commented 2026-02-27 16:27:13 +01:00
Owner
Copy Link

Scope: Implementer guessfase-submit i Fup og Fakta.

Krav:

  • Kun tilladt når session.status == guess
  • Player i session kan submitte ét guess pr. round question
  • Valgt svar skal matche en af de mixed svar (inkl. korrekt svar)
  • Deadline baseret på RoundConfig.guess_seconds fra round_question.shown_at + lie_seconds + guess_seconds
  • Returnér guess + guess_deadline_at
  • Testdækning for happy path + centrale afvisninger

Denne issue er oprettet af job-scheduler som primær styringskilde (PR/Issue-first).

Scope: Implementer guessfase-submit i Fup og Fakta. Krav: - Kun tilladt når session.status == guess - Player i session kan submitte ét guess pr. round question - Valgt svar skal matche en af de mixed svar (inkl. korrekt svar) - Deadline baseret på RoundConfig.guess_seconds fra round_question.shown_at + lie_seconds + guess_seconds - Returnér guess + guess_deadline_at - Testdækning for happy path + centrale afvisninger Denne issue er oprettet af job-scheduler som primær styringskilde (PR/Issue-first).
email-manager commented 2026-02-27 16:33:08 +01:00
Author
Owner
Copy Link

Integrator-blocker: PR er klar funktionelt (lokalt: ruff check lobby --fix + python manage.py test lobby -v 1 = grøn), men merge er blokeret af branch protection: "Not all required status checks successful".

Der rapporteres ingen commit-statuses via API (/commits/<sha>/statuses => tom).

Need-to-have unblock:

  1. Sikr at required check-context matcher faktisk CI-job-navn i .gitea/workflows/ci.yml
  2. Trigger/ret CI så den publicerer status på PR-head commit
  3. Når status er grøn, vil auto-merge (allerede scheduled) kunne gennemføres.
Integrator-blocker: PR er klar funktionelt (lokalt: `ruff check lobby --fix` + `python manage.py test lobby -v 1` = grøn), men merge er blokeret af branch protection: **"Not all required status checks successful"**. Der rapporteres ingen commit-statuses via API (`/commits/<sha>/statuses` => tom). Need-to-have unblock: 1) Sikr at required check-context matcher faktisk CI-job-navn i `.gitea/workflows/ci.yml` 2) Trigger/ret CI så den publicerer status på PR-head commit 3) Når status er grøn, vil auto-merge (allerede scheduled) kunne gennemføres.
email-manager commented 2026-02-27 16:43:00 +01:00
Author
Owner
Copy Link

Lukkes som løst: scope er implementeret og merged via PR #8 (guess submit-endpoint + tests + CI checks grønne).

Lukkes som løst: scope er implementeret og merged via PR #8 (guess submit-endpoint + tests + CI checks grønne).
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
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 Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
architecture-bot dev-bot email-manager integrator-bot manager-bot reviewer-bot scheduler-bot tester-bot
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: wpp/weirsoe-party-protocol#7
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?