Paradaux

PAR-208

0

Show a firm's proprietor on the firm page (economy explorer)

Pending ReleaseUnassignedEconomy ExplorerFeature

Goal

Surface a firm's proprietor (owner) on the firm drilldown in the economy explorer. Today the firm page lists employees, roles and accounts but never shows who actually owns the firm — even though we already have the data.

State (grounded in the repo)

  • lib/sql/firm.tsfindFirmByDisplayName already selects proprietor_uuid_bin AS proprietorUuid and FirmRow already carries proprietor_uuid: string | null — so the proprietor UUID is fetched but unused.
  • The proprietor's display name is not resolved. The name cache is firm_players.current_name (the same source listFirmEmployees joins via LEFT JOIN firm_players fp ON fp.player_uuid_bin = …).
  • The firm detail page is app/firms/[firmName]/page.tsx.

Change

  1. In lib/sql/firm.ts, resolve the proprietor's name alongside the firm row — LEFT JOIN firm_players fp ON fp.player_uuid_bin = f.proprietor_uuid_bin (mirroring the employee join) and expose proprietor_name on FirmRow (null when the UUID isn't cached → fall back to the UUID/short form).
  2. On app/firms/[firmName]/page.tsx, render the proprietor prominently (e.g. in the firm header / summary), linking to the player drilldown (/players/... or the existing player view) where one exists.
  3. Optionally flag the proprietor within the employee list (the role model has is_proprietor_like) so it's obvious who the owner is there too.

Privacy

Firm ownership is public information (visible in-game via /business), and the proprietor is an identity, not financial data — so this is not behind the per-entity money drilldown login gate. No balances are exposed by this change. Keep it SSR like the rest of the explorer.

Acceptance

  • The firm page shows the proprietor's name (linked to their player page when available); UUID/fallback shown gracefully when the name isn't in firm_players.
  • Firms with a null proprietor render without error.
  • Read-only DAL change in lib/sql/firm.ts (no business logic leaks into the query layer).

Resources

Comments

No comments yet.

Activity

  • ParadauxIO linked a pull request — PR #2 open — Last major release
  • ParadauxIO linked a pull request — PR #2 open — Last major release
  • ParadauxIO linked a pull request — PR #2 open — Last major release
  • ParadauxIO linked a pull request — PR #2 open — Last major release
  • ParadauxIO linked a pull request — PR #2 open — Last major release
  • ParadauxIO linked a pull request — PR #2 open — Last major release
  • ParadauxIO linked a pull request — PR #2 open — Last major release
  • tesks changed status to Status → Pending Release
  • ParadauxIO linked a commit — Commit de3fd14 — Show a firm's proprietor on the explorer firm page (PAR-208)
  • tesks changed status to Status → In Progress
  • tesks created the issue