Skip to content

feat: implement goal-based savings tracking and dashboard summary#922

Open
Sp1kyss wants to merge 2 commits intorohitdash08:mainfrom
Sp1kyss:feat/savings-goals
Open

feat: implement goal-based savings tracking and dashboard summary#922
Sp1kyss wants to merge 2 commits intorohitdash08:mainfrom
Sp1kyss:feat/savings-goals

Conversation

@Sp1kyss
Copy link
Copy Markdown

@Sp1kyss Sp1kyss commented Apr 21, 2026

Overview:
I've implemented a complete Savings Goal tracking system that integrates seamlessly with the existing FinMind architecture.

Key Changes:

Savings Page: New dedicated route at /savings featuring progress bars and goal management.

Dashboard Widget: Added a "Savings Progress" metric card to the main dashboard for at-a-glance tracking.

Responsive Navigation: Integrated the "Savings" link with the Lucide Target icon into the global Navbar.

Design System Compliance: Used native .chart-fill and .page-wrap classes to ensure UI consistency.

Proof of Work:
I've verified the UI on local build v3001.
finmind_dashboard_final_1776814799633

Closes #133. /attempt #133

@Sp1kyss Sp1kyss requested a review from rohitdash08 as a code owner April 21, 2026 23:44
@Sp1kyss
Copy link
Copy Markdown
Author

Sp1kyss commented Apr 22, 2026

Hi @rohitdash08, I've implemented the goal-based savings tracking and dashboard summary as requested. I've verified the logic locally. Please let me know if you can approve the GitHub Actions workflows so the tests can run, or if you'd like any adjustments to the UI!

@Sp1kyss
Copy link
Copy Markdown
Author

Sp1kyss commented Apr 22, 2026

@rohitdash08, just a quick follow-up—I've verified the dashboard widget layout locally. Ready to address any feedback once the workflows are approved!

@rohitdash08
Copy link
Copy Markdown
Owner

@Sp1kyss approved the workflow... our team will review your submission can you please connect over discord @geekster007

@rohitdash08
Copy link
Copy Markdown
Owner

@Sp1kyss CI failed

@Sp1kyss
Copy link
Copy Markdown
Author

Sp1kyss commented Apr 24, 2026

@Sp1kyss CI failed

Pushed a fix to address the frontend CI failure. The TrendingUp import was unused, which triggered the linting error.

@rohitdash08 The PR is now ready for the workflows to be re-run and for final review. I've also reached via Discord as requested. Thanks!

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.

Goal-based savings tracking & milestones

2 participants