Skip to content

Commit

Permalink
fix(transfer): fix txFee not updated
Browse files Browse the repository at this point in the history
  • Loading branch information
pwltr committed Feb 28, 2024
1 parent 18caee7 commit e9a923c
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
4 changes: 2 additions & 2 deletions src/screens/Lightning/CustomConfirm.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -53,9 +53,9 @@ const CustomConfirm = ({
const fiatTransactionFee = useDisplayValues(transactionFee);
const clientBalance = useDisplayValues(order?.clientBalanceSat ?? 0);

// avoid flashing different price after confirmation
// TODO: avoid flashing different price & allocation after confirmation
// eslint-disable-next-line react-hooks/exhaustive-deps

Check warning on line 57 in src/screens/Lightning/CustomConfirm.tsx

View workflow job for this annotation

GitHub Actions / Run lint check

'react-hooks/exhaustive-deps' rule is disabled but never reported
const txFee = useMemo(() => fiatTransactionFee.fiatValue, [orderId]);
const txFee = fiatTransactionFee.fiatValue;
const lspFee = purchaseFeeValue.fiatValue - clientBalance.fiatValue;

const handleConfirm = async (): Promise<void> => {
Expand Down
4 changes: 2 additions & 2 deletions src/screens/Lightning/QuickConfirm.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -48,9 +48,9 @@ const QuickConfirm = ({
const fiatTransactionFee = useDisplayValues(transactionFee);
const clientBalance = useDisplayValues(order?.clientBalanceSat ?? 0);

// avoid flashing different price after confirmation
// TODO: avoid flashing different price & allocation after confirmation
// eslint-disable-next-line react-hooks/exhaustive-deps

Check warning on line 52 in src/screens/Lightning/QuickConfirm.tsx

View workflow job for this annotation

GitHub Actions / Run lint check

'react-hooks/exhaustive-deps' rule is disabled but never reported
const txFee = useMemo(() => fiatTransactionFee.fiatValue, [orderId]);
const txFee = fiatTransactionFee.fiatValue;
const lspFee = purchaseFeeValue.fiatValue - clientBalance.fiatValue;

const savingsAmount = onchainBalance - spendingAmount;
Expand Down
4 changes: 2 additions & 2 deletions src/screens/Transfer/Confirm.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -56,9 +56,9 @@ const Confirm = ({
const fiatTransactionFee = useDisplayValues(transactionFee);
const clientBalance = useDisplayValues(order?.clientBalanceSat ?? 0);

// avoid flashing different price after confirmation
// TODO: avoid flashing different price & allocation after confirmation
// eslint-disable-next-line react-hooks/exhaustive-deps

Check warning on line 60 in src/screens/Transfer/Confirm.tsx

View workflow job for this annotation

GitHub Actions / Run lint check

'react-hooks/exhaustive-deps' rule is disabled but never reported
const txFee = useMemo(() => fiatTransactionFee.fiatValue, [orderId]);
const txFee = fiatTransactionFee.fiatValue;
const lspFee = purchaseFeeValue.fiatValue - clientBalance.fiatValue;

const handleConfirm = async (): Promise<void> => {
Expand Down

0 comments on commit e9a923c

Please sign in to comment.