Skip to content

Fix account batch action routes and hover help#9

Open
MisonL wants to merge 3 commits intoDestinyCycloid:mainfrom
MisonL:feature/account-batch-action-tooltips
Open

Fix account batch action routes and hover help#9
MisonL wants to merge 3 commits intoDestinyCycloid:mainfrom
MisonL:feature/account-batch-action-tooltips

Conversation

@MisonL
Copy link
Copy Markdown

@MisonL MisonL commented Apr 2, 2026

Summary

  • fix the three broken async batch action routes on the accounts page
  • add submission failure handling so registered async tasks do not get stuck in pending when executor submission fails
  • keep the three button labels stable in idle state and replace native title hints with hover bubbles shown below the buttons
  • add a review record for the branch verification

Verification

  • python3 -m py_compile src/web/routes/accounts.py src/web/routes/payment.py
  • isolated runtime on http://127.0.0.1:16667 with password admin123
  • login and accounts page returned 200
  • async task creation and polling both returned 200 for:
    • /api/accounts/batch-refresh/async
    • /api/accounts/batch-validate/async
    • /api/payment/accounts/batch-check-subscription/async
    • /api/accounts/tasks/{task_id}
    • /api/payment/ops/tasks/{task_id}
  • review record: docs/reviews/CR-ACCOUNT-BATCH-ACTIONS-2026-04-02.md

Review Note

  • coderabbit review --prompt-only --base upstream/main -t committed did not return a new finding after the hardening patch, but the request timed out during remote review processing.

@MisonL MisonL force-pushed the feature/account-batch-action-tooltips branch from 301bab5 to 310476f Compare April 2, 2026 15:14
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