Skip to content

fix: avoid cooldown on failed result calculation#140

Merged
adrunkhuman merged 2 commits intomasterfrom
fix/admin-results-cooldown-failure
Mar 22, 2026
Merged

fix: avoid cooldown on failed result calculation#140
adrunkhuman merged 2 commits intomasterfrom
fix/admin-results-cooldown-failure

Conversation

@adrunkhuman
Copy link
Owner

Summary

  • move the /admin results calculate cooldown write to the successful scoring path so validation failures do not throttle admins
  • add a regression test covering the failed-calculation retry case

Testing

  • uv run pytest tests/test_admin_commands.py -k "results_calculate"
  • uv run ruff check typer_bot/commands/admin_commands.py tests/test_admin_commands.py

@railway-app railway-app bot temporarily deployed to patient-quietude / matchday-typer-pr-140 March 22, 2026 01:23 Destroyed
@railway-app
Copy link

railway-app bot commented Mar 22, 2026

🚅 Deployed to the matchday-typer-pr-140 environment in patient-quietude

Service Status Web Updated (UTC)
matchday-typer ✅ Success (View Logs) Mar 22, 2026 at 1:24 am

@adrunkhuman adrunkhuman merged commit d1013c8 into master Mar 22, 2026
2 checks passed
@adrunkhuman adrunkhuman deleted the fix/admin-results-cooldown-failure branch March 22, 2026 01:43
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