Skip to content

feat: add TreasuryFlowChart inflow/outflow analytics component - #731

Merged
thegreatfeez merged 1 commit into
Ac0rdP:mainfrom
vic2430:feat/treasury-flow-chart
Sep 28, 2026
Merged

thegreatfeez merged 1 commit into
Ac0rdP:mainfrom
vic2430:feat/treasury-flow-chart

Conversation

@vic2430

@vic2430 vic2430 commented Sep 28, 2026

Copy link
Copy Markdown
  • Add TreasuryFlowChart component with grouped bar chart (inflow vs outflow per window) using recharts BarChart
  • Support day/week/month granularity selection with controlled and uncontrolled modes
  • Merge per-token API buckets into single display points via bucketsToBars() helper
  • Handle loading, empty, and error states via AnalyticsSectionState
  • Add role=img aria-label and sr-only summary for accessibility
  • Add toTreasuryFlowChart() helper in AnalyticsPage for CSV export path
  • Wire TreasuryFlowChart into AnalyticsPage replacing the broken inline LineChart treasury section
  • Fix pre-existing syntax error in useRoles.ts (missing reduce closure)
  • Rewrite AnalyticsPage.test.tsx removing dead tests referencing undefined symbols; 8 tests pass
  • Add TreasuryFlowChart.test.tsx with 11 tests covering all states, granularity controls, token merging, and accessibility
  • Vite build passes; 45/45 analytics tests green

Summary

Type of Change

  • Bug fix
  • New feature
  • Documentation
  • Refactor

Related Issue

Closes #

Testing Checklist

  • Existing tests pass locally
  • New tests added to cover this change (where applicable)
  • Manually verified the change works as expected
  • Lint / type checks pass
    Closes Add an inflow/outflow chart #665

- Add TreasuryFlowChart component with grouped bar chart (inflow vs
  outflow per window) using recharts BarChart
- Support day/week/month granularity selection with controlled and
  uncontrolled modes
- Merge per-token API buckets into single display points via
  bucketsToBars() helper
- Handle loading, empty, and error states via AnalyticsSectionState
- Add role=img aria-label and sr-only summary for accessibility
- Add toTreasuryFlowChart() helper in AnalyticsPage for CSV export path
- Wire TreasuryFlowChart into AnalyticsPage replacing the broken inline
  LineChart treasury section
- Fix pre-existing syntax error in useRoles.ts (missing reduce closure)
- Rewrite AnalyticsPage.test.tsx removing dead tests referencing
  undefined symbols; 8 tests pass
- Add TreasuryFlowChart.test.tsx with 11 tests covering all states,
  granularity controls, token merging, and accessibility
- Vite build passes; 45/45 analytics tests green
@vercel

vercel Bot commented Sep 28, 2026

Copy link
Copy Markdown

@vic2430 is attempting to deploy a commit to the thegreatfeez's projects Team on Vercel.

A member of the Team first needs to authorize it.

@drips-wave

drips-wave Bot commented Sep 28, 2026

Copy link
Copy Markdown

@vic2430 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@thegreatfeez
thegreatfeez merged commit 0c16e06 into Ac0rdP:main Sep 28, 2026
1 check failed
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.

Add an inflow/outflow chart

2 participants