moex/moex-vibe
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

codex/broker-accounts-overview #22

Merged
ksv741 merged 3 commits from codex/broker-accounts-overview into main 2026-06-19 14:48:37 +03:00
Conversation 0 Commits 3 Files Changed 18 +2311 -130
ksv741 commented 2026-06-19 14:48:27 +03:00
Owner
Copy Link

Summary

  • turn /broker from a plain account list into a broker accounts overview
  • add aggregated summary by available account portfolios without implicit currency conversion
  • add richer account cards with portfolio value, daily result, expected yield, allocation, loading and retry states

What Changed

  • added frontend-only broker accounts aggregation with currency-safe totals and daily percent calculation
  • added parallel portfolio loading for account list with shared TanStack Query keys
  • redesigned /broker page with:
    • top summary section
    • informative account cards
    • allocation bars
    • skeleton, empty state, partial-error state, local retry
  • removed technical account id and raw T-Bank enum values from account cards
  • updated feature tasks and roadmap status
  • documented the frontend aggregation decision in ADR-012 earlier in this branch

Verification

  • npm test -w apps/frontend
  • npm run lint -w apps/frontend
  • npm run build -w apps/frontend
  • npm run build -w apps/docs

Notes

  • aggregation stays on the frontend by design
  • values from different currencies are shown separately
  • one failed account does not block the rest of the page
## Summary - turn `/broker` from a plain account list into a broker accounts overview - add aggregated summary by available account portfolios without implicit currency conversion - add richer account cards with portfolio value, daily result, expected yield, allocation, loading and retry states ## What Changed - added frontend-only broker accounts aggregation with currency-safe totals and daily percent calculation - added parallel portfolio loading for account list with shared TanStack Query keys - redesigned `/broker` page with: - top summary section - informative account cards - allocation bars - skeleton, empty state, partial-error state, local retry - removed technical account id and raw T-Bank enum values from account cards - updated feature tasks and roadmap status - documented the frontend aggregation decision in ADR-012 earlier in this branch ## Verification - `npm test -w apps/frontend` - `npm run lint -w apps/frontend` - `npm run build -w apps/frontend` - `npm run build -w apps/docs` ## Notes - aggregation stays on the frontend by design - values from different currencies are shown separately - one failed account does not block the rest of the page
ksv741 added 3 commits 2026-06-19 14:48:28 +03:00
feat: add broker accounts overview
All checks were successful
CI / ci (pull_request) Successful in 3m21s
Details
CI / ci (push) Successful in 3m6s
Details
1e007de5be
ksv741 merged commit 1e007de5be into main 2026-06-19 14:48:37 +03:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
ksv741
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: moex/moex-vibe#22
No description provided.
Delete Branch "codex/broker-accounts-overview"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?