Skip to content

fix(wallets): render Ibex transaction history in the account display currency (#284) - #520

Closed
nul-quill wants to merge 1 commit into
lnflash:mainfrom
nul-quill:jmd-display-price-284
Closed

nul-quill wants to merge 1 commit into
lnflash:mainfrom
nul-quill:jmd-display-price-284

Conversation

@nul-quill

Copy link
Copy Markdown

Threads the account displayCurrency into getTransactionsForWallets so the Ibex transaction history is labelled and converted in the configured currency instead of hardcoded USD; builds settlementDisplayPrice with the display currency major exponent so base/offset match the settlementPrice derivation; converts each row with its own exchangeRateCurrencySats and rounds instead of flooring so sub-cent BTC receipts no longer collapse to 0. CSV export path keeps the arg optional. Covers all three points in #284. Tests: jest --config ./test/flash/unit/jest.config.js -> 46 suites / 512 tests passed.

…currency

- thread the account displayCurrency into getTransactionsForWallets so the
  settlement display label/conversion is no longer hardcoded to USD
- build settlementDisplayPrice with the display currency's major exponent so
  base/offset stay consistent with the settlementPrice derivation
- convert each row with its own exchangeRateCurrencySats and round instead of
  floor, so sub-cent BTC receipts no longer collapse to 0
- resolvers (btc/usd/usdt wallet) and getTransactionsForAccount pass the
  currency from the GraphQL/account context; CSV export keeps the optional arg

Refs lnflash#284
@nul-quill

Copy link
Copy Markdown
Author

Handle: nul-quill. Payout wallets: Solana USDC BdznDjc4xwt5Y6WJT5UnrFQzDptka8YnC9oqmmkKpwkd, TRC20 TNEHCVwR2aPMrj5AKfWCQ2rFL3bEDQmsg7.

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.

2 participants