refactor(FE): Fix dependency arrays and improve error handling in

purchase components
This commit is contained in:
rstubryan
2026-03-05 14:17:42 +07:00
parent 15c7452d7b
commit a5ebc6d1ae
2 changed files with 6 additions and 3 deletions
@@ -351,13 +351,13 @@ const PurchaseOrderDetail = ({
refreshApprovals();
refetchData?.();
staffApprovalModal.closeModal();
}, [refreshApprovals, refetchData]);
}, [refreshApprovals, refetchData, staffApprovalModal]);
const handleEditModalClose = useCallback(() => {
refreshApprovals();
refetchData?.();
editModal.closeModal();
}, [refreshApprovals, refetchData]);
}, [refreshApprovals, refetchData, editModal]);
// ===== DELETE HANDLER =====
const deleteItemsHandler = useCallback(async () => {
@@ -399,7 +399,7 @@ const PurchaseOrderDetail = ({
deleteModal.closeModal();
setSelectedItem(null);
setRowSelection({});
} catch (error) {
} catch {
toast.error('Terjadi kesalahan saat menghapus item pembelian');
} finally {
setIsDeleteLoading(false);
@@ -410,6 +410,8 @@ const PurchaseOrderDetail = ({
selectedItem,
selectedRowIds,
refetchData,
refreshApprovals,
deleteModal,
]);
// ===== APPROVAL/REJECTION HANDLERS =====
@@ -263,6 +263,7 @@ const PurchaseOrderInvoice = ({ data }: PurchaseOrderInvoiceProps) => {
<Page size='A4' style={pdfStyles.page}>
{/* Header Section */}
<View style={pdfStyles.header}>
{/* eslint-disable-next-line jsx-a11y/alt-text */}
<Image
src={'https://placehold.co/120x30/png'}
style={pdfStyles.logo}