Skip to content
This repository was archived by the owner on Jul 27, 2025. It is now read-only.

Show Uncategorized segment currency as per budget currency#2058

Merged
zachgoll merged 2 commits intomaybe-finance:mainfrom
the-spectator:fix_budget
Apr 8, 2025
Merged

Show Uncategorized segment currency as per budget currency#2058
zachgoll merged 2 commits intomaybe-finance:mainfrom
the-spectator:fix_budget

Conversation

@the-spectator
Copy link
Contributor

Updated the Uncategorized budget segment UI to reflect the actual currency symbol from budget instead of hardcoding $.

Before:
image

After:
image

the-spectator and others added 2 commits April 6, 2025 23:43
Previously, the symbol for the 'Uncategorized' segment defaulted to `$`, which is incorrect for non-USD budgets. This change ensures the correct currency symbol is shown based on the budget's currency.
Copy link
Contributor

@zachgoll zachgoll left a comment

Choose a reason for hiding this comment

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

Looks good!

@zachgoll zachgoll merged commit 2bc3887 into maybe-finance:main Apr 8, 2025
5 checks passed
@the-spectator the-spectator deleted the fix_budget branch April 8, 2025 17:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants