Skip to content

Fix: Add new Pages to Crypto Dashboard#291

Open
manny-uncharted wants to merge 4 commits intorio-labs:mainfrom
manny-uncharted:feat-217-add-pages-to-crypto-dashboard
Open

Fix: Add new Pages to Crypto Dashboard#291
manny-uncharted wants to merge 4 commits intorio-labs:mainfrom
manny-uncharted:feat-217-add-pages-to-crypto-dashboard

Conversation

@manny-uncharted
Copy link

@manny-uncharted manny-uncharted commented Oct 1, 2025

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

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

Transaction Page Improvements

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

New Components

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

@manny-uncharted
Copy link
Author

@mad-moo

@iwr-redmond
Copy link

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

@mad-moo
Copy link
Contributor

mad-moo commented Oct 2, 2025

Thank you!

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

@manny-uncharted
Copy link
Author

@iwr-redmond I'd do this now

@iwr-redmond
Copy link

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

@iwr-redmond
Copy link

iwr-redmond commented Oct 13, 2025

From @Sn3llius in #217 (comment):

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

@iwr-redmond
Copy link

Is there any update on reviewing this PR?

@mad-moo
Copy link
Contributor

mad-moo commented Dec 31, 2025

Hey man, sorry for the delay.

I've reviewed this together with Sn3llius and we couldn't get it to run. The code tries to access values that don't exist. Some literal values don't exist in Rio either (e.g. "heading4", but Rio has no heading 4).

Does this run on your end? Was it maybe developed against a branch or older version?

@iwr-redmond
Copy link

Looking at the form builder code, you may wish to change heading4 to heading3.

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 new Pages to Crypto Dashboard

3 participants