From b0658be041367837ba80bc6a30410c2fcaf25051 Mon Sep 17 00:00:00 2001
From: Ahmed Bouhuolia <a.bouhuolia@gmail.com>
Date: Wed, 25 Sep 2024 15:20:24 +0200
Subject: [PATCH] fix: Invoice customize bugs

---
 packages/webapp/src/constants/PdfTemplates.ts |  9 ++++++++
 .../CreditNotePaperTemplate.tsx               | 18 ++++++++++-----
 .../EstimatePaperTemplate.tsx                 | 23 ++++++++++++++-----
 .../InvoiceCustomize/InvoicePaperTemplate.tsx | 16 +++++++++----
 .../PaymentReceivedPaperTemplate.tsx          |  2 +-
 .../ReceiptCustomize/ReceiptPaperTemplate.tsx | 18 ++++++++++-----
 6 files changed, 62 insertions(+), 24 deletions(-)
 create mode 100644 packages/webapp/src/constants/PdfTemplates.ts

diff --git a/packages/webapp/src/constants/PdfTemplates.ts b/packages/webapp/src/constants/PdfTemplates.ts
new file mode 100644
index 000000000..6b1ec8b96
--- /dev/null
+++ b/packages/webapp/src/constants/PdfTemplates.ts
@@ -0,0 +1,9 @@
+export const DefaultPdfTemplateTerms = 'All services provided are non-refundable. For any disputes, please contact us within 7 days of receiving this invoice.';
+
+export const DefaultPdfTemplateStatement =
+  'Thank you for your business. We look forward to working with you again!';
+
+export const DefaultPdfTemplateItemName = 'Web development';
+
+export const DefaultPdfTemplateItemDescription =
+  'Website development with content and SEO optimization';
diff --git a/packages/webapp/src/containers/Sales/CreditNotes/CreditNoteCustomize/CreditNotePaperTemplate.tsx b/packages/webapp/src/containers/Sales/CreditNotes/CreditNoteCustomize/CreditNotePaperTemplate.tsx
index 93cef8956..2e575fc11 100644
--- a/packages/webapp/src/containers/Sales/CreditNotes/CreditNoteCustomize/CreditNotePaperTemplate.tsx
+++ b/packages/webapp/src/containers/Sales/CreditNotes/CreditNoteCustomize/CreditNotePaperTemplate.tsx
@@ -1,8 +1,14 @@
-import { Group, Stack } from '@/components';
+import { Stack } from '@/components';
 import {
   PaperTemplate,
   PaperTemplateProps,
 } from '../../Invoices/InvoiceCustomize/PaperTemplate';
+import {
+  DefaultPdfTemplateTerms,
+  DefaultPdfTemplateItemDescription,
+  DefaultPdfTemplateStatement,
+  DefaultPdfTemplateItemName,
+} from '@/constants/PdfTemplates';
 
 export interface CreditNotePaperTemplateProps extends PaperTemplateProps {
   // Address
@@ -95,18 +101,18 @@ export function CreditNotePaperTemplate({
 
   // Customer note
   showCustomerNote = true,
-  customerNote = 'It is a long established fact that a reader will be distracted by the readable content of a page when looking at its layout.',
+  customerNote = DefaultPdfTemplateStatement,
   customerNoteLabel = 'Customer Note',
 
   // Terms & conditions
   showTermsConditions = true,
-  termsConditions = 'It is a long established fact that a reader will be distracted by the readable content of a page when looking at its layout.',
+  termsConditions = DefaultPdfTemplateTerms,
   termsConditionsLabel = 'Terms & Conditions',
 
   lines = [
     {
-      item: 'Simply dummy text',
-      description: 'Simply dummy text of the printing and typesetting',
+      item: DefaultPdfTemplateItemName,
+      description: DefaultPdfTemplateItemDescription,
       rate: '1',
       quantity: '1000',
       total: '$1000.00',
@@ -161,7 +167,7 @@ export function CreditNotePaperTemplate({
           <PaperTemplate.Table
             columns={[
               { label: 'Item', accessor: 'item' },
-              { label: 'Description', accessor: 'item' },
+              { label: 'Description', accessor: 'description' },
               { label: 'Rate', accessor: 'rate', align: 'right' },
               { label: 'Total', accessor: 'total', align: 'right' },
             ]}
diff --git a/packages/webapp/src/containers/Sales/Estimates/EstimateCustomize/EstimatePaperTemplate.tsx b/packages/webapp/src/containers/Sales/Estimates/EstimateCustomize/EstimatePaperTemplate.tsx
index 7b1e6335f..5985ba3cb 100644
--- a/packages/webapp/src/containers/Sales/Estimates/EstimateCustomize/EstimatePaperTemplate.tsx
+++ b/packages/webapp/src/containers/Sales/Estimates/EstimateCustomize/EstimatePaperTemplate.tsx
@@ -3,20 +3,30 @@ import {
   PaperTemplate,
   PaperTemplateProps,
 } from '../../Invoices/InvoiceCustomize/PaperTemplate';
+import {
+  DefaultPdfTemplateTerms,
+  DefaultPdfTemplateItemDescription,
+  DefaultPdfTemplateStatement,
+  DefaultPdfTemplateItemName,
+} from '@/constants/PdfTemplates';
 
 export interface EstimatePaperTemplateProps extends PaperTemplateProps {
+  // # Estimate number
   estimateNumebr?: string;
   estimateNumberLabel?: string;
   showEstimateNumber?: boolean;
 
+  // # Expiration date
   expirationDate?: string;
   showExpirationDate?: boolean;
   expirationDateLabel?: string;
 
+  // # Estimate date
   estimateDateLabel?: string;
   showEstimateDate?: boolean;
   estimateDate?: string;
 
+  // # Customer name
   companyName?: string;
 
   // Address
@@ -36,11 +46,12 @@ export interface EstimatePaperTemplateProps extends PaperTemplateProps {
   showSubtotal?: boolean;
   subtotalLabel?: string;
 
-  // Statements
+  // # Statements
   showCustomerNote?: boolean;
   customerNote?: string;
   customerNoteLabel?: string;
 
+  // # Terms & conditions
   showTermsConditions?: boolean;
   termsConditions?: string;
   termsConditionsLabel?: string;
@@ -91,17 +102,17 @@ export function EstimatePaperTemplate({
   showSubtotal = true,
 
   showCustomerNote = true,
-  customerNote = 'It is a long established fact that a reader will be distracted by the readable content of a page when looking at its layout.',
+  customerNote = DefaultPdfTemplateStatement,
   customerNoteLabel = 'Customer Note',
 
   showTermsConditions = true,
-  termsConditions = 'It is a long established fact that a reader will be distracted by the readable content of a page when looking at its layout.',
+  termsConditions = DefaultPdfTemplateTerms,
   termsConditionsLabel = 'Terms & Conditions',
 
   lines = [
     {
-      item: 'Simply dummy text',
-      description: 'Simply dummy text of the printing and typesetting',
+      item: DefaultPdfTemplateItemName,
+      description: DefaultPdfTemplateItemDescription,
       rate: '1',
       quantity: '1000',
       total: '$1000.00',
@@ -165,7 +176,7 @@ export function EstimatePaperTemplate({
           <PaperTemplate.Table
             columns={[
               { label: 'Item', accessor: 'item' },
-              { label: 'Description', accessor: 'item' },
+              { label: 'Description', accessor: 'description' },
               { label: 'Rate', accessor: 'rate', align: 'right' },
               { label: 'Total', accessor: 'total', align: 'right' },
             ]}
diff --git a/packages/webapp/src/containers/Sales/Invoices/InvoiceCustomize/InvoicePaperTemplate.tsx b/packages/webapp/src/containers/Sales/Invoices/InvoiceCustomize/InvoicePaperTemplate.tsx
index 3e56cafd8..6620f0e28 100644
--- a/packages/webapp/src/containers/Sales/Invoices/InvoiceCustomize/InvoicePaperTemplate.tsx
+++ b/packages/webapp/src/containers/Sales/Invoices/InvoiceCustomize/InvoicePaperTemplate.tsx
@@ -1,7 +1,12 @@
 import React from 'react';
 import { PaperTemplate, PaperTemplateTotalBorder } from './PaperTemplate';
 import { Stack } from '@/components';
-
+import {
+  DefaultPdfTemplateTerms,
+  DefaultPdfTemplateItemDescription,
+  DefaultPdfTemplateStatement,
+  DefaultPdfTemplateItemName,
+} from '@/constants/PdfTemplates';
 interface PapaerLine {
   item?: string;
   description?: string;
@@ -145,12 +150,12 @@ export function InvoicePaperTemplate({
   // Footer paragraphs.
   termsConditionsLabel = 'Terms & Conditions',
   showTermsConditions = true,
-  termsConditions = 'It is a long established fact that a reader will be distracted by the readable content of a page when looking at its layout.',
+  termsConditions = DefaultPdfTemplateTerms,
 
   lines = [
     {
-      item: 'Simply dummy text',
-      description: 'Simply dummy text of the printing and typesetting',
+      item: DefaultPdfTemplateItemName,
+      description: DefaultPdfTemplateItemDescription,
       rate: '1',
       quantity: '1000',
       total: '$1000.00',
@@ -163,7 +168,7 @@ export function InvoicePaperTemplate({
 
   statementLabel = 'Statement',
   showStatement = true,
-  statement = 'It is a long established fact that a reader will be distracted by the readable content of a page when looking at its layout.',
+  statement = DefaultPdfTemplateStatement,
   billedToAddress = [
     'Bigcapital Technology, Inc.',
     '131 Continental Dr Suite 305 Newark,',
@@ -286,6 +291,7 @@ export function InvoicePaperTemplate({
               {termsConditions}
             </PaperTemplate.Statement>
           )}
+
           {showStatement && statement && (
             <PaperTemplate.Statement label={statementLabel}>
               {statement}
diff --git a/packages/webapp/src/containers/Sales/PaymentsReceived/PaymentReceivedCustomize/PaymentReceivedPaperTemplate.tsx b/packages/webapp/src/containers/Sales/PaymentsReceived/PaymentReceivedCustomize/PaymentReceivedPaperTemplate.tsx
index d8068c797..114842bf8 100644
--- a/packages/webapp/src/containers/Sales/PaymentsReceived/PaymentReceivedCustomize/PaymentReceivedPaperTemplate.tsx
+++ b/packages/webapp/src/containers/Sales/PaymentsReceived/PaymentReceivedCustomize/PaymentReceivedPaperTemplate.tsx
@@ -1,4 +1,4 @@
-import { Group, Stack } from '@/components';
+import { Stack } from '@/components';
 import {
   PaperTemplate,
   PaperTemplateProps,
diff --git a/packages/webapp/src/containers/Sales/Receipts/ReceiptCustomize/ReceiptPaperTemplate.tsx b/packages/webapp/src/containers/Sales/Receipts/ReceiptCustomize/ReceiptPaperTemplate.tsx
index 13b409558..ed00a2775 100644
--- a/packages/webapp/src/containers/Sales/Receipts/ReceiptCustomize/ReceiptPaperTemplate.tsx
+++ b/packages/webapp/src/containers/Sales/Receipts/ReceiptCustomize/ReceiptPaperTemplate.tsx
@@ -3,6 +3,12 @@ import {
   PaperTemplate,
   PaperTemplateProps,
 } from '../../Invoices/InvoiceCustomize/PaperTemplate';
+import {
+  DefaultPdfTemplateTerms,
+  DefaultPdfTemplateItemDescription,
+  DefaultPdfTemplateStatement,
+  DefaultPdfTemplateItemName,
+} from '@/constants/PdfTemplates';
 
 export interface ReceiptPaperTemplateProps extends PaperTemplateProps {
   // Addresses
@@ -22,7 +28,7 @@ export interface ReceiptPaperTemplateProps extends PaperTemplateProps {
   showSubtotal?: boolean;
   subtotalLabel?: string;
 
-// Customer Note
+  // Customer Note
   showCustomerNote?: boolean;
   customerNote?: string;
   customerNoteLabel?: string;
@@ -93,17 +99,17 @@ export function ReceiptPaperTemplate({
   showSubtotal = true,
 
   showCustomerNote = true,
-  customerNote = 'It is a long established fact that a reader will be distracted by the readable content of a page when looking at its layout.',
   customerNoteLabel = 'Customer Note',
+  customerNote = DefaultPdfTemplateStatement,
 
   showTermsConditions = true,
-  termsConditions = 'It is a long established fact that a reader will be distracted by the readable content of a page when looking at its layout.',
   termsConditionsLabel = 'Terms & Conditions',
+  termsConditions = DefaultPdfTemplateTerms,
 
   lines = [
     {
-      item: 'Simply dummy text',
-      description: 'Simply dummy text of the printing and typesetting',
+      item: DefaultPdfTemplateItemName,
+      description: DefaultPdfTemplateItemDescription,
       rate: '1',
       quantity: '1000',
       total: '$1000.00',
@@ -156,7 +162,7 @@ export function ReceiptPaperTemplate({
           <PaperTemplate.Table
             columns={[
               { label: 'Item', accessor: 'item' },
-              { label: 'Description', accessor: 'item' },
+              { label: 'Description', accessor: 'description' },
               { label: 'Rate', accessor: 'rate', align: 'right' },
               { label: 'Total', accessor: 'total', align: 'right' },
             ]}