rio icon indicating copy to clipboard operation
rio copied to clipboard

Fix: Add new Pages to Crypto Dashboard

Open manny-uncharted opened this issue 2 months ago • 7 comments

Closes #217

Summary

This PR implements the missing Statistics and Transaction pages for the Crypto Dashboard template, addressing the core functionality gaps identified in issue #217.

The Statistics page now provides comprehensive portfolio returns analysis with base 100 normalization, allowing users to compare past returns of all coins in their wallet with flexible date range selection. The Transaction page has been completely redesigned with enhanced management capabilities, search/filter functionality, and transaction analytics.

Changes Made

Statistics Page Enhancements

  • [x] Implemented past returns visualization with base 100 normalization
  • [x] Added flexible date range selection with preset options (1M, 3M, 6M, 1Y, 2Y, All, Custom)
  • [x] Created interactive asset selection for comparing individual coins or portfolio performance
  • [x] Added performance summary cards showing portfolio metrics and best performing assets
  • [x] Integrated advanced portfolio analytics with allocation breakdown, performance metrics, and risk analysis

Transaction Page Improvements

  • [x] Fixed duplicate class definitions in original implementation
  • [x] Added comprehensive transaction management with add, edit, and delete capabilities
  • [x] Implemented search and filter functionality by type, asset, date, and text search
  • [x] Created transaction analytics dashboard with summary statistics
  • [x] Enhanced add transaction dialog with validation and real-time calculations
  • [x] Added sorting capabilities for all table columns
  • [x] Implemented export functionality framework

New Components

  • [x] Created PortfolioAnalytics component with comprehensive financial metrics
  • [x] Added AssetComparisonChart component for normalized returns comparison
  • [x] Implemented risk metrics including Sharpe ratio, max drawdown, and Value at Risk
  • [x] Added asset correlation analysis for portfolio diversification insights

manny-uncharted avatar Oct 01 '25 14:10 manny-uncharted

@mad-moo

manny-uncharted avatar Oct 01 '25 14:10 manny-uncharted

You may need to add faker to the list of dependencies in meta.json.

iwr-redmond avatar Oct 01 '25 19:10 iwr-redmond

Thank you!

@Sn3llius can you have a look at this? Is this what you had in mind?

mad-moo avatar Oct 02 '25 12:10 mad-moo

@iwr-redmond I'd do this now

manny-uncharted avatar Oct 02 '25 12:10 manny-uncharted

If @Sn3llius is unavailable this PR should be reviewed by the core contributors instead.

iwr-redmond avatar Oct 09 '25 22:10 iwr-redmond

From @Sn3llius in #217 (comment):

Thanks, and sorry for the late reply. I’ll check it out tomorrow.

iwr-redmond avatar Oct 13 '25 05:10 iwr-redmond

Is there any update on reviewing this PR?

iwr-redmond avatar Nov 17 '25 03:11 iwr-redmond