Extend PR #3089 – Complete Fiat Input Support in Dual Currency Field#3394

Open

Dunsin-cyber wants to merge 21 commits into

getAlby:masterfrom

Dunsin-cyber:refine-dual-currency-3089

Commits on Jul 1, 2025

feat: dual currency input supports fiat input

riccardobl
authored and
Dunsin-cyber
committed
feat: prefix dual currency input with main symbol, add currency to input placeholder

riccardobl
authored and
Dunsin-cyber
committed
fix: amount in Satoshis -> amount in sats

riccardobl
authored and
Dunsin-cyber
committed
fix: attempt at making unit tests happy

riccardobl
authored and
Dunsin-cyber
committed
fix: fix some unit tests

riccardobl
authored and
Dunsin-cyber
committed
fix: dual currency input event wrapping

riccardobl
authored and
Dunsin-cyber
committed
fix: more unit tests fixing

riccardobl
authored and
Dunsin-cyber
committed
fix: currency toggle issue for default values, improve some naming

riccardobl
authored and
Dunsin-cyber
committed
fix: input value

riccardobl
authored and
Dunsin-cyber
committed
fix: naming, empty input, do not change component to uncontrolled when value is empty

riccardobl
authored and
Dunsin-cyber
committed
fix: fixes and improvements

riccardobl
authored and
Dunsin-cyber
committed
fix: light theme

riccardobl
authored and
Dunsin-cyber
committed
fix: dualcurrency component test

pavanjoshi914
authored and
Dunsin-cyber
committed
fix: clone target to avoid side-effects when changing value

riccardobl
authored and
Dunsin-cyber
committed
fix: tests and implementation

riccardobl
authored and
Dunsin-cyber
committed
fix: allow 4 decimals in fiat input and add some comments

riccardobl
authored and
Dunsin-cyber
committed
fix: preset workaround

riccardobl
authored and
Dunsin-cyber
committed
test(ConfirmPayment): mock getFormattedInCurrency to fix tests
Dunsin-cyber
committed
fix: prevent redundant re-renders when value prop matches current input
Dunsin-cyber
committed
test(SitePreferences): revamp 'set new budget' test with formatting checks and save flow
Dunsin-cyber
committed
fix(SendToBitcoinAddress): fix linting issue in the page
Dunsin-cyber
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Extend PR #3089 – Complete Fiat Input Support in Dual Currency Field#3394

Extend PR #3089 – Complete Fiat Input Support in Dual Currency Field#3394
Dunsin-cyber wants to merge 21 commits into
getAlby:masterfrom
Dunsin-cyber:refine-dual-currency-3089

Commits on Jul 1, 2025

feat: dual currency input supports fiat input

feat: prefix dual currency input with main symbol, add currency to input placeholder

fix: amount in Satoshis -> amount in sats

fix: attempt at making unit tests happy

fix: fix some unit tests

fix: dual currency input event wrapping

fix: more unit tests fixing

fix: currency toggle issue for default values, improve some naming

fix: input value

fix: naming, empty input, do not change component to uncontrolled when value is empty

fix: fixes and improvements

fix: light theme

fix: dualcurrency component test

fix: clone target to avoid side-effects when changing value

fix: tests and implementation

fix: allow 4 decimals in fiat input and add some comments

fix: preset workaround

test(ConfirmPayment): mock getFormattedInCurrency to fix tests

fix: prevent redundant re-renders when value prop matches current input

test(SitePreferences): revamp 'set new budget' test with formatting checks and save flow

fix(SendToBitcoinAddress): fix linting issue in the page