From 9b091840d6590bfefafb8104ce7d7f5896b77076 Mon Sep 17 00:00:00 2001
From: "pramod.mahajan" <pramod.mahajan@marcoaiot.com>
Date: Wed, 5 Nov 2025 18:10:24 +0530
Subject: [PATCH] added payee default

---
 src/components/Expenses/ExpenseSchema.js      | 10 +--
 src/components/Expenses/ViewExpense.jsx       | 10 +--
 .../PaymentRequest/ManagePaymentRequest.jsx   | 90 +++++++++++++------
 .../PaymentRequest/PaymentRequestSchema.js    |  4 +-
 src/components/common/Avatar.jsx              |  2 +-
 5 files changed, 75 insertions(+), 41 deletions(-)

diff --git a/src/components/Expenses/ExpenseSchema.js b/src/components/Expenses/ExpenseSchema.js
index 65c05133..04110339 100644
--- a/src/components/Expenses/ExpenseSchema.js
+++ b/src/components/Expenses/ExpenseSchema.js
@@ -114,7 +114,7 @@ export const ExpenseActionScheam = (
       reimburseTransactionId: z.string().nullable().optional(),
       reimburseDate: z.string().nullable().optional(),
       reimburseById: z.string().nullable().optional(),
-      tdsPercentage: z.number().optional(),
+      tdsPercentage: z.number().nullable().optional(),
       baseAmount: z.string().nullable().optional(),
       taxAmount: z.string().nullable().optional(),
     })
@@ -166,10 +166,10 @@ export const defaultActionValues = {
 
   reimburseTransactionId: null,
   reimburseDate: null,
-  reimburseById: "",
-  tdsPercentage: 0,
-  baseAmount: "",
-  taxAmount: "",
+  reimburseById: null,
+  tdsPercentage: null,
+  baseAmount:null,
+  taxAmount: null,
 };
 
 export const SearchSchema = z.object({
diff --git a/src/components/Expenses/ViewExpense.jsx b/src/components/Expenses/ViewExpense.jsx
index 1d408cad..36997786 100644
--- a/src/components/Expenses/ViewExpense.jsx
+++ b/src/components/Expenses/ViewExpense.jsx
@@ -393,7 +393,7 @@ const ViewExpense = ({ ExpenseId }) => {
                 {IsPaymentProcess && nextStatusWithPermission?.length > 0 && (
                   <div className="row ">
                     <div className="col-12 col-md-6 text-start">
-                      <label className="form-label">Transaction Id </label>
+                      <Label className="form-label" required>Transaction Id </Label>
                       <input
                         type="text"
                         className="form-control form-control-sm"
@@ -405,8 +405,8 @@ const ViewExpense = ({ ExpenseId }) => {
                         </small>
                       )}
                     </div>
-                    <div className="col-12 col-md-6 text-start mb-2">
-                      <label className="form-label">Transaction Date </label>
+                    <div className="col-12 col-md-6 text-start mb-1">
+                      <Label className="form-label" required>Transaction Date </Label>
                       <DatePicker
                         name="reimburseDate"
                         control={control}
@@ -419,7 +419,7 @@ const ViewExpense = ({ ExpenseId }) => {
                         </small>
                       )}
                     </div>
-                    <div className="col-12 col-md-6 text-start mb-2">
+                    <div className="col-12 col-md-6 text-start mb-1">
                       <Label className="form-label" required>
                         Reimburse By{" "}
                       </Label>
@@ -476,7 +476,7 @@ const ViewExpense = ({ ExpenseId }) => {
                     </div>
                   </div>
                 )}
-                <div className="col-12 mb-3 text-start mt-2">
+                <div className="col-12 mb-3 text-start mt-1">
                   {((nextStatusWithPermission.length > 0 &&
                     !IsRejectedExpense) ||
                     (IsRejectedExpense && isCreatedBy)) && (
diff --git a/src/components/PaymentRequest/ManagePaymentRequest.jsx b/src/components/PaymentRequest/ManagePaymentRequest.jsx
index 77bead76..52459a56 100644
--- a/src/components/PaymentRequest/ManagePaymentRequest.jsx
+++ b/src/components/PaymentRequest/ManagePaymentRequest.jsx
@@ -1,7 +1,7 @@
 import React, { useEffect, useState } from "react";
 import { useCurrencies, useProjectName } from "../../hooks/useProjects";
 import Label from "../common/Label";
-import { useForm } from "react-hook-form";
+import { Controller, useForm } from "react-hook-form";
 import { useExpenseCategory } from "../../hooks/masterHook/useMaster";
 import DatePicker from "../common/DatePicker";
 import {
@@ -50,7 +50,12 @@ function ManagePaymentRequest({ closeModal, requestToEdit = null }) {
   } = useExpenseCategory();
 
   const { profile } = useProfile();
- const {data:Payees,isLoading:isPayeeLoaing,isError:isPayeeError,error:payeeError} = usePayee()
+  const {
+    data: Payees,
+    isLoading: isPayeeLoaing,
+    isError: isPayeeError,
+    error: payeeError,
+  } = usePayee();
   const schema = PaymentRequestSchema(ExpenseCategories);
   const {
     register,
@@ -113,7 +118,6 @@ function ManagePaymentRequest({ closeModal, requestToEdit = null }) {
       reader.onerror = (error) => reject(error);
     });
   const removeFile = (index) => {
-    debugger
     if (requestToEdit) {
       const newFiles = files.map((file, i) => {
         if (file.documentId !== index) return file;
@@ -183,7 +187,7 @@ function ManagePaymentRequest({ closeModal, requestToEdit = null }) {
     let payload = {
       ...fromdata,
       dueDate: localToUtc(fromdata.dueDate),
-      payee: isItself ? profile?.employeeInfo?.id : fromdata.payee,
+      payee: isItself ? `${profile?.employeeInfo?.firstName}  ${profile?.employeeInfo?.lastName}` : fromdata.payee,
     };
     if (requestToEdit) {
       const editPayload = {
@@ -198,7 +202,12 @@ function ManagePaymentRequest({ closeModal, requestToEdit = null }) {
   };
   const handleSetItSelf = (e) => {
     setisItself(e.target.value);
-    setValue("payee", `${profile?.employeeInfo.firstName} ${profile?.employeeInfo.lastName}`);
+    let name = `${profile?.employeeInfo.firstName} ${profile?.employeeInfo.lastName}`
+
+    setValue(
+      "payee",
+      name
+    );
   };
 
   return (
@@ -280,26 +289,53 @@ function ManagePaymentRequest({ closeModal, requestToEdit = null }) {
             )}
           </div>
 
-          <div className="col-md-6">
-            <Label htmlFor="isAdvancePayment" className="form-label" required>
-              Advance Payment
+          <div className="col-md-6 ">
+            <Label htmlFor="isAdvance" className="form-label">
+              Is Advance Payment
             </Label>
-            <select
-              id="isAdvancePayment"
-              className="form-select form-select-sm"
-              {...register("isAdvancePayment", {
-                setValueAs: (v) =>
-                  v === "true" ? true : v === "false" ? false : undefined,
-              })}
-            >
-              <option value="">Select Option</option>
-              <option value="true">True</option>
-              <option value="false">False</option>
-            </select>
-            {errors.isAdvancePayment && (
-              <small className="danger-text">
-                {errors.isAdvancePayment.message}
-              </small>
+
+            <Controller
+              name="isAdvancePayment"
+              control={control}
+              defaultValue={defaultPaymentRequest.isAdvancePayment ?? false}
+              render={({ field }) => (
+                <div className="d-flex align-items-center gap-3">
+                  <div className="form-check d-flex flex-row m-0 gap-2">
+                    <input
+                      type="radio"
+                      id="isAdvancePayment"
+                      className="form-check-input m-0"
+                      // mark checked when the controlled value is true
+                      checked={field.value === true}
+                      onChange={() => field.onChange(true)} // send boolean true
+                    />
+                    <Label
+                      className="form-check-label"
+                    >
+                      Yes
+                    </Label>
+                  </div>
+
+                  <div className="form-check d-flex flex-row m-0 gap-2">
+                    <input
+                      type="radio"
+                      id="isVariableFalse"
+                      className="form-check-input m-0"
+                      checked={field.value === false}
+                      onChange={() => field.onChange(false)} // send boolean false
+                    />
+                    <Label
+                      className="form-check-label"
+                    >
+                      No
+                    </Label>
+                  </div>
+                </div>
+              )}
+            />
+
+            {errors.isVariable && (
+              <small className="danger-text">{errors.isVariable.message}</small>
             )}
           </div>
         </div>
@@ -355,7 +391,7 @@ function ManagePaymentRequest({ closeModal, requestToEdit = null }) {
               }
               error={errors.payee?.message}
             />
-            
+
             {errors.payee && (
               <small className="danger-text">{errors.payee.message}</small>
             )}
@@ -425,9 +461,7 @@ function ManagePaymentRequest({ closeModal, requestToEdit = null }) {
         {/* Upload Document  */}
         <div className="row my-2 text-start">
           <div className="col-md-12">
-            <Label className="form-label">
-              Upload Bill{" "}
-            </Label>
+            <Label className="form-label">Upload Bill </Label>
 
             <div
               className="border border-secondary border-dashed rounded p-4 text-center bg-textMuted position-relative"
diff --git a/src/components/PaymentRequest/PaymentRequestSchema.js b/src/components/PaymentRequest/PaymentRequestSchema.js
index 7e2289fe..d89fe7d2 100644
--- a/src/components/PaymentRequest/PaymentRequestSchema.js
+++ b/src/components/PaymentRequest/PaymentRequestSchema.js
@@ -18,7 +18,7 @@ export const PaymentRequestSchema = (expenseTypes, isItself) => {
     dueDate: z.string().min(1, { message: "Date is required" }),
     description: z.string().min(1, { message: "Description is required" }),
     payee: z.string().min(1, { message: "Supplier name is required" }),
-    isAdvancePayment: z.boolean().optional(),
+    isAdvancePayment: z.boolean().optional().default(false),
     amount: z.coerce
       .number({
         invalid_type_error: "Amount is required and must be a number",
@@ -56,7 +56,7 @@ export const defaultPaymentRequest = {
   dueDate: "",
   projectId: "",
   expenseCategoryId: "",
-  isAdvancePayment: boolean,
+  isAdvancePayment: false,
   billAttachments: [],
 };
 
diff --git a/src/components/common/Avatar.jsx b/src/components/common/Avatar.jsx
index 5c3f25e6..34c14e50 100644
--- a/src/components/common/Avatar.jsx
+++ b/src/components/common/Avatar.jsx
@@ -52,7 +52,7 @@ const Avatar = ({ firstName, lastName, size = "sm", classAvatar }) => {
   return (
     <div className="avatar-wrapper p-1">
      <div className={`avatar avatar-${size} me-2 ${classAvatar}`}>
-        <span className={`avatar-initial rounded-circle ${bgClass}`}>
+        <span className={`avatar-initial rounded-circle text-white ${bgClass}`}>
           {generateAvatarText(firstName, lastName)}
         </span>
       </div>