search

manuelalferez / flowinance

An ultra-fast financial management app with automatic transaction categorization.
https://flowinance.com
GNU General Public License v3.0
25 stars 33 forks source link

showing currency symbols before amount #89 #98

Closed Shailendra1703 closed 1 month ago

Shailendra1703 commented 1 month ago

Pull Request Template

What

This PR is related to issue #89, where I have worked on showing the currency symbols before the total amount

Here is the final version, tacked all the possibilities where the currency symbols are appearing from pie-chart to line-chart I covered them all

Before

image

After

image

How

Most of the changes are simply done via putting the currency ticker before the amount, but in balance section I have used the usememo hook to perform an optimized calculation between renders to avoid functioncalling each time.

Additional Notes

I faced a lot of errors while setting up the project but thanks to @manuelalferez readme update which clarifies each and everything.

vercel[bot] commented 1 month ago

@Shailendra1703 is attempting to deploy a commit to the manuelalferez's projects Team on Vercel.

A member of the Team first needs to authorize it.

vercel[bot] commented 1 month ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
flowinance ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 11, 2024 10:50am
manuelalferez commented 1 month ago

You nailed it! Everything is good to merge. Thanks, you make it easy to review it