Paradaux
IssuesPAR-93Done
0

Disbanding a Firm Confirmation Action

Make disbanding a firm a confirmable action like transfer proprietorship, making it clear that firm funds are given to the proprietor.

Comments

tesks · Jun 5, 2026, 11:23 AM

Shipped on develop (commit 871a127), together with PAR-86.

/firm disband <firm> is now a two-step confirmable action (in the spirit of the proprietorship-transfer flow, but lighter — no DB-backed request):

  1. Warn — validates firm/proprietorship, then shows: the firm's balance returns to your personal account and the name cannot be reused, with a clickable /firm disband <firm> confirm to run within 60s.
  2. Confirm — re-validates firm existence, archived state, and proprietorship, consumes the pending confirmation, then disbands.

Backed by FirmDisbandConfirmationService (in-memory, clock-injectable): one pending confirmation per player, single-use, matched to the exact firm so you can't warn on one firm and confirm-disband another, 60s TTL. Tested (within-TTL, no-request, wrong-firm, expiry, single-use); coverage gate green.

Activity

  • ParadauxIO linked a commit — Commit 871a127 — Require confirmation before disbanding a firm (PAR-93, PAR-86)Jun 7, 2026, 2:49 PM
  • ParadauxIO changed status to Status → DoneJun 7, 2026, 2:49 PM
  • ParadauxIO linked a pull request — PR #6 merged — Release: develop → mainJun 7, 2026, 2:49 PM
  • ParadauxIO linked a pull request — PR #6 open — Release: develop → mainJun 7, 2026, 12:36 AM
  • ParadauxIO linked a pull request — PR #6 open — Release: develop → mainJun 6, 2026, 11:49 PM
  • ParadauxIO linked a pull request — PR #6 open — Release: develop → mainJun 6, 2026, 12:44 PM
  • ParadauxIO linked a pull request — PR #6 open — Release: develop → mainJun 6, 2026, 11:17 AM
  • ParadauxIO linked a pull request — PR #6 open — Release: develop → mainJun 6, 2026, 11:11 AM
  • ParadauxIO linked a pull request — PR #6 open — Release: develop → mainJun 5, 2026, 12:35 PM
  • tesks commentedJun 5, 2026, 11:23 AM
  • tesks changed status to Status → Pending ReleaseJun 5, 2026, 11:23 AM
  • ParadauxIO linked a commit — Commit 871a127 — Require confirmation before disbanding a firm (PAR-93, PAR-86)Jun 5, 2026, 11:23 AM
  • tesks changed status to Status → In ProgressJun 5, 2026, 11:20 AM
  • paradaux set priority to Priority → HighJun 5, 2026, 7:02 AM
  • paradaux updated labels (Labels updated)Jun 5, 2026, 7:02 AM
  • paradaux changed status to Status → PlannedJun 5, 2026, 7:02 AM
  • tesks created the issueJun 5, 2026, 7:00 AM