Skip to content

Commit

Permalink
Merge pull request #44736 from Expensify/lucien/fix-qbo-taxes-lhn
Browse files Browse the repository at this point in the history
Disable taxes if we need to disconnect a previous integration first
  • Loading branch information
jasperhuangg authored Jul 3, 2024
2 parents fdb7410 + 3ef6702 commit 979466d
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions src/components/ConnectToQuickbooksOnlineButton/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,8 @@ function ConnectToQuickbooksOnlineButton({policyID, shouldDisconnectIntegrationB
{shouldDisconnectIntegrationBeforeConnecting && integrationToDisconnect && isDisconnectModalOpen && (
<AccountingConnectionConfirmationModal
onConfirm={() => {
// Since QBO doesn't support Taxes, we should disable them from the LHN when connecting to QBO
PolicyAction.enablePolicyTaxes(policyID, false);
removePolicyConnection(policyID, integrationToDisconnect);
Link.openLink(getQuickBooksOnlineSetupLink(policyID), environmentURL);
setIsDisconnectModalOpen(false);
Expand Down

0 comments on commit 979466d

Please sign in to comment.