Skip to content

dynamic bar#50

Open
VedVar43789 wants to merge 2 commits intomainfrom
bar_graph
Open

dynamic bar#50
VedVar43789 wants to merge 2 commits intomainfrom
bar_graph

Conversation

@VedVar43789
Copy link
Collaborator

Changes

What changes did you make? Include screenshots if applicable, or explain how to view the changes.

Made the bar on History page dynamic

Testing

How did you confirm your changes work? (Automated tests, manual verification, etc.)

  • TODO

Tracking

Add your issue number below.

Resolves #

@VedVar43789 VedVar43789 linked an issue Mar 6, 2025 that may be closed by this pull request
@shree-venkatesh shree-venkatesh self-requested a review March 6, 2025 23:22
Copy link
Collaborator

@shree-venkatesh shree-venkatesh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@shree-venkatesh shree-venkatesh self-requested a review March 6, 2025 23:23
Copy link
Collaborator

@shree-venkatesh shree-venkatesh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

look into why the lint tests are failing, and fix them

Copy link
Collaborator

@shree-venkatesh shree-venkatesh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

to make the bar truly dynamic, pull a list of transactions from the backend, make a sum of them, and use that as the current value

every time a new transaction is added, the bar should update.

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.

[FEATURE] Dynamic bar chart for the homepage

2 participants