Skip to content

fix(dashboard): hide wallet accounting summary from overview - #185

Merged
mirror29 merged 1 commit into
mainfrom
codex/hide-wallet-overview
Sep 30, 2026
Merged

mirror29 merged 1 commit into
mainfrom
codex/hide-wallet-overview

Conversation

@mirror29

Copy link
Copy Markdown
Owner

Remove the account/run-wallet accounting summary and legacy-ledger notice from the overview, as requested. The existing equity KPI, runner list, positions, orders, API payload and accounting behavior remain unchanged. This presentation change does not resume legacy strategy runs paused during the isolated-wallet rollout.

Validation: Dashboard typecheck and 109 tests passed; consistency checks passed. Independent read-only review confirmed only the overview presentation block and its unused translation hook were removed.

@github-actions

Copy link
Copy Markdown

Review base: cd0a182

🤖 DeepSeek V4 Pro PR Review

Reconstructed intent: this PR removes the "wallet accounting" summary section (main equity / book equity / per-run wallets / legacy-positions note) from the dashboard overview and drops the now-unused useTranslations("walletAccounting") binding.

Checks performed:

  • Dangling references: the only consumers of the accounting translator were the removed lines; the declaration is removed together with its usages, so no unresolved symbol remains.
  • Imports/hooks: useTranslations is still required for t / tStatus, so no now-unused import is introduced.
  • Data flow: data.account access was confined to the removed block; other sections consume data / data.evolutionSummary, and the KpiBar/EvolutionSummary props are untouched, so no behavioral coupling is broken.
  • Layout: removal happens before the live-runner/positions grid; the following grid comment/children are unaffected, so no structural regression is evident from the diff.

No correctness, contract, state-flow, or resource issues are supported by the evidence in this diff. LGTM.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying inalpha-web with  Cloudflare Pages  Cloudflare Pages

Latest commit: d6da879
Status: ✅  Deploy successful!
Preview URL: https://76450f17.inalpha-web.pages.dev
Branch Preview URL: https://codex-hide-wallet-overview.inalpha-web.pages.dev

View logs

@mirror29
mirror29 merged commit 622d1b3 into main Sep 30, 2026
15 checks passed
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