hot-fix(currency): adjust small decimal formatting logic#207
hot-fix(currency): adjust small decimal formatting logic#207abhinavdroid wants to merge 1 commit into
Conversation
https://app.asana.com/1/34125054317482/project/1200301693863971/task/1214652452633418?focus=true Signed-off-by: Abhinav Gupta <a.gupta@flick2know.com>
bef6b57 to
4ad8dd0
Compare
🤖 AI PR ReviewTL;DR: 🔴 Block merge — no test coverage and no documentation. 📋 Summary
Automated Quality Checks🔴 Critical: No Test Coverage — 1 source file(s) changed/added but ZERO test files changed/added. Tests are mandatory for all code changes. 🔴 Critical: No Documentation — 1 source file(s) changed/added but no documentation updated. Update README, inline docs, or API docs for any code change. 🟡 Medium: No Reviewers Assigned — PR has zero reviewers. Add at least one peer reviewer before merging. Deep Code Analysis
Changes
Reverse Dependencies (other repos calling this code)
Generated by FANexus PR Reviewer |
Description
Moved dynamic decimal precision handling into
CurrencyUtil.formatNumber()to centralize formatting behavior and remove duplicate UI-level logic.Examples:
6->66.2->6.26.26->6.260.3->0.31250->1.2KLinks :
Asana: https://app.asana.com/1/34125054317482/project/1200301693863971/task/1214652452633418?focus=true
Commits
Tests
I added the following tests:
No Test(s) added.
Checklist
Before you create this PR, confirm that it meets all requirements listed below by checking the relevant checkboxes (
[x]). This will ensure a smooth and quick review process.///).Breaking Change
Should be merge type: