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
This commit is contained in:
@@ -11,14 +11,14 @@ interface ChangeStatusFormProps {
|
|||||||
onSubmit: (data: TChangeInquiryStatusCredentials) => void;
|
onSubmit: (data: TChangeInquiryStatusCredentials) => void;
|
||||||
onCancel: () => void;
|
onCancel: () => void;
|
||||||
currentStatus: string;
|
currentStatus: string;
|
||||||
isSubmitting?: boolean;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
const ChangeStatusForm = ({
|
const ChangeStatusForm = ({
|
||||||
onSubmit,
|
onSubmit,
|
||||||
onCancel,
|
onCancel,
|
||||||
currentStatus,
|
currentStatus,
|
||||||
isSubmitting,
|
|
||||||
}: ChangeStatusFormProps) => {
|
}: ChangeStatusFormProps) => {
|
||||||
const {
|
const {
|
||||||
register,
|
register,
|
||||||
|
|||||||
@@ -38,7 +38,6 @@ const InquiriesTable = () => {
|
|||||||
setIsChangeStatusModalOpen,
|
setIsChangeStatusModalOpen,
|
||||||
deleteInquiry,
|
deleteInquiry,
|
||||||
handleChangeStatus,
|
handleChangeStatus,
|
||||||
isChangingStatus,
|
|
||||||
filterRegister,
|
filterRegister,
|
||||||
handleFilterSubmit,
|
handleFilterSubmit,
|
||||||
search,
|
search,
|
||||||
@@ -175,7 +174,7 @@ const InquiriesTable = () => {
|
|||||||
onSubmit={handleChangeStatus}
|
onSubmit={handleChangeStatus}
|
||||||
onCancel={() => setIsChangeStatusModalOpen(false)}
|
onCancel={() => setIsChangeStatusModalOpen(false)}
|
||||||
currentStatus={currentInquiry?.status ?? "pending"}
|
currentStatus={currentInquiry?.status ?? "pending"}
|
||||||
isSubmitting={isChangingStatus}
|
|
||||||
/>
|
/>
|
||||||
</Modal>
|
</Modal>
|
||||||
{isDeleteModalOpen && (
|
{isDeleteModalOpen && (
|
||||||
|
|||||||
@@ -53,7 +53,7 @@ const useInquiriesListPresenter = () => {
|
|||||||
|
|
||||||
const { data, isPending } = useGetInquiries(params);
|
const { data, isPending } = useGetInquiries(params);
|
||||||
const { mutate } = useDeleteInquiry(() => setIsDeleteModalOpen(false));
|
const { mutate } = useDeleteInquiry(() => setIsDeleteModalOpen(false));
|
||||||
const { mutate: changeStatus, isPending: isChangingStatus } =
|
const { mutate: changeStatus } =
|
||||||
useChangeInquiryStatus(() => setIsChangeStatusModalOpen(false));
|
useChangeInquiryStatus(() => setIsChangeStatusModalOpen(false));
|
||||||
|
|
||||||
const search = (data: any) => {
|
const search = (data: any) => {
|
||||||
@@ -92,7 +92,7 @@ const useInquiriesListPresenter = () => {
|
|||||||
setCurrentInquiry,
|
setCurrentInquiry,
|
||||||
deleteInquiry,
|
deleteInquiry,
|
||||||
handleChangeStatus,
|
handleChangeStatus,
|
||||||
isChangingStatus,
|
|
||||||
filterRegister,
|
filterRegister,
|
||||||
handleFilterSubmit,
|
handleFilterSubmit,
|
||||||
search,
|
search,
|
||||||
|
|||||||
Reference in New Issue
Block a user