OUT-2052: Amount-related issues

src/app/api/quickbooks/invoice/invoice.service.ts

@@ -451,6 +451,7 @@ export class InvoiceService extends BaseService {
     await this.createQBInvoice(invoicePayload)
 
     // update/ create the record in sync log table
+    const totalWithTax = actualTotalAmount + totalTax
     await this.logSync(
       invoiceResource.id,
       {
@@ -459,7 +460,7 @@ export class InvoiceService extends BaseService {
       },
       EventType.CREATED,
       {
-        amount: (actualTotalAmount * 100).toFixed(2), // convert to cents for logs
+        amount: (totalWithTax * 100).toFixed(2),
         taxAmount: (totalTax * 100).toFixed(2), // convert to cents for logs
         customerName: recipientInfo.displayName,
         customerEmail: recipientInfo.email,
@@ -473,13 +474,13 @@ export class InvoiceService extends BaseService {
     if (invoiceResource.status === InvoiceStatus.PAID) {
       const paymentService = new PaymentService(this.user)
       const qbPaymentPayload = {
-        TotalAmt: actualTotalAmount,
+        TotalAmt: totalWithTax,
         CustomerRef: {
           value: customerRefValue,
         },
         Line: [
           {
-            Amount: actualTotalAmount,
+            Amount: totalWithTax,
             LinkedTxn: [
               {
                 TxnId: invoiceRes.Invoice.Id,
@@ -638,15 +639,11 @@ export class InvoiceService extends BaseService {
       )
     }
 
-    const syncLog = await this.syncLogService.getOneByCopilotIdAndEventType(
-      payload.data.id,
-      EventType.VOIDED,
-    )
-
-    if (syncLog?.status === LogStatus.SUCCESS) {
-      console.info(
-        'WebhookService#webhookInvoiceVoided | Invoice already voided',
+    if (invoiceSync.status !== InvoiceStatus.OPEN) {
+      console.error(
+        'WebhookService#handleInvoiceVoided | Invoices void was requested for non-open record',
       )
+      return // return early if invoice is not open
     }
 
     // get invoice sync log
@@ -726,6 +723,7 @@ export class InvoiceService extends BaseService {
       console.error(
         'WebhookService#handleInvoiceDeleted | Invoices delete was requested for non-voided record',
       )
+      return // return early if invoice is not voided
     }
 
     // get invoice sync log

src/components/dashboard/settings/sections/product/ProductMappingTable.tsx

@@ -30,7 +30,7 @@ const MapItemComponent = ({
                 currency: 'USD',
               }).format(
                 currentlyMapped.unitPrice
-                  ? parseFloat(currentlyMapped.unitPrice)
+                  ? parseFloat(currentlyMapped.unitPrice) / 100
                   : 0,
               )}
           </div>

src/hook/useQuickbooks.ts

@@ -16,7 +16,7 @@
 ) => {
   const [loading, setLoading] = useState(false)
   const [isReconnecting, setIsReconnecting] = useState(reconnect)
-  const { setAppParams } = useApp()
+  const { setAppParams, portalConnectionStatus } = useApp()
 
   useEffect(() => {
     const supabase = SupabaseClient.getInstance()
@@ -93,7 +93,7 @@
     return () => {
       supabase.removeChannel(realtimeSyncChannel)
     }
   }, [])
 
   // handle reconnect logic
   useEffect(() => {
@@ -105,7 +105,7 @@
     if (reconnect) {
       handleAppConnect()
     }
   }, [reconnect])
 
   const getAuthUrl = async (type?: string) => {
     const redirectUrl = copilotDashboardUrl
@@ -128,7 +128,7 @@
 
     // set time-out in case if user closes the popped up window. This will prevent the app from the infinite "connecting" state
     const timeout = setTimeout(() => {
-      if (!hasConnection) {
+      if (!portalConnectionStatus) {
         setLoading(false)
       }
     }, 120000) // timeout after 2 minutes

src/utils/intuitAPI.ts

@@ -412,7 +412,7 @@ export default class IntuitAPI {
 
   async _voidInvoice(payload: QBDestructiveInvoicePayloadSchema) {
     console.info(
-      `IntuitAPI#voidInvoice | invoice void creation start for realmId: ${this.tokens.intuitRealmId}. Payload: `,
+      `IntuitAPI#voidInvoice | invoice void start for realmId: ${this.tokens.intuitRealmId}. Payload: `,
       payload,
     )
     const url = `${intuitBaseUrl}/v3/company/${this.tokens.intuitRealmId}/invoice?operation=void&minorversion=${intuitApiMinorVersion}`