refactor(FE): Fix dependency array in ExpenseRealizationForm useCallback

This commit is contained in:
rstubryan
2026-03-06 14:05:11 +07:00
parent 46483af4c2
commit 633deece21
2 changed files with 5 additions and 1 deletions
@@ -66,7 +66,7 @@ const ExpenseRealizationForm = ({
toast.success(createExpenseRes?.message as string);
router.push('/expense');
},
[router]
[router, initialValues?.id]
);
const updateExpenseHandler = useCallback(
@@ -178,12 +178,14 @@ const ExpenseRequestForm = ({
setInputValue: setLocationInputValue,
options: locationOptions,
isLoadingOptions: isLoadingLocationOptions,
loadMore: loadMoreLocationOptions,
} = useSelect<Location>(LocationApi.basePath, 'id', 'name');
const {
setInputValue: setVendorInputValue,
options: supplierOptions,
isLoadingOptions: isLoadingVendorOptions,
loadMore: loadMoreVendorOptions,
} = useSelect<Supplier>(SupplierApi.basePath, 'id', 'name');
const categoryChangeHandler = (val: OptionType | OptionType[] | null) => {
@@ -414,6 +416,7 @@ const ExpenseRequestForm = ({
errorMessage={formik.errors.location_id as string}
isClearable
className={{ wrapper: 'col-span-12 sm:col-span-4' }}
onMenuScrollToBottom={loadMoreLocationOptions}
/>
<DateInput
@@ -457,6 +460,7 @@ const ExpenseRequestForm = ({
}
errorMessage={formik.errors.supplier_id as string}
className={{ wrapper: 'col-span-12' }}
onMenuScrollToBottom={loadMoreVendorOptions}
/>
<RequirePermission permissions='lti.expense.document'>