Revert "Fix tax updating for changing waypoints or currency"

[blimpich]

Reverts #73045

[melvin-bot]

@srikarparsi Please copy/paste the Reviewer Checklist from here into a new comment on this PR and complete it. If you have the K2 extension, you can simply click: [this button]

[github-actions]

❌ PERF-6 (docs)

In useEffect, useMemo, and useCallback, specify individual object properties as dependencies instead of passing entire objects.

Replace the transaction dependency with its specific properties that are actually used:

}, [customUnitRateID, policy, previousCustomUnitRateID, previousTransactionCurrency, previousTransactionModifiedCurrency, shouldShowTax, transaction?.taxCode, transactionID]);

[github-actions]

❌ PERF-6 (docs)

In useEffect, useMemo, and useCallback, specify individual object properties as dependencies instead of passing entire objects.

Replace the transaction dependency with its specific properties that are actually used in this effect:

    }, [
        policy,
        shouldShowTax,
        previousTransactionAmount,
        previousTransactionCurrency,
        transaction?.amount,
        transaction?.currency,
        transaction?.taxCode,
        transaction?.taxAmount,
        transaction?.transactionID,
        isDistanceRequest,
        customUnitRateID,
        previousCustomUnitRateID,
        previousTaxCode,
        distance,
    ]);

[codecov]

Codecov Report

❌ Patch coverage is 39.13043% with 14 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
src/components/MoneyRequestConfirmationList.tsx	39.13%	14 Missing ⚠️

Files with missing lines	Coverage Δ
src/components/MoneyRequestConfirmationList.tsx	50.83% <39.13%> (-0.28%)	⬇️

... and 11 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.