From 05e4346b1b96ea918499e64f899c23ec7d59c172 Mon Sep 17 00:00:00 2001 From: AmirReza Jamali Date: Tue, 25 Nov 2025 15:47:40 +0330 Subject: [PATCH] refactor(inquiries): Remove unused isSubmitting state from status change - Remove isSubmitting prop from ChangeStatusForm component interface - Remove isSubmitting parameter from ChangeStatusForm component destructuring - Remove isChangingStatus variable from useChangeInquiryStatus hook - Remove isSubmitting prop passed to ChangeStatusForm in InquiriesTable - Remove isChangingStatus from presenter return object - Simplify status change form by eliminating unused loading state management --- src/components/Tables/inquiries/ChangeStatusForm.tsx | 4 ++-- src/components/Tables/inquiries/index.tsx | 3 +-- src/components/Tables/inquiries/useInquiriesListPresenter.ts | 4 ++-- 3 files changed, 5 insertions(+), 6 deletions(-) diff --git a/src/components/Tables/inquiries/ChangeStatusForm.tsx b/src/components/Tables/inquiries/ChangeStatusForm.tsx index d5d6c50..ca01634 100644 --- a/src/components/Tables/inquiries/ChangeStatusForm.tsx +++ b/src/components/Tables/inquiries/ChangeStatusForm.tsx @@ -11,14 +11,14 @@ interface ChangeStatusFormProps { onSubmit: (data: TChangeInquiryStatusCredentials) => void; onCancel: () => void; currentStatus: string; - isSubmitting?: boolean; + } const ChangeStatusForm = ({ onSubmit, onCancel, currentStatus, - isSubmitting, + }: ChangeStatusFormProps) => { const { register, diff --git a/src/components/Tables/inquiries/index.tsx b/src/components/Tables/inquiries/index.tsx index dfcdadd..db79e9a 100644 --- a/src/components/Tables/inquiries/index.tsx +++ b/src/components/Tables/inquiries/index.tsx @@ -38,7 +38,6 @@ const InquiriesTable = () => { setIsChangeStatusModalOpen, deleteInquiry, handleChangeStatus, - isChangingStatus, filterRegister, handleFilterSubmit, search, @@ -175,7 +174,7 @@ const InquiriesTable = () => { onSubmit={handleChangeStatus} onCancel={() => setIsChangeStatusModalOpen(false)} currentStatus={currentInquiry?.status ?? "pending"} - isSubmitting={isChangingStatus} + /> {isDeleteModalOpen && ( diff --git a/src/components/Tables/inquiries/useInquiriesListPresenter.ts b/src/components/Tables/inquiries/useInquiriesListPresenter.ts index 4490e04..ca3849d 100644 --- a/src/components/Tables/inquiries/useInquiriesListPresenter.ts +++ b/src/components/Tables/inquiries/useInquiriesListPresenter.ts @@ -53,7 +53,7 @@ const useInquiriesListPresenter = () => { const { data, isPending } = useGetInquiries(params); const { mutate } = useDeleteInquiry(() => setIsDeleteModalOpen(false)); - const { mutate: changeStatus, isPending: isChangingStatus } = + const { mutate: changeStatus } = useChangeInquiryStatus(() => setIsChangeStatusModalOpen(false)); const search = (data: any) => { @@ -92,7 +92,7 @@ const useInquiriesListPresenter = () => { setCurrentInquiry, deleteInquiry, handleChangeStatus, - isChangingStatus, + filterRegister, handleFilterSubmit, search,