From 7ceb25ea71dc421b68bcc24a7d006a16e23c52ee Mon Sep 17 00:00:00 2001 From: rstubryan Date: Wed, 8 Oct 2025 15:26:45 +0700 Subject: [PATCH 01/55] feat(FE-62,65): add inventory movement management with API and form validation --- .../movement/form/MovementForm.schema.ts | 67 +++++ src/config/constant.ts | 241 ++++++++++-------- src/services/api/inventory.ts | 12 + src/types/api/inventory/movement.d.ts | 51 ++++ 4 files changed, 262 insertions(+), 109 deletions(-) create mode 100644 src/components/pages/inventory/movement/form/MovementForm.schema.ts create mode 100644 src/services/api/inventory.ts create mode 100644 src/types/api/inventory/movement.d.ts diff --git a/src/components/pages/inventory/movement/form/MovementForm.schema.ts b/src/components/pages/inventory/movement/form/MovementForm.schema.ts new file mode 100644 index 00000000..cdabe355 --- /dev/null +++ b/src/components/pages/inventory/movement/form/MovementForm.schema.ts @@ -0,0 +1,67 @@ +import * as Yup from 'yup'; + +export const MovementFormSchema = Yup.object({ + alasan_transfer: Yup.string() + .required('Alasan Transfer wajib diisi!'), + tanggal_transfer: Yup.date() + .required('Tanggal Transfer wajib diisi!') + .typeError('Tanggal Transfer tidak valid!'), + warehouse_asal: Yup.object({ + value: Yup.number().min(1).required(), + label: Yup.string().required(), + }).nullable(), + warehouse_asal_id: Yup.number() + .required('Gudang Asal wajib diisi!'), + warehouse_tujuan: Yup.object({ + value: Yup.number().min(1).required(), + label: Yup.string().required(), + }).nullable(), + warehouse_tujuan_id: Yup.number() + .required('Gudang Tujuan wajib diisi!'), + alasan: Yup.string() + .required('Alasan wajib diisi!'), + product: Yup.object({ + value: Yup.number().min(1).required(), + label: Yup.string().required(), + }).nullable(), + product_id: Yup.array() + .of(Yup.number()).min(1, 'Pilih minimal 1 produk') + .required('Produk wajib diisi!'), + qty_product: Yup.array() + .of(Yup.number().min(1, 'Kuantitas minimal 1')) + .min(1, 'Pilih minimal 1 produk') + .required('Kuantitas wajib diisi!'), + ekspedisi: Yup.array().of( + Yup.object({ + product: Yup.object({ + value: Yup.number().min(1).required(), + label: Yup.string().required(), + }).nullable(), + product_id: Yup.number() + .required('Produk wajib diisi!'), + qty: Yup.number().min(1, 'Kuantitas minimal 1') + .required('Kuantitas wajib diisi!'), + supplier: Yup.object({ + value: Yup.number().min(1).required(), + label: Yup.string().required(), + }).nullable(), + supplier_id: Yup.number() + .required('Supplier wajib diisi!'), + plat_nomor: Yup.string() + .required('Plat Nomor wajib diisi!'), + no_surat_jalan: Yup.string() + .required('No Surat Jalan wajib diisi!'), + dokumen: Yup.mixed() + .required('Dokumen wajib diisi!'), + biaya_ekspedisi: Yup.number() + .min(0, 'Biaya Ekspedisi minimal 0') + .required('Biaya Ekspedisi wajib diisi!'), + nama_sopir: Yup.string() + .required('Nama Sopir wajib diisi!'), + }) + ).min(1, 'Pilih minimal 1 ekspedisi').required('Ekspedisi wajib diisi!'), +}); + +export const UpdateMovementFormSchema = MovementFormSchema; + +export type MovementFormValues = Yup.InferType; \ No newline at end of file diff --git a/src/config/constant.ts b/src/config/constant.ts index 1fbef81f..870002bf 100644 --- a/src/config/constant.ts +++ b/src/config/constant.ts @@ -1,122 +1,145 @@ export const MAIN_DRAWER_LINKS = [ - { - title: 'Dashboard', - link: '/dashboard', - icon: 'gg:chart', - }, + { + title: 'Dashboard', + link: '/dashboard', + icon: 'gg:chart', + }, - { - title: 'Master Data', - link: '/master-data', - icon: 'majesticons:data-line', - submenu: [ - { - title: 'Product', - link: '/master-data/product', - icon: 'fluent-mdl2:product-variant', - }, - { - title: 'Product Category', - link: '/master-data/product-category', - icon: 'carbon:categories', - }, - { - title: 'Bank', - link: '/master-data/bank', - icon: 'mdi:bank-outline', - }, - { - title: 'Area', - link: '/master-data/area', - icon: 'majesticons:map-marker-area-line', - }, - { - title: 'Location', - link: '/master-data/location', - icon: 'mingcute:location-line', - }, - { - title: 'Kandang', - link: '/master-data/kandang', - icon: 'mdi:farm-home-outline', - }, - { - title: 'Warehouse', - link: '/master-data/warehouse', - icon: 'hugeicons:warehouse', - }, - { - title: 'Customer', - link: '/master-data/customer', - icon: 'ix:customer', - }, - { - title: 'UOM', - link: '/master-data/uom', - icon: 'lsicon:measure-outline', - }, - { - title: 'Non-Stock', - link: '/master-data/nonstock', - icon: 'fluent:box-32-regular', - }, - { - title: 'FCR', - link: '/master-data/FCR', - icon: 'fluent:food-chicken-leg-16-regular', - }, - { - title: 'Supplier', - link: '/master-data/supplier', - icon: 'material-symbols:add-business-outline-rounded', - }, - ], - }, + { + title: 'Persediaan', + link: '/inventory', + icon: 'mdi:warehouse', + submenu: [ + { + title: 'Product', + link: '/inventory/product', + icon: 'mdi:package-variant-closed', + }, + { + title: 'Penyesuaian Stok', + link: '/inventory/adjustment', + icon: 'mdi:database-edit', + }, + { + title: 'Transfer Stok', + link: '/inventory/movement', + icon: 'mdi:swap-horizontal', + }, + ], + }, + + { + title: 'Master Data', + link: '/master-data', + icon: 'majesticons:data-line', + submenu: [ + { + title: 'Product', + link: '/master-data/product', + icon: 'fluent-mdl2:product-variant', + }, + { + title: 'Product Category', + link: '/master-data/product-category', + icon: 'carbon:categories', + }, + { + title: 'Bank', + link: '/master-data/bank', + icon: 'mdi:bank-outline', + }, + { + title: 'Area', + link: '/master-data/area', + icon: 'majesticons:map-marker-area-line', + }, + { + title: 'Location', + link: '/master-data/location', + icon: 'mingcute:location-line', + }, + { + title: 'Kandang', + link: '/master-data/kandang', + icon: 'mdi:farm-home-outline', + }, + { + title: 'Warehouse', + link: '/master-data/warehouse', + icon: 'hugeicons:warehouse', + }, + { + title: 'Customer', + link: '/master-data/customer', + icon: 'ix:customer', + }, + { + title: 'UOM', + link: '/master-data/uom', + icon: 'lsicon:measure-outline', + }, + { + title: 'Non-Stock', + link: '/master-data/nonstock', + icon: 'fluent:box-32-regular', + }, + { + title: 'FCR', + link: '/master-data/FCR', + icon: 'fluent:food-chicken-leg-16-regular', + }, + { + title: 'Supplier', + link: '/master-data/supplier', + icon: 'material-symbols:add-business-outline-rounded', + }, + ], + }, ] as const; export const ROWS_OPTIONS = [ - { - label: '10', - value: 10, - }, - { - label: '20', - value: 20, - }, - { - label: '50', - value: 50, - }, - { - label: '100', - value: 100, - }, + { + label: '10', + value: 10, + }, + { + label: '20', + value: 20, + }, + { + label: '50', + value: 50, + }, + { + label: '100', + value: 100, + }, ]; export const WAREHOUSE_TYPE_OPTIONS = [ - { - label: 'AREA', - value: 'AREA', - }, - { - label: 'LOKASI', - value: 'LOKASI', - }, - { - label: 'KANDANG', - value: 'KANDANG', - }, + { + label: 'AREA', + value: 'AREA', + }, + { + label: 'LOKASI', + value: 'LOKASI', + }, + { + label: 'KANDANG', + value: 'KANDANG', + }, ]; export const PRODUCT_FLAG_OPTIONS = [ - { label: 'DOC', value: 'DOC' }, - { label: 'PAKAN', value: 'PAKAN' }, - { label: 'PRE-STARTER', value: 'PRE-STARTER' }, - { label: 'STARTER', value: 'STARTER' }, - { label: 'FINISHER', value: 'FINISHER' }, - { label: 'OVK', value: 'OVK' }, - { label: 'OBAT', value: 'OBAT' }, - { label: 'VITAMIN', value: 'VITAMIN' }, - { label: 'KIMIA', value: 'KIMIA' }, + {label: 'DOC', value: 'DOC'}, + {label: 'PAKAN', value: 'PAKAN'}, + {label: 'PRE-STARTER', value: 'PRE-STARTER'}, + {label: 'STARTER', value: 'STARTER'}, + {label: 'FINISHER', value: 'FINISHER'}, + {label: 'OVK', value: 'OVK'}, + {label: 'OBAT', value: 'OBAT'}, + {label: 'VITAMIN', value: 'VITAMIN'}, + {label: 'KIMIA', value: 'KIMIA'}, ]; diff --git a/src/services/api/inventory.ts b/src/services/api/inventory.ts new file mode 100644 index 00000000..3e3115e2 --- /dev/null +++ b/src/services/api/inventory.ts @@ -0,0 +1,12 @@ +import { + CreateMovementPayload, + Movement, + UpdateMovementPayload, +} from "@/types/api/inventory/movement"; +import {BaseApiService} from "@/services/api/base"; + +export const MovementApi = new BaseApiService< + Movement, + CreateMovementPayload, + UpdateMovementPayload +>('/inventory/movements'); \ No newline at end of file diff --git a/src/types/api/inventory/movement.d.ts b/src/types/api/inventory/movement.d.ts new file mode 100644 index 00000000..fe2996ba --- /dev/null +++ b/src/types/api/inventory/movement.d.ts @@ -0,0 +1,51 @@ +import {BaseMetadata} from '@/types/api/api-general'; +import {Product} from "@/types/api/master-data/product"; +import {Supplier} from "@/types/api/master-data/supplier"; +import {Warehouse} from "@/types/api/master-data/warehouse"; + +export type BaseMovement = { + id: number; + alasan_transfer: string; + tanggal_transfer: string; + warehouse_asal: Warehouse; + warehouse_tujuan: Warehouse; + product: Array<{ + product: Product; + qty_product: number; + }>; + ekspedisi: Array<{ + product_id: number; + qty: number; + supplier: Supplier; + plat_nomor: string; + no_surat_jalan: string; + dokumen: string; + biaya_ekspedisi: number; + nama_sopir: string; + }>; + name: string; +}; + +export type Movement = BaseMetadata & BaseMovement; + +export type CreateMovementPayload = { + alasan: string; + warehouse_asal_id: number; + warehouse_tujuan_id: number; + product: Array<{ + product_id: number; + qty_product: number; + }>; + ekspedisi: Array<{ + product_id: number; + qty: number; + supplier_id: number; + plat_nomor: string; + no_surat_jalan: string; + dokumen: string; + biaya_ekspedisi: number; + nama_sopir: string; + }>; +} + +export type UpdateMovementPayload = CreateMovementPayload; \ No newline at end of file From 3f97ec45f8b14d8c77347ad5104540c647b4d926 Mon Sep 17 00:00:00 2001 From: rstubryan Date: Wed, 8 Oct 2025 16:02:52 +0700 Subject: [PATCH 02/55] feat(FE-64): add MovementTable component for inventory movement management --- src/app/inventory/movement/page.tsx | 11 + .../inventory/movement/MovementTable.tsx | 651 ++++++++++++++++++ 2 files changed, 662 insertions(+) create mode 100644 src/app/inventory/movement/page.tsx create mode 100644 src/components/pages/inventory/movement/MovementTable.tsx diff --git a/src/app/inventory/movement/page.tsx b/src/app/inventory/movement/page.tsx new file mode 100644 index 00000000..12fe795b --- /dev/null +++ b/src/app/inventory/movement/page.tsx @@ -0,0 +1,11 @@ +import MovementTable from '@/components/pages/inventory/movement/MovementTable'; + +const Product = () => { + return ( +
+ +
+ ); +}; + +export default Product; diff --git a/src/components/pages/inventory/movement/MovementTable.tsx b/src/components/pages/inventory/movement/MovementTable.tsx new file mode 100644 index 00000000..288cb2a6 --- /dev/null +++ b/src/components/pages/inventory/movement/MovementTable.tsx @@ -0,0 +1,651 @@ +'use client'; + +import { useState, useMemo } from 'react'; +import { CellContext, ColumnDef, SortingState } from '@tanstack/react-table'; +import { Icon } from '@iconify/react'; +import Table from '@/components/Table'; +import DebouncedTextInput from '@/components/input/DebouncedTextInput'; +import Button from '@/components/Button'; +import { useModal } from '@/components/Modal'; +import ConfirmationModal from '@/components/modal/ConfirmationModal'; +import SelectInput, { OptionType } from '@/components/input/SelectInput'; +import RowDropdownOptions from '@/components/table/RowDropdownOptions'; +import RowCollapseOptions from '@/components/table/RowCollapseOptions'; +import { cn } from '@/lib/helper'; +import { ROWS_OPTIONS } from '@/config/constant'; +import { Movement } from '@/types/api/inventory/movement'; +import { BaseMetadata } from '@/types/api/api-general'; + +// Dummy data +const baseMetadata: BaseMetadata = { + created_user: { + id: 1, + id_user: 1, + email: 'user@example.com', + name: 'User', + }, + created_at: '2024-06-01T00:00:00Z', + updated_at: '2024-06-01T00:00:00Z', +}; + +const dummyMovements: Movement[] = [ + { + ...baseMetadata, + id: 1, + alasan_transfer: 'Restock', + tanggal_transfer: '2024-06-01', + warehouse_asal: { + ...baseMetadata, + id: 1, + name: 'Warehouse A', + type: 'AREA', + area: { id: 1, name: 'Area 1' }, + }, + warehouse_tujuan: { + ...baseMetadata, + id: 2, + name: 'Warehouse B', + type: 'AREA', + area: { id: 2, name: 'Area 2' }, + }, + product: [ + { + product: { + ...baseMetadata, + id: 1, + name: 'Product X', + brand: 'Brand X', + sku: 'SKU-X', + product_price: 10000, + selling_price: 12000, + tax: 10, + expiry_period: 365, + uom: { + ...baseMetadata, + id: 1, + name: 'PCS', + }, + product_category: { + ...baseMetadata, + id: 1, + code: 'CAT-1', + name: 'Category 1', + }, + suppliers: [], + flags: [], + }, + qty_product: 10, + }, + ], + ekspedisi: [ + { + product_id: 1, + qty: 10, + supplier: { + ...baseMetadata, + id: 1, + name: 'Supplier 1', + alias: 'S1', + category: 'General', + pic: 'PIC 1', + type: 'Type 1', + phone: '08123456789', + email: 'supplier1@example.com', + address: 'Address 1', + account_number: '1234567890', + balance: 0, + due_date: 30, + }, + plat_nomor: 'B 1234 CD', + no_surat_jalan: 'SJ-001', + dokumen: 'doc1.pdf', + biaya_ekspedisi: 50000, + nama_sopir: 'Andi', + }, + ], + name: 'Movement 1', + }, + { + ...baseMetadata, + id: 2, + alasan_transfer: 'Mutasi Stok', + tanggal_transfer: '2024-06-02', + warehouse_asal: { + ...baseMetadata, + id: 2, + name: 'Warehouse B', + type: 'AREA', + area: { id: 2, name: 'Area 2' }, + }, + warehouse_tujuan: { + ...baseMetadata, + id: 3, + name: 'Warehouse C', + type: 'AREA', + area: { id: 3, name: 'Area 3' }, + }, + product: [ + { + product: { + ...baseMetadata, + id: 2, + name: 'Product Y', + brand: 'Brand Y', + sku: 'SKU-Y', + product_price: 20000, + selling_price: 25000, + tax: 5, + expiry_period: 180, + uom: { + ...baseMetadata, + id: 2, + name: 'BOX', + }, + product_category: { + ...baseMetadata, + id: 2, + code: 'CAT-2', + name: 'Category 2', + }, + suppliers: [], + flags: [], + }, + qty_product: 5, + }, + ], + ekspedisi: [ + { + product_id: 2, + qty: 5, + supplier: { + ...baseMetadata, + id: 2, + name: 'Supplier 2', + alias: 'S2', + category: 'Special', + pic: 'PIC 2', + type: 'Type 2', + phone: '08123456780', + email: 'supplier2@example.com', + address: 'Address 2', + account_number: '1234567891', + balance: 1000, + due_date: 15, + }, + plat_nomor: 'D 5678 EF', + no_surat_jalan: 'SJ-002', + dokumen: 'doc2.pdf', + biaya_ekspedisi: 60000, + nama_sopir: 'Budi', + }, + ], + name: 'Movement 2', + }, + { + ...baseMetadata, + id: 3, + alasan_transfer: 'Pengembalian', + tanggal_transfer: '2024-06-03', + warehouse_asal: { + ...baseMetadata, + id: 3, + name: 'Warehouse C', + type: 'AREA', + area: { id: 3, name: 'Area 3' }, + }, + warehouse_tujuan: { + ...baseMetadata, + id: 1, + name: 'Warehouse A', + type: 'AREA', + area: { id: 1, name: 'Area 1' }, + }, + product: [ + { + product: { + ...baseMetadata, + id: 3, + name: 'Product Z', + brand: 'Brand Z', + sku: 'SKU-Z', + product_price: 15000, + selling_price: 18000, + tax: 8, + expiry_period: 90, + uom: { + ...baseMetadata, + id: 3, + name: 'KG', + }, + product_category: { + ...baseMetadata, + id: 3, + code: 'CAT-3', + name: 'Category 3', + }, + suppliers: [], + flags: [], + }, + qty_product: 8, + }, + ], + ekspedisi: [ + { + product_id: 3, + qty: 8, + supplier: { + ...baseMetadata, + id: 3, + name: 'Supplier 3', + alias: 'S3', + category: 'Return', + pic: 'PIC 3', + type: 'Type 3', + phone: '08123456781', + email: 'supplier3@example.com', + address: 'Address 3', + account_number: '1234567892', + balance: 500, + due_date: 10, + }, + plat_nomor: 'F 9101 GH', + no_surat_jalan: 'SJ-003', + dokumen: 'doc3.pdf', + biaya_ekspedisi: 40000, + nama_sopir: 'Cici', + }, + ], + name: 'Movement 3', + }, + { + ...baseMetadata, + id: 4, + alasan_transfer: 'Transfer Internal', + tanggal_transfer: '2024-06-04', + warehouse_asal: { + ...baseMetadata, + id: 4, + name: 'Warehouse D', + type: 'AREA', + area: { id: 4, name: 'Area 4' }, + }, + warehouse_tujuan: { + ...baseMetadata, + id: 5, + name: 'Warehouse E', + type: 'AREA', + area: { id: 5, name: 'Area 5' }, + }, + product: [ + { + product: { + ...baseMetadata, + id: 4, + name: 'Product A', + brand: 'Brand A', + sku: 'SKU-A', + product_price: 5000, + selling_price: 7000, + tax: 0, + expiry_period: 60, + uom: { + ...baseMetadata, + id: 4, + name: 'LITER', + }, + product_category: { + ...baseMetadata, + id: 4, + code: 'CAT-4', + name: 'Category 4', + }, + suppliers: [], + flags: [], + }, + qty_product: 20, + }, + ], + ekspedisi: [ + { + product_id: 4, + qty: 20, + supplier: { + ...baseMetadata, + id: 4, + name: 'Supplier 4', + alias: 'S4', + category: 'Internal', + pic: 'PIC 4', + type: 'Type 4', + phone: '08123456782', + email: 'supplier4@example.com', + address: 'Address 4', + account_number: '1234567893', + balance: 200, + due_date: 20, + }, + plat_nomor: 'H 2345 IJ', + no_surat_jalan: 'SJ-004', + dokumen: 'doc4.pdf', + biaya_ekspedisi: 30000, + nama_sopir: 'Dedi', + }, + ], + name: 'Movement 4', + }, + { + ...baseMetadata, + id: 5, + alasan_transfer: 'Distribusi', + tanggal_transfer: '2024-06-05', + warehouse_asal: { + ...baseMetadata, + id: 5, + name: 'Warehouse E', + type: 'AREA', + area: { id: 5, name: 'Area 5' }, + }, + warehouse_tujuan: { + ...baseMetadata, + id: 1, + name: 'Warehouse A', + type: 'AREA', + area: { id: 1, name: 'Area 1' }, + }, + product: [ + { + product: { + ...baseMetadata, + id: 5, + name: 'Product B', + brand: 'Brand B', + sku: 'SKU-B', + product_price: 8000, + selling_price: 9500, + tax: 2, + expiry_period: 120, + uom: { + ...baseMetadata, + id: 5, + name: 'PAK', + }, + product_category: { + ...baseMetadata, + id: 5, + code: 'CAT-5', + name: 'Category 5', + }, + suppliers: [], + flags: [], + }, + qty_product: 15, + }, + ], + ekspedisi: [ + { + product_id: 5, + qty: 15, + supplier: { + ...baseMetadata, + id: 5, + name: 'Supplier 5', + alias: 'S5', + category: 'Distribusi', + pic: 'PIC 5', + type: 'Type 5', + phone: '08123456783', + email: 'supplier5@example.com', + address: 'Address 5', + account_number: '1234567894', + balance: 300, + due_date: 25, + }, + plat_nomor: 'K 6789 KL', + no_surat_jalan: 'SJ-005', + dokumen: 'doc5.pdf', + biaya_ekspedisi: 70000, + nama_sopir: 'Eka', + }, + ], + name: 'Movement 5', + }, +]; + +const RowOptionsMenu = ({ + type = 'dropdown', + props, + deleteClickHandler, +}: { + type: 'dropdown' | 'collapse'; + props: CellContext; + deleteClickHandler: () => void; +}) => ( +
+ + + +
+); + +const MovementTable = () => { + const [search, setSearch] = useState(''); + const [page, setPage] = useState(1); + const [pageSize, setPageSize] = useState(10); + const [sorting, setSorting] = useState([]); + const [selectedMovement, setSelectedMovement] = useState< + Movement | undefined + >(undefined); + const [isDeleteLoading, setIsDeleteLoading] = useState(false); + + const paginatedData = useMemo(() => { + const start = (page - 1) * pageSize; + return dummyMovements.slice(start, start + pageSize); + }, [page, pageSize]); + + const movementsColumns: ColumnDef[] = [ + { + header: '#', + cell: (props) => pageSize * (page - 1) + props.row.index + 1, + }, + { + accessorKey: 'warehouse_asal', + header: 'Gudang Asal', + cell: (props) => props.row.original.warehouse_asal.name, + }, + { + accessorKey: 'warehouse_tujuan', + header: 'Gudang Tujuan', + cell: (props) => props.row.original.warehouse_tujuan.name, + }, + { + accessorKey: 'product', + header: 'Nama Produk', + cell: (props) => props.row.original.product.map((p) => p.product.name), + }, + { + accessorKey: 'alasan_transfer', + header: 'Catatan', + }, + { + accessorKey: 'biaya_ekspedisi', + header: 'Biaya Ekspedisi', + cell: (props) => + props.row.original.ekspedisi.map((e) => e.biaya_ekspedisi), + }, + { + header: 'Aksi', + cell: (props) => { + const currentPageSize = props.table.getPaginationRowModel().rows.length; + const currentPageRows = props.table.getPaginationRowModel().flatRows; + const currentRowRelativeIndex = + currentPageRows.findIndex((r) => r.id === props.row.id) + 1; + + const isLast2Rows = currentRowRelativeIndex > currentPageSize - 2; + + const deleteClickHandler = () => { + setSelectedMovement(props.row.original); + deleteModal.openModal(); + }; + + return ( + <> + {currentPageSize > 2 && ( + + + + )} + {currentPageSize <= 2 && ( + + + + )} + + ); + }, + }, + ]; + + const deleteModal = useModal(); + const confirmationModalDeleteClickHandler = async () => { + setIsDeleteLoading(true); + setTimeout(() => { + setIsDeleteLoading(false); + deleteModal.closeModal(); + }, 1000); + }; + + const searchChangeHandler: React.ChangeEventHandler = ( + e + ) => { + setSearch(e.target.value); + setPage(1); + }; + + const pageSizeChangeHandler = (val: OptionType | OptionType[] | null) => { + const newVal = val as OptionType; + setPageSize(newVal.value as number); + setPage(1); + }; + + return ( + <> +
+
+
+
+ +
+ +
+
+ +
+
+ + data={paginatedData} + columns={movementsColumns} + pageSize={pageSize} + page={page} + totalItems={dummyMovements.length} + onPageChange={setPage} + isLoading={false} + sorting={sorting} + setSorting={setSorting} + className={{ + containerClassName: cn({ + 'mb-20': paginatedData.length === 0, + }), + tableWrapperClassName: 'overflow-x-auto min-h-full!', + tableClassName: 'font-inter w-full table-auto min-h-full!', + headerRowClassName: 'border-b border-b-gray-200', + headerColumnClassName: + 'px-6 py-3 text-xs font-semibold text-gray-500 last:flex last:flex-row last:justify-end', + bodyRowClassName: 'border-b border-b-gray-200', + bodyColumnClassName: + 'px-6 py-3 last:flex last:flex-row last:justify-end', + }} + /> +
+ + + ); +}; + +export default MovementTable; From ddbf8b0896618e5810108f4dde360ceaa69f7707 Mon Sep 17 00:00:00 2001 From: rstubryan Date: Wed, 8 Oct 2025 16:16:12 +0700 Subject: [PATCH 03/55] refactor(FE-62): rename Product component to Movement for clarity --- src/app/inventory/movement/page.tsx | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/app/inventory/movement/page.tsx b/src/app/inventory/movement/page.tsx index 12fe795b..a2c25612 100644 --- a/src/app/inventory/movement/page.tsx +++ b/src/app/inventory/movement/page.tsx @@ -1,6 +1,6 @@ import MovementTable from '@/components/pages/inventory/movement/MovementTable'; -const Product = () => { +const Movement = () => { return (
@@ -8,4 +8,4 @@ const Product = () => { ); }; -export default Product; +export default Movement; From 1ea9ee30695459fb5c0eac38e564235fb1732a1d Mon Sep 17 00:00:00 2001 From: rstubryan Date: Thu, 9 Oct 2025 14:30:05 +0700 Subject: [PATCH 04/55] feat(FE-62,63,65): implement MovementForm component for managing inventory movements --- src/app/inventory/movement/add/page.tsx | 11 + .../movement/form/MovementForm.schema.ts | 121 +-- .../inventory/movement/form/MovementForm.tsx | 751 ++++++++++++++++++ src/types/api/inventory/movement.d.ts | 86 +- 4 files changed, 866 insertions(+), 103 deletions(-) create mode 100644 src/app/inventory/movement/add/page.tsx create mode 100644 src/components/pages/inventory/movement/form/MovementForm.tsx diff --git a/src/app/inventory/movement/add/page.tsx b/src/app/inventory/movement/add/page.tsx new file mode 100644 index 00000000..f883de95 --- /dev/null +++ b/src/app/inventory/movement/add/page.tsx @@ -0,0 +1,11 @@ +import MovementForm from '@/components/pages/inventory/movement/form/MovementForm'; + +const AddMovement = () => { + return ( +
+ +
+ ); +}; + +export default AddMovement; diff --git a/src/components/pages/inventory/movement/form/MovementForm.schema.ts b/src/components/pages/inventory/movement/form/MovementForm.schema.ts index cdabe355..11c40fe5 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.schema.ts +++ b/src/components/pages/inventory/movement/form/MovementForm.schema.ts @@ -1,67 +1,68 @@ import * as Yup from 'yup'; export const MovementFormSchema = Yup.object({ - alasan_transfer: Yup.string() - .required('Alasan Transfer wajib diisi!'), - tanggal_transfer: Yup.date() - .required('Tanggal Transfer wajib diisi!') - .typeError('Tanggal Transfer tidak valid!'), - warehouse_asal: Yup.object({ - value: Yup.number().min(1).required(), - label: Yup.string().required(), - }).nullable(), - warehouse_asal_id: Yup.number() - .required('Gudang Asal wajib diisi!'), - warehouse_tujuan: Yup.object({ - value: Yup.number().min(1).required(), - label: Yup.string().required(), - }).nullable(), - warehouse_tujuan_id: Yup.number() - .required('Gudang Tujuan wajib diisi!'), - alasan: Yup.string() - .required('Alasan wajib diisi!'), - product: Yup.object({ - value: Yup.number().min(1).required(), - label: Yup.string().required(), - }).nullable(), - product_id: Yup.array() - .of(Yup.number()).min(1, 'Pilih minimal 1 produk') - .required('Produk wajib diisi!'), - qty_product: Yup.array() - .of(Yup.number().min(1, 'Kuantitas minimal 1')) - .min(1, 'Pilih minimal 1 produk') - .required('Kuantitas wajib diisi!'), - ekspedisi: Yup.array().of( - Yup.object({ - product: Yup.object({ - value: Yup.number().min(1).required(), - label: Yup.string().required(), - }).nullable(), - product_id: Yup.number() - .required('Produk wajib diisi!'), - qty: Yup.number().min(1, 'Kuantitas minimal 1') - .required('Kuantitas wajib diisi!'), - supplier: Yup.object({ - value: Yup.number().min(1).required(), - label: Yup.string().required(), - }).nullable(), - supplier_id: Yup.number() - .required('Supplier wajib diisi!'), - plat_nomor: Yup.string() - .required('Plat Nomor wajib diisi!'), - no_surat_jalan: Yup.string() - .required('No Surat Jalan wajib diisi!'), - dokumen: Yup.mixed() - .required('Dokumen wajib diisi!'), - biaya_ekspedisi: Yup.number() - .min(0, 'Biaya Ekspedisi minimal 0') - .required('Biaya Ekspedisi wajib diisi!'), - nama_sopir: Yup.string() - .required('Nama Sopir wajib diisi!'), - }) - ).min(1, 'Pilih minimal 1 ekspedisi').required('Ekspedisi wajib diisi!'), + alasan_transfer: Yup.string().required('Alasan transfer wajib diisi!'), + tanggal_transfer: Yup.string().required('Tanggal transfer wajib diisi!'), + warehouse_asal: Yup.object({ + value: Yup.number().min(1).required(), + label: Yup.string().required(), + }).nullable(), + warehouse_asal_id: Yup.number() + .required('Gudang asal wajib diisi!') + .typeError('Gudang asal wajib diisi!'), + warehouse_tujuan: Yup.object({ + value: Yup.number().min(1).required(), + label: Yup.string().required(), + }).nullable(), + warehouse_tujuan_id: Yup.number() + .required('Gudang tujuan wajib diisi!') + .typeError('Gudang tujuan wajib diisi!'), + product: Yup.array() + .of( + Yup.object({ + product: Yup.object({ + value: Yup.number().min(1).required(), + label: Yup.string().required(), + }).nullable(), + product_id: Yup.number().required('Produk wajib diisi!'), + qty_product: Yup.number() + .required('Qty wajib diisi!') + .min(1, 'Qty minimal 1!') + .typeError('Qty harus berupa angka!'), + }) + ) + .min(1, 'Minimal harus ada 1 produk!'), + ekspedisi: Yup.array() + .of( + Yup.object({ + product: Yup.object({ + value: Yup.number().min(1).required(), + label: Yup.string().required(), + }).nullable(), + product_id: Yup.number().required('Produk wajib diisi!'), + qty: Yup.number() + .required('Qty wajib diisi!') + .min(1, 'Qty minimal 1!') + .typeError('Qty harus berupa angka!'), + supplier: Yup.object({ + value: Yup.number().min(1).required(), + label: Yup.string().required(), + }).nullable(), + supplier_id: Yup.number().required('Supplier wajib diisi!'), + plat_nomor: Yup.string().required('Plat nomor wajib diisi!'), + no_surat_jalan: Yup.string().required('No surat jalan wajib diisi!'), + dokumen: Yup.mixed().required('Dokumen wajib diisi!'), + biaya_ekspedisi: Yup.number() + .required('Biaya ekspedisi wajib diisi!') + .min(0, 'Biaya minimal 0!') + .typeError('Biaya harus berupa angka!'), + nama_sopir: Yup.string().required('Nama sopir wajib diisi!'), + }) + ) + .optional() + .default([]), }); export const UpdateMovementFormSchema = MovementFormSchema; -export type MovementFormValues = Yup.InferType; \ No newline at end of file +export type MovementFormValues = Yup.InferType; diff --git a/src/components/pages/inventory/movement/form/MovementForm.tsx b/src/components/pages/inventory/movement/form/MovementForm.tsx new file mode 100644 index 00000000..0105768d --- /dev/null +++ b/src/components/pages/inventory/movement/form/MovementForm.tsx @@ -0,0 +1,751 @@ +'use client'; + +import { useCallback, useEffect, useMemo, useState } from 'react'; +import { useRouter } from 'next/navigation'; +import { FieldArray, FormikProvider, useFormik } from 'formik'; +import { toast } from 'react-hot-toast'; +import useSWR from 'swr'; + +import { Icon } from '@iconify/react'; +import Button from '@/components/Button'; +import TextInput from '@/components/input/TextInput'; +import SelectInput, { OptionType } from '@/components/input/SelectInput'; +import { useModal } from '@/components/Modal'; +import ConfirmationModal from '@/components/modal/ConfirmationModal'; + +import { + MovementFormSchema, + MovementFormValues, + UpdateMovementFormSchema, +} from '@/components/pages/inventory/movement/form/MovementForm.schema'; +import { isResponseError, isResponseSuccess } from '@/lib/api-helper'; +import { + Movement, + CreateMovementPayload, + UpdateMovementPayload, +} from '@/types/api/inventory/movement'; +import { + ProductApi, + WarehouseApi, + SupplierApi, +} from '@/services/api/master-data'; +import { MovementApi } from '@/services/api/inventory'; +import { cn } from '@/lib/helper'; + +interface MovementFormProps { + type?: 'add' | 'edit' | 'detail'; + initialValues?: Movement; +} + +const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { + const router = useRouter(); + const deleteModal = useModal(); + + const [movementFormErrorMessage, setMovementFormErrorMessage] = useState(''); + const [isDeleteLoading, setIsDeleteLoading] = useState(false); + + const createMovementHandler = useCallback( + async (payload: CreateMovementPayload) => { + const res = await MovementApi.create(payload); + if (isResponseError(res)) { + setMovementFormErrorMessage(res.message); + return; + } + toast.success(res?.message as string); + router.push('/inventory/movement'); + }, + [router] + ); + + const updateMovementHandler = useCallback( + async (movementId: number, payload: UpdateMovementPayload) => { + const res = await MovementApi.update(movementId, payload); + if (res?.status === 'error') { + setMovementFormErrorMessage(res.message); + return; + } + toast.success(res?.message as string); + router.refresh(); + router.push('/inventory/movement'); + }, + [router] + ); + + const formikInitialValues = useMemo( + () => ({ + alasan_transfer: initialValues?.alasan_transfer ?? '', + tanggal_transfer: initialValues?.tanggal_transfer ?? '', + warehouse_asal: initialValues?.warehouse_asal + ? { + value: initialValues.warehouse_asal.id, + label: initialValues.warehouse_asal.name, + } + : null, + warehouse_asal_id: initialValues?.warehouse_asal?.id ?? 0, + warehouse_tujuan: initialValues?.warehouse_tujuan + ? { + value: initialValues.warehouse_tujuan.id, + label: initialValues.warehouse_tujuan.name, + } + : null, + warehouse_tujuan_id: initialValues?.warehouse_tujuan?.id ?? 0, + product: + initialValues?.product?.map((p) => ({ + product: { value: p.product.id, label: p.product.name }, + product_id: p.product.id, + qty_product: p.qty_product, + })) ?? [], + ekspedisi: + initialValues?.ekspedisi?.map((e) => ({ + product: { value: e.product_id, label: '' }, // Need to fetch product details + product_id: e.product_id, + qty: e.qty, + supplier: { value: e.supplier.id, label: e.supplier.name }, + supplier_id: e.supplier.id, + plat_nomor: e.plat_nomor, + no_surat_jalan: e.no_surat_jalan, + dokumen: e.dokumen, + biaya_ekspedisi: e.biaya_ekspedisi, + nama_sopir: e.nama_sopir, + })) ?? [], + }), + [initialValues] + ); + + const formik = useFormik({ + initialValues: formikInitialValues, + validationSchema: + type === 'edit' ? UpdateMovementFormSchema : MovementFormSchema, + onSubmit: async (values) => { + setMovementFormErrorMessage(''); + const payload: CreateMovementPayload = { + alasan_transfer: values.alasan_transfer, + tanggal_transfer: values.tanggal_transfer, + warehouse_asal_id: values.warehouse_asal_id, + warehouse_tujuan_id: values.warehouse_tujuan_id, + product: (values.product ?? []).map((p) => ({ + product_id: p.product_id, + qty_product: p.qty_product, + })), + ekspedisi: (values.ekspedisi ?? []).map((e) => ({ + product_id: e.product_id, + qty: e.qty, + supplier_id: e.supplier_id, + plat_nomor: e.plat_nomor, + no_surat_jalan: e.no_surat_jalan, + dokumen: + e.dokumen instanceof File ? e.dokumen : (e.dokumen as string), + biaya_ekspedisi: e.biaya_ekspedisi, + nama_sopir: e.nama_sopir, + })), + }; + + switch (type) { + case 'add': + await createMovementHandler(payload); + break; + case 'edit': + await updateMovementHandler(initialValues?.id as number, payload); + break; + } + }, + }); + + // Warehouse selection + const [warehouseSelectInputValue, setWarehouseSelectInputValue] = + useState(''); + const warehousesUrl = `${WarehouseApi.basePath}?${new URLSearchParams({ search: warehouseSelectInputValue }).toString()}`; + const { data: warehouses, isLoading: isLoadingWarehouses } = useSWR( + warehousesUrl, + WarehouseApi.getAllFetcher + ); + const warehouseOptions = isResponseSuccess(warehouses) + ? warehouses?.data.map((w) => ({ value: w.id, label: w.name })) + : []; + + // Product selection + const [productSelectInputValue, setProductSelectInputValue] = useState(''); + const productsUrl = `${ProductApi.basePath}?${new URLSearchParams({ search: productSelectInputValue }).toString()}`; + const { data: products, isLoading: isLoadingProducts } = useSWR( + productsUrl, + ProductApi.getAllFetcher + ); + const productOptions = isResponseSuccess(products) + ? products?.data.map((p) => ({ value: p.id, label: p.name })) + : []; + + // Supplier selection + const [supplierSelectInputValue, setSupplierSelectInputValue] = useState(''); + const suppliersUrl = `${SupplierApi.basePath}?${new URLSearchParams({ search: supplierSelectInputValue }).toString()}`; + const { data: suppliers, isLoading: isLoadingSuppliers } = useSWR( + suppliersUrl, + SupplierApi.getAllFetcher + ); + const supplierOptions = isResponseSuccess(suppliers) + ? suppliers?.data.map((s) => ({ value: s.id, label: s.name })) + : []; + + const deleteMovementClickHandler = () => { + deleteModal.openModal(); + }; + + const confirmationModalDeleteClickHandler = async () => { + setIsDeleteLoading(true); + await MovementApi.delete(initialValues?.id as number); + deleteModal.closeModal(); + toast.success('Successfully delete Movement!'); + setIsDeleteLoading(false); + router.push('/inventory/movement'); + }; + + const { setValues: formikSetValues } = formik; + + useEffect(() => { + formikSetValues(formikInitialValues); + }, [formikSetValues, formikInitialValues]); + + return ( + <> +
+
+ +

+ {type === 'add' && 'Tambah Movement'} + {type === 'edit' && 'Edit Movement'} + {type === 'detail' && 'Detail Movement'} +

+
+ + +
+ {/* Top card - Movement details */} +
+
+
+ + +
+
+
+ + {/* Warehouse cards */} +
+
+
+ { + formik.setFieldValue('warehouse_asal', val); + formik.setFieldValue( + 'warehouse_asal_id', + (val as OptionType)?.value + ); + }} + options={warehouseOptions} + onInputChange={setWarehouseSelectInputValue} + isLoading={isLoadingWarehouses} + isError={ + formik.touched.warehouse_asal_id && + Boolean(formik.errors.warehouse_asal_id) + } + errorMessage={formik.errors.warehouse_asal_id as string} + isDisabled={type === 'detail'} + isClearable + /> +
+
+ +
+
+ { + formik.setFieldValue('warehouse_tujuan', val); + formik.setFieldValue( + 'warehouse_tujuan_id', + (val as OptionType)?.value + ); + }} + options={warehouseOptions} + onInputChange={setWarehouseSelectInputValue} + isLoading={isLoadingWarehouses} + isError={ + formik.touched.warehouse_tujuan_id && + Boolean(formik.errors.warehouse_tujuan_id) + } + errorMessage={formik.errors.warehouse_tujuan_id as string} + isDisabled={type === 'detail'} + isClearable + /> +
+
+
+ + {/* Products table */} +
+
+

Produk

+ + {({ push, remove }) => ( + <> + {typeof formik.errors.product === 'string' && ( +
+ {formik.errors.product} +
+ )} + + + + + + + + + + {formik.values.product?.map((_, index) => ( + + + + + + )) ?? []} + +
ProdukQtyAksi
+ { + formik.setFieldValue( + `product.${index}.product`, + val + ); + formik.setFieldValue( + `product.${index}.product_id`, + (val as OptionType)?.value + ); + }} + options={productOptions} + onInputChange={setProductSelectInputValue} + isLoading={isLoadingProducts} + isDisabled={type === 'detail'} + isClearable + /> + + + formik.setFieldValue( + `product.${index}.qty_product`, + e.target.value + ) + } + readOnly={type === 'detail'} + /> + + {type !== 'detail' && ( + + )} +
+ {type !== 'detail' && ( + + )} + + )} +
+
+
+ + {/* Ekspedisi table */} +
+
+

Ekspedisi

+ + {({ push, remove }) => ( + <> + {typeof formik.errors.ekspedisi === 'string' && ( +
+ {formik.errors.ekspedisi} +
+ )} + + + + + + + + + + + + + + + + {formik.values.ekspedisi?.map((ekspedisi, index) => ( + + + + + + + + + + + + )) ?? []} + +
ProdukQtySupplierPlat NomorNo Surat JalanDokumenBiaya EkspedisiNama SopirAksi
+ { + formik.setFieldValue( + `ekspedisi.${index}.product`, + val + ); + formik.setFieldValue( + `ekspedisi.${index}.product_id`, + (val as OptionType)?.value + ); + }} + options={productOptions} + onInputChange={setProductSelectInputValue} + isLoading={isLoadingProducts} + isDisabled={type === 'detail'} + isClearable + /> + + + formik.setFieldValue( + `ekspedisi.${index}.qty`, + e.target.value + ) + } + readOnly={type === 'detail'} + /> + + { + formik.setFieldValue( + `ekspedisi.${index}.supplier`, + val + ); + formik.setFieldValue( + `ekspedisi.${index}.supplier_id`, + (val as OptionType)?.value + ); + }} + options={supplierOptions} + onInputChange={setSupplierSelectInputValue} + isLoading={isLoadingSuppliers} + isDisabled={type === 'detail'} + isClearable + /> + + + formik.setFieldValue( + `ekspedisi.${index}.plat_nomor`, + e.target.value + ) + } + readOnly={type === 'detail'} + /> + + + formik.setFieldValue( + `ekspedisi.${index}.no_surat_jalan`, + e.target.value + ) + } + readOnly={type === 'detail'} + /> + + { + const file = e.target.files?.[0]; + if (file) { + formik.setFieldValue( + `ekspedisi.${index}.dokumen`, + file + ); + } + }} + readOnly={type === 'detail'} + /> + + + formik.setFieldValue( + `ekspedisi.${index}.biaya_ekspedisi`, + e.target.value + ) + } + readOnly={type === 'detail'} + /> + + + formik.setFieldValue( + `ekspedisi.${index}.nama_sopir`, + e.target.value + ) + } + readOnly={type === 'detail'} + /> + + {type !== 'detail' && ( + + )} +
+ {type !== 'detail' && ( + + )} + + )} +
+
+
+ + {/* Action buttons */} +
+ {type !== 'add' && ( +
+ + {type !== 'edit' && ( + + )} +
+ )} + {type !== 'detail' && ( +
+ + +
+ )} +
+ + {movementFormErrorMessage && ( +
+ + {movementFormErrorMessage} +
+ )} +
+
+
+ + {type !== 'add' && ( + + )} + + ); +}; + +export default MovementForm; diff --git a/src/types/api/inventory/movement.d.ts b/src/types/api/inventory/movement.d.ts index fe2996ba..0d58b8e1 100644 --- a/src/types/api/inventory/movement.d.ts +++ b/src/types/api/inventory/movement.d.ts @@ -1,51 +1,51 @@ -import {BaseMetadata} from '@/types/api/api-general'; -import {Product} from "@/types/api/master-data/product"; -import {Supplier} from "@/types/api/master-data/supplier"; -import {Warehouse} from "@/types/api/master-data/warehouse"; +import { BaseMetadata } from '@/types/api/api-general'; +import { Product } from '@/types/api/master-data/product'; +import { Supplier } from '@/types/api/master-data/supplier'; +import { Warehouse } from '@/types/api/master-data/warehouse'; export type BaseMovement = { - id: number; - alasan_transfer: string; - tanggal_transfer: string; - warehouse_asal: Warehouse; - warehouse_tujuan: Warehouse; - product: Array<{ - product: Product; - qty_product: number; - }>; - ekspedisi: Array<{ - product_id: number; - qty: number; - supplier: Supplier; - plat_nomor: string; - no_surat_jalan: string; - dokumen: string; - biaya_ekspedisi: number; - nama_sopir: string; - }>; - name: string; + id: number; + alasan_transfer: string; + tanggal_transfer: string; + warehouse_asal: Warehouse; + warehouse_tujuan: Warehouse; + product: { + product: Product; + qty_product: number; + }[]; + ekspedisi: { + product_id: number; + qty: number; + supplier: Supplier; + plat_nomor: string; + no_surat_jalan: string; + dokumen: string; + biaya_ekspedisi: number; + nama_sopir: string; + }[]; }; export type Movement = BaseMetadata & BaseMovement; export type CreateMovementPayload = { - alasan: string; - warehouse_asal_id: number; - warehouse_tujuan_id: number; - product: Array<{ - product_id: number; - qty_product: number; - }>; - ekspedisi: Array<{ - product_id: number; - qty: number; - supplier_id: number; - plat_nomor: string; - no_surat_jalan: string; - dokumen: string; - biaya_ekspedisi: number; - nama_sopir: string; - }>; -} + alasan_transfer: string; + tanggal_transfer: string; + warehouse_asal_id: number; + warehouse_tujuan_id: number; + product: { + product_id: number; + qty_product: number; + }[]; + ekspedisi: { + product_id: number; + qty: number; + supplier_id: number; + plat_nomor: string; + no_surat_jalan: string; + dokumen: string | File; + biaya_ekspedisi: number; + nama_sopir: string; + }[]; +}; -export type UpdateMovementPayload = CreateMovementPayload; \ No newline at end of file +export type UpdateMovementPayload = CreateMovementPayload; From c17ffc6aff22bc2dd58e9aba7fe88e6e2b1d55dc Mon Sep 17 00:00:00 2001 From: rstubryan Date: Thu, 9 Oct 2025 14:34:25 +0700 Subject: [PATCH 05/55] feat(FE-62): add MovementEdit and MovementDetail components for inventory movement management --- .../inventory/movement/detail/edit/page.tsx | 48 +++++++++++++++++++ src/app/inventory/movement/detail/layout.tsx | 11 +++++ src/app/inventory/movement/detail/page.tsx | 48 +++++++++++++++++++ 3 files changed, 107 insertions(+) create mode 100644 src/app/inventory/movement/detail/edit/page.tsx create mode 100644 src/app/inventory/movement/detail/layout.tsx create mode 100644 src/app/inventory/movement/detail/page.tsx diff --git a/src/app/inventory/movement/detail/edit/page.tsx b/src/app/inventory/movement/detail/edit/page.tsx new file mode 100644 index 00000000..bde4ece1 --- /dev/null +++ b/src/app/inventory/movement/detail/edit/page.tsx @@ -0,0 +1,48 @@ +'use client'; + +import { useRouter, useSearchParams } from 'next/navigation'; +import useSWR from 'swr'; + +import MovementForm from '@/components/pages/inventory/movement/form/MovementForm'; +import { MovementApi } from '@/services/api/inventory'; +import { isResponseError, isResponseSuccess } from '@/lib/api-helper'; + +const MovementEdit = () => { + const router = useRouter(); + const searchParams = useSearchParams(); + + const movementId = searchParams.get('movementId'); + + const { data: movement, isLoading: isLoadingMovement } = useSWR( + movementId, + (id: number) => MovementApi.getSingle(id) + ); + + if (!movementId) { + router.back(); + + return ( +
+ +
+ ); + } + + if (!isLoadingMovement && (!movement || isResponseError(movement))) { + router.replace('/404'); + return; + } + + return ( +
+ {isLoadingMovement && ( + + )} + {!isLoadingMovement && isResponseSuccess(movement) && ( + + )} +
+ ); +}; + +export default MovementEdit; diff --git a/src/app/inventory/movement/detail/layout.tsx b/src/app/inventory/movement/detail/layout.tsx new file mode 100644 index 00000000..7220dfa1 --- /dev/null +++ b/src/app/inventory/movement/detail/layout.tsx @@ -0,0 +1,11 @@ +import SuspenseHelper from '@/components/helper/SuspenseHelper'; + +const Layout = ({ + children, +}: Readonly<{ + children: React.ReactNode; +}>) => { + return {children}; +}; + +export default Layout; diff --git a/src/app/inventory/movement/detail/page.tsx b/src/app/inventory/movement/detail/page.tsx new file mode 100644 index 00000000..5947cd1b --- /dev/null +++ b/src/app/inventory/movement/detail/page.tsx @@ -0,0 +1,48 @@ +'use client'; + +import { useRouter, useSearchParams } from 'next/navigation'; +import useSWR from 'swr'; + +import MovementForm from '@/components/pages/inventory/movement/form/MovementForm'; +import { MovementApi } from '@/services/api/inventory'; +import { isResponseError, isResponseSuccess } from '@/lib/api-helper'; + +const MovementDetail = () => { + const router = useRouter(); + const searchParams = useSearchParams(); + + const movementId = searchParams.get('movementId'); + + const { data: movement, isLoading: isLoadingMovement } = useSWR( + movementId, + (id: number) => MovementApi.getSingle(id) + ); + + if (!movementId) { + router.back(); + + return ( +
+ +
+ ); + } + + if (!isLoadingMovement && (!movement || isResponseError(movement))) { + router.replace('/404'); + return; + } + + return ( +
+ {isLoadingMovement && ( + + )} + {!isLoadingMovement && isResponseSuccess(movement) && ( + + )} +
+ ); +}; + +export default MovementDetail; From 7dbf8802284755a52318d7d9fdf7ada55965913c Mon Sep 17 00:00:00 2001 From: rstubryan Date: Fri, 10 Oct 2025 08:39:34 +0700 Subject: [PATCH 06/55] feat(FE-62): add FormActions and FormHeader components for form management --- src/components/helper/form/FormActions.tsx | 82 +++++++++++++++++++ src/components/helper/form/FormHeader.tsx | 24 ++++++ .../movement/form/useMovementFormHandlers.ts | 69 ++++++++++++++++ 3 files changed, 175 insertions(+) create mode 100644 src/components/helper/form/FormActions.tsx create mode 100644 src/components/helper/form/FormHeader.tsx create mode 100644 src/components/pages/inventory/movement/form/useMovementFormHandlers.ts diff --git a/src/components/helper/form/FormActions.tsx b/src/components/helper/form/FormActions.tsx new file mode 100644 index 00000000..25b586c3 --- /dev/null +++ b/src/components/helper/form/FormActions.tsx @@ -0,0 +1,82 @@ +import { Icon } from '@iconify/react'; +import { FormikContextType } from 'formik'; +import Button from '@/components/Button'; +import { cn } from '@/lib/helper'; + +interface FormActionsProps { + type: 'add' | 'edit' | 'detail'; + formik: FormikContextType; + editUrl?: string; + onDelete?: () => void; +} + +export const FormActions = ({ + type, + formik, + editUrl, + onDelete, +}: FormActionsProps) => { + return ( +
+ {type !== 'add' && onDelete && ( +
+ + {type !== 'edit' && editUrl && ( + + )} +
+ )} + {type !== 'detail' && ( +
+ + +
+ )} +
+ ); +}; diff --git a/src/components/helper/form/FormHeader.tsx b/src/components/helper/form/FormHeader.tsx new file mode 100644 index 00000000..ebc1d7ae --- /dev/null +++ b/src/components/helper/form/FormHeader.tsx @@ -0,0 +1,24 @@ +import Button from '@/components/Button'; +import { Icon } from '@iconify/react'; + +interface FormHeaderProps { + type: 'add' | 'edit' | 'detail'; + title: string; + backUrl: string; +} + +export const FormHeader = ({ type, title, backUrl }: FormHeaderProps) => { + return ( +
+ +

+ {type === 'add' && `Tambah ${title}`} + {type === 'edit' && `Edit ${title}`} + {type === 'detail' && `Detail ${title}`} +

+
+ ); +}; diff --git a/src/components/pages/inventory/movement/form/useMovementFormHandlers.ts b/src/components/pages/inventory/movement/form/useMovementFormHandlers.ts new file mode 100644 index 00000000..f6531bf4 --- /dev/null +++ b/src/components/pages/inventory/movement/form/useMovementFormHandlers.ts @@ -0,0 +1,69 @@ +import { useCallback, useState } from 'react'; +import { useRouter } from 'next/navigation'; +import { toast } from 'react-hot-toast'; +import { useModal } from '@/components/Modal'; +import { MovementApi } from '@/services/api/inventory'; +import { + CreateMovementPayload, + UpdateMovementPayload, +} from '@/types/api/inventory/movement'; +import { isResponseError } from '@/lib/api-helper'; + +export const useMovementFormHandlers = (initialValuesId?: number) => { + const router = useRouter(); + const deleteModal = useModal(); + const [movementFormErrorMessage, setMovementFormErrorMessage] = useState(''); + const [isDeleteLoading, setIsDeleteLoading] = useState(false); + + const createMovementHandler = useCallback( + async (payload: CreateMovementPayload) => { + const res = await MovementApi.create(payload); + if (isResponseError(res)) { + setMovementFormErrorMessage(res.message); + return; + } + toast.success(res?.message as string); + router.push('/inventory/movement'); + }, + [router] + ); + + const updateMovementHandler = useCallback( + async (movementId: number, payload: UpdateMovementPayload) => { + const res = await MovementApi.update(movementId, payload); + if (res?.status === 'error') { + setMovementFormErrorMessage(res.message); + return; + } + toast.success(res?.message as string); + router.refresh(); + router.push('/inventory/movement'); + }, + [router] + ); + + const deleteMovementClickHandler = useCallback(() => { + deleteModal.openModal(); + }, [deleteModal]); + + const confirmationModalDeleteClickHandler = useCallback(async () => { + if (!initialValuesId) return; + + setIsDeleteLoading(true); + await MovementApi.delete(initialValuesId); + deleteModal.closeModal(); + toast.success('Successfully delete Movement!'); + setIsDeleteLoading(false); + router.push('/inventory/movement'); + }, [deleteModal, initialValuesId, router]); + + return { + deleteModal, + movementFormErrorMessage, + isDeleteLoading, + createMovementHandler, + updateMovementHandler, + deleteMovementClickHandler, + confirmationModalDeleteClickHandler, + }; +}; From a9cdea73182518bdc81c03bce4825d5b4f2d63fb Mon Sep 17 00:00:00 2001 From: rstubryan Date: Fri, 10 Oct 2025 08:40:17 +0700 Subject: [PATCH 07/55] feat(FE-65): enhance MovementForm with initial values handling and refactor components --- .../movement/form/MovementForm.schema.ts | 41 ++++ .../inventory/movement/form/MovementForm.tsx | 219 +++--------------- 2 files changed, 78 insertions(+), 182 deletions(-) diff --git a/src/components/pages/inventory/movement/form/MovementForm.schema.ts b/src/components/pages/inventory/movement/form/MovementForm.schema.ts index 11c40fe5..7a34dfbd 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.schema.ts +++ b/src/components/pages/inventory/movement/form/MovementForm.schema.ts @@ -1,4 +1,5 @@ import * as Yup from 'yup'; +import { Movement } from '@/types/api/inventory/movement'; export const MovementFormSchema = Yup.object({ alasan_transfer: Yup.string().required('Alasan transfer wajib diisi!'), @@ -66,3 +67,43 @@ export const MovementFormSchema = Yup.object({ export const UpdateMovementFormSchema = MovementFormSchema; export type MovementFormValues = Yup.InferType; + +export const getMovementFormInitialValues = ( + initialValues?: Movement +): MovementFormValues => ({ + alasan_transfer: initialValues?.alasan_transfer ?? '', + tanggal_transfer: initialValues?.tanggal_transfer ?? '', + warehouse_asal: initialValues?.warehouse_asal + ? { + value: initialValues.warehouse_asal.id, + label: initialValues.warehouse_asal.name, + } + : null, + warehouse_asal_id: initialValues?.warehouse_asal?.id ?? 0, + warehouse_tujuan: initialValues?.warehouse_tujuan + ? { + value: initialValues.warehouse_tujuan.id, + label: initialValues.warehouse_tujuan.name, + } + : null, + warehouse_tujuan_id: initialValues?.warehouse_tujuan?.id ?? 0, + product: + initialValues?.product?.map((p) => ({ + product: { value: p.product.id, label: p.product.name }, + product_id: p.product.id, + qty_product: p.qty_product, + })) ?? [], + ekspedisi: + initialValues?.ekspedisi?.map((e) => ({ + product: { value: e.product_id, label: '' }, + product_id: e.product_id, + qty: e.qty, + supplier: { value: e.supplier.id, label: e.supplier.name }, + supplier_id: e.supplier.id, + plat_nomor: e.plat_nomor, + no_surat_jalan: e.no_surat_jalan, + dokumen: e.dokumen, + biaya_ekspedisi: e.biaya_ekspedisi, + nama_sopir: e.nama_sopir, + })) ?? [], +}); diff --git a/src/components/pages/inventory/movement/form/MovementForm.tsx b/src/components/pages/inventory/movement/form/MovementForm.tsx index 0105768d..839b83fe 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.tsx +++ b/src/components/pages/inventory/movement/form/MovementForm.tsx @@ -1,36 +1,33 @@ 'use client'; -import { useCallback, useEffect, useMemo, useState } from 'react'; -import { useRouter } from 'next/navigation'; +import { useEffect, useMemo, useState } from 'react'; import { FieldArray, FormikProvider, useFormik } from 'formik'; -import { toast } from 'react-hot-toast'; import useSWR from 'swr'; import { Icon } from '@iconify/react'; import Button from '@/components/Button'; import TextInput from '@/components/input/TextInput'; import SelectInput, { OptionType } from '@/components/input/SelectInput'; -import { useModal } from '@/components/Modal'; import ConfirmationModal from '@/components/modal/ConfirmationModal'; - +import { FormHeader } from '@/components/helper/form/FormHeader'; +import { FormActions } from '@/components/helper/form/FormActions'; +import { + CreateMovementPayload, + Movement, +} from '@/types/api/inventory/movement'; +import { isResponseSuccess } from '@/lib/api-helper'; import { MovementFormSchema, MovementFormValues, UpdateMovementFormSchema, + getMovementFormInitialValues, } from '@/components/pages/inventory/movement/form/MovementForm.schema'; -import { isResponseError, isResponseSuccess } from '@/lib/api-helper'; -import { - Movement, - CreateMovementPayload, - UpdateMovementPayload, -} from '@/types/api/inventory/movement'; +import { useMovementFormHandlers } from './useMovementFormHandlers'; import { ProductApi, - WarehouseApi, SupplierApi, + WarehouseApi, } from '@/services/api/master-data'; -import { MovementApi } from '@/services/api/inventory'; -import { cn } from '@/lib/helper'; interface MovementFormProps { type?: 'add' | 'edit' | 'detail'; @@ -38,77 +35,20 @@ interface MovementFormProps { } const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { - const router = useRouter(); - const deleteModal = useModal(); + const [, setMovementFormErrorMessage] = useState(''); - const [movementFormErrorMessage, setMovementFormErrorMessage] = useState(''); - const [isDeleteLoading, setIsDeleteLoading] = useState(false); - - const createMovementHandler = useCallback( - async (payload: CreateMovementPayload) => { - const res = await MovementApi.create(payload); - if (isResponseError(res)) { - setMovementFormErrorMessage(res.message); - return; - } - toast.success(res?.message as string); - router.push('/inventory/movement'); - }, - [router] - ); - - const updateMovementHandler = useCallback( - async (movementId: number, payload: UpdateMovementPayload) => { - const res = await MovementApi.update(movementId, payload); - if (res?.status === 'error') { - setMovementFormErrorMessage(res.message); - return; - } - toast.success(res?.message as string); - router.refresh(); - router.push('/inventory/movement'); - }, - [router] - ); + const { + deleteModal, + movementFormErrorMessage, + isDeleteLoading, + createMovementHandler, + updateMovementHandler, + deleteMovementClickHandler, + confirmationModalDeleteClickHandler, + } = useMovementFormHandlers(initialValues?.id); const formikInitialValues = useMemo( - () => ({ - alasan_transfer: initialValues?.alasan_transfer ?? '', - tanggal_transfer: initialValues?.tanggal_transfer ?? '', - warehouse_asal: initialValues?.warehouse_asal - ? { - value: initialValues.warehouse_asal.id, - label: initialValues.warehouse_asal.name, - } - : null, - warehouse_asal_id: initialValues?.warehouse_asal?.id ?? 0, - warehouse_tujuan: initialValues?.warehouse_tujuan - ? { - value: initialValues.warehouse_tujuan.id, - label: initialValues.warehouse_tujuan.name, - } - : null, - warehouse_tujuan_id: initialValues?.warehouse_tujuan?.id ?? 0, - product: - initialValues?.product?.map((p) => ({ - product: { value: p.product.id, label: p.product.name }, - product_id: p.product.id, - qty_product: p.qty_product, - })) ?? [], - ekspedisi: - initialValues?.ekspedisi?.map((e) => ({ - product: { value: e.product_id, label: '' }, // Need to fetch product details - product_id: e.product_id, - qty: e.qty, - supplier: { value: e.supplier.id, label: e.supplier.name }, - supplier_id: e.supplier.id, - plat_nomor: e.plat_nomor, - no_surat_jalan: e.no_surat_jalan, - dokumen: e.dokumen, - biaya_ekspedisi: e.biaya_ekspedisi, - nama_sopir: e.nama_sopir, - })) ?? [], - }), + () => getMovementFormInitialValues(initialValues), [initialValues] ); @@ -185,19 +125,6 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { ? suppliers?.data.map((s) => ({ value: s.id, label: s.name })) : []; - const deleteMovementClickHandler = () => { - deleteModal.openModal(); - }; - - const confirmationModalDeleteClickHandler = async () => { - setIsDeleteLoading(true); - await MovementApi.delete(initialValues?.id as number); - deleteModal.closeModal(); - toast.success('Successfully delete Movement!'); - setIsDeleteLoading(false); - router.push('/inventory/movement'); - }; - const { setValues: formikSetValues } = formik; useEffect(() => { @@ -207,22 +134,11 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { return ( <>
-
- -

- {type === 'add' && 'Tambah Movement'} - {type === 'edit' && 'Edit Movement'} - {type === 'detail' && 'Detail Movement'} -

-
- +
{ {({ push, remove }) => ( <> - {typeof formik.errors.product === 'string' && ( -
- {formik.errors.product} -
- )} @@ -433,11 +344,6 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { {({ push, remove }) => ( <> - {typeof formik.errors.ekspedisi === 'string' && ( -
- {formik.errors.ekspedisi} -
- )}
@@ -652,67 +558,16 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { {/* Action buttons */} -
- {type !== 'add' && ( -
- - {type !== 'edit' && ( - - )} -
- )} - {type !== 'detail' && ( -
- - -
- )} -
+ + type={type} + formik={formik} + editUrl={ + initialValues + ? `/inventory/movement/detail/edit/?movementId=${initialValues.id}` + : undefined + } + onDelete={deleteMovementClickHandler} + /> {movementFormErrorMessage && (
From 095190d75783841e574f742efc8461c2e16838eb Mon Sep 17 00:00:00 2001 From: rstubryan Date: Fri, 10 Oct 2025 10:19:56 +0700 Subject: [PATCH 08/55] refactor(FE-62,65): refactor MovementForm schema and component for improved product and ekspedisi handling --- .../movement/form/MovementForm.schema.ts | 110 +- .../inventory/movement/form/MovementForm.tsx | 950 ++++++++++-------- 2 files changed, 577 insertions(+), 483 deletions(-) diff --git a/src/components/pages/inventory/movement/form/MovementForm.schema.ts b/src/components/pages/inventory/movement/form/MovementForm.schema.ts index 7a34dfbd..230244be 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.schema.ts +++ b/src/components/pages/inventory/movement/form/MovementForm.schema.ts @@ -1,6 +1,72 @@ import * as Yup from 'yup'; import { Movement } from '@/types/api/inventory/movement'; +export type ProductSchema = { + product: { + value: number; + label: string; + } | null; + product_id: number; + qty_product: number; +}; + +export type EkspedisiSchema = { + product: { + value: number; + label: string; + } | null; + product_id: number; + qty: number; + supplier: { + value: number; + label: string; + } | null; + supplier_id: number; + plat_nomor: string; + no_surat_jalan: string; + dokumen: string | File; + biaya_ekspedisi: number; + nama_sopir: string; +}; + +// Define schemas for nested objects +const ProductObjectSchema: Yup.ObjectSchema = Yup.object({ + product: Yup.object({ + value: Yup.number().min(1).required(), + label: Yup.string().required(), + }).nullable(), + product_id: Yup.number().required('Produk wajib diisi!'), + qty_product: Yup.number() + .required('Qty wajib diisi!') + .min(1, 'Qty minimal 1!') + .typeError('Qty harus berupa angka!'), +}); + +const EkspedisiObjectSchema: Yup.ObjectSchema = Yup.object({ + product: Yup.object({ + value: Yup.number().min(1).required(), + label: Yup.string().required(), + }).nullable(), + product_id: Yup.number().required('Produk wajib diisi!'), + qty: Yup.number() + .required('Qty wajib diisi!') + .min(1, 'Qty minimal 1!') + .typeError('Qty harus berupa angka!'), + supplier: Yup.object({ + value: Yup.number().min(1).required(), + label: Yup.string().required(), + }).nullable(), + supplier_id: Yup.number().required('Supplier wajib diisi!'), + plat_nomor: Yup.string().required('Plat nomor wajib diisi!'), + no_surat_jalan: Yup.string().required('No surat jalan wajib diisi!'), + dokumen: Yup.mixed().required('Dokumen wajib diisi!'), + biaya_ekspedisi: Yup.number() + .required('Biaya ekspedisi wajib diisi!') + .min(0, 'Biaya minimal 0!') + .typeError('Biaya harus berupa angka!'), + nama_sopir: Yup.string().required('Nama sopir wajib diisi!'), +}); + export const MovementFormSchema = Yup.object({ alasan_transfer: Yup.string().required('Alasan transfer wajib diisi!'), tanggal_transfer: Yup.string().required('Tanggal transfer wajib diisi!'), @@ -19,49 +85,9 @@ export const MovementFormSchema = Yup.object({ .required('Gudang tujuan wajib diisi!') .typeError('Gudang tujuan wajib diisi!'), product: Yup.array() - .of( - Yup.object({ - product: Yup.object({ - value: Yup.number().min(1).required(), - label: Yup.string().required(), - }).nullable(), - product_id: Yup.number().required('Produk wajib diisi!'), - qty_product: Yup.number() - .required('Qty wajib diisi!') - .min(1, 'Qty minimal 1!') - .typeError('Qty harus berupa angka!'), - }) - ) + .of(ProductObjectSchema) .min(1, 'Minimal harus ada 1 produk!'), - ekspedisi: Yup.array() - .of( - Yup.object({ - product: Yup.object({ - value: Yup.number().min(1).required(), - label: Yup.string().required(), - }).nullable(), - product_id: Yup.number().required('Produk wajib diisi!'), - qty: Yup.number() - .required('Qty wajib diisi!') - .min(1, 'Qty minimal 1!') - .typeError('Qty harus berupa angka!'), - supplier: Yup.object({ - value: Yup.number().min(1).required(), - label: Yup.string().required(), - }).nullable(), - supplier_id: Yup.number().required('Supplier wajib diisi!'), - plat_nomor: Yup.string().required('Plat nomor wajib diisi!'), - no_surat_jalan: Yup.string().required('No surat jalan wajib diisi!'), - dokumen: Yup.mixed().required('Dokumen wajib diisi!'), - biaya_ekspedisi: Yup.number() - .required('Biaya ekspedisi wajib diisi!') - .min(0, 'Biaya minimal 0!') - .typeError('Biaya harus berupa angka!'), - nama_sopir: Yup.string().required('Nama sopir wajib diisi!'), - }) - ) - .optional() - .default([]), + ekspedisi: Yup.array().of(EkspedisiObjectSchema).optional().default([]), }); export const UpdateMovementFormSchema = MovementFormSchema; diff --git a/src/components/pages/inventory/movement/form/MovementForm.tsx b/src/components/pages/inventory/movement/form/MovementForm.tsx index 839b83fe..bca69015 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.tsx +++ b/src/components/pages/inventory/movement/form/MovementForm.tsx @@ -1,7 +1,7 @@ 'use client'; import { useEffect, useMemo, useState } from 'react'; -import { FieldArray, FormikProvider, useFormik } from 'formik'; +import { useFormik } from 'formik'; import useSWR from 'swr'; import { Icon } from '@iconify/react'; @@ -21,6 +21,8 @@ import { MovementFormValues, UpdateMovementFormSchema, getMovementFormInitialValues, + ProductSchema, + EkspedisiSchema, } from '@/components/pages/inventory/movement/form/MovementForm.schema'; import { useMovementFormHandlers } from './useMovementFormHandlers'; import { @@ -91,6 +93,72 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { }, }); + const addProduct = () => { + const newProducts = [ + ...(formik.values.product || []), + { + product: null, + product_id: 0, + qty_product: 0, + }, + ]; + formik.setFieldValue('product', newProducts); + }; + + const removeProduct = (index: number) => { + const newProducts = formik.values.product?.filter( + (_, idx) => idx !== index + ); + formik.setFieldValue('product', newProducts); + }; + + const addEkspedisi = () => { + const newEkspedisi = [ + ...(formik.values.ekspedisi || []), + { + product: null, + product_id: 0, + qty: 0, + supplier: null, + supplier_id: 0, + plat_nomor: '', + no_surat_jalan: '', + dokumen: '', + biaya_ekspedisi: 0, + nama_sopir: '', + }, + ]; + formik.setFieldValue('ekspedisi', newEkspedisi); + }; + + const removeEkspedisi = (index: number) => { + const newEkspedisi = formik.values.ekspedisi?.filter( + (_, idx) => idx !== index + ); + formik.setFieldValue('ekspedisi', newEkspedisi); + }; + + const isRepeaterInputError = ( + arrayName: T, + column: T extends 'product' ? keyof ProductSchema : keyof EkspedisiSchema, + idx: number + ) => { + if ( + !formik.touched[arrayName] || + !Array.isArray(formik.touched[arrayName]) + ) { + return false; + } + + const touchedField = formik.touched[arrayName]?.[idx]?.[column as string]; + const errorField = formik.errors[arrayName]?.[idx] as Record< + string, + unknown + >; + + return touchedField && Boolean(errorField?.[column as string]); + }; + // Warehouse selection const [warehouseSelectInputValue, setWarehouseSelectInputValue] = useState(''); @@ -139,448 +207,448 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { title='Movement' backUrl='/inventory/movement' /> - - - {/* Top card - Movement details */} -
-
-
- - -
-
-
- - {/* Warehouse cards */} -
-
-
- { - formik.setFieldValue('warehouse_asal', val); - formik.setFieldValue( - 'warehouse_asal_id', - (val as OptionType)?.value - ); - }} - options={warehouseOptions} - onInputChange={setWarehouseSelectInputValue} - isLoading={isLoadingWarehouses} - isError={ - formik.touched.warehouse_asal_id && - Boolean(formik.errors.warehouse_asal_id) - } - errorMessage={formik.errors.warehouse_asal_id as string} - isDisabled={type === 'detail'} - isClearable - /> -
-
- -
-
- { - formik.setFieldValue('warehouse_tujuan', val); - formik.setFieldValue( - 'warehouse_tujuan_id', - (val as OptionType)?.value - ); - }} - options={warehouseOptions} - onInputChange={setWarehouseSelectInputValue} - isLoading={isLoadingWarehouses} - isError={ - formik.touched.warehouse_tujuan_id && - Boolean(formik.errors.warehouse_tujuan_id) - } - errorMessage={formik.errors.warehouse_tujuan_id as string} - isDisabled={type === 'detail'} - isClearable - /> -
-
-
- - {/* Products table */} -
-
-

Produk

- - {({ push, remove }) => ( - <> -
- - - - - - - - - {formik.values.product?.map((_, index) => ( - - - - - - )) ?? []} - -
ProdukQtyAksi
- { - formik.setFieldValue( - `product.${index}.product`, - val - ); - formik.setFieldValue( - `product.${index}.product_id`, - (val as OptionType)?.value - ); - }} - options={productOptions} - onInputChange={setProductSelectInputValue} - isLoading={isLoadingProducts} - isDisabled={type === 'detail'} - isClearable - /> - - - formik.setFieldValue( - `product.${index}.qty_product`, - e.target.value - ) - } - readOnly={type === 'detail'} - /> - - {type !== 'detail' && ( - - )} -
- {type !== 'detail' && ( - - )} - - )} -
- - - - {/* Ekspedisi table */} -
-
-

Ekspedisi

- - {({ push, remove }) => ( - <> - - - - - - - - - - - - - - - - {formik.values.ekspedisi?.map((ekspedisi, index) => ( - - - - - - - - - - - - )) ?? []} - -
ProdukQtySupplierPlat NomorNo Surat JalanDokumenBiaya EkspedisiNama SopirAksi
- { - formik.setFieldValue( - `ekspedisi.${index}.product`, - val - ); - formik.setFieldValue( - `ekspedisi.${index}.product_id`, - (val as OptionType)?.value - ); - }} - options={productOptions} - onInputChange={setProductSelectInputValue} - isLoading={isLoadingProducts} - isDisabled={type === 'detail'} - isClearable - /> - - - formik.setFieldValue( - `ekspedisi.${index}.qty`, - e.target.value - ) - } - readOnly={type === 'detail'} - /> - - { - formik.setFieldValue( - `ekspedisi.${index}.supplier`, - val - ); - formik.setFieldValue( - `ekspedisi.${index}.supplier_id`, - (val as OptionType)?.value - ); - }} - options={supplierOptions} - onInputChange={setSupplierSelectInputValue} - isLoading={isLoadingSuppliers} - isDisabled={type === 'detail'} - isClearable - /> - - - formik.setFieldValue( - `ekspedisi.${index}.plat_nomor`, - e.target.value - ) - } - readOnly={type === 'detail'} - /> - - - formik.setFieldValue( - `ekspedisi.${index}.no_surat_jalan`, - e.target.value - ) - } - readOnly={type === 'detail'} - /> - - { - const file = e.target.files?.[0]; - if (file) { - formik.setFieldValue( - `ekspedisi.${index}.dokumen`, - file - ); - } - }} - readOnly={type === 'detail'} - /> - - - formik.setFieldValue( - `ekspedisi.${index}.biaya_ekspedisi`, - e.target.value - ) - } - readOnly={type === 'detail'} - /> - - - formik.setFieldValue( - `ekspedisi.${index}.nama_sopir`, - e.target.value - ) - } - readOnly={type === 'detail'} - /> - - {type !== 'detail' && ( - - )} -
- {type !== 'detail' && ( - - )} - - )} -
-
-
- - {/* Action buttons */} - - type={type} - formik={formik} - editUrl={ - initialValues - ? `/inventory/movement/detail/edit/?movementId=${initialValues.id}` - : undefined - } - onDelete={deleteMovementClickHandler} - /> - - {movementFormErrorMessage && ( -
- + {/* Top card - Movement details */} +
+
+
+ + - {movementFormErrorMessage}
- )} - - +
+
+ + {/* Warehouse cards */} +
+
+
+ { + formik.setFieldValue('warehouse_asal', val); + formik.setFieldValue( + 'warehouse_asal_id', + (val as OptionType)?.value + ); + }} + options={warehouseOptions} + onInputChange={setWarehouseSelectInputValue} + isLoading={isLoadingWarehouses} + isError={ + formik.touched.warehouse_asal_id && + Boolean(formik.errors.warehouse_asal_id) + } + errorMessage={formik.errors.warehouse_asal_id as string} + isDisabled={type === 'detail'} + isClearable + /> +
+
+ +
+
+ { + formik.setFieldValue('warehouse_tujuan', val); + formik.setFieldValue( + 'warehouse_tujuan_id', + (val as OptionType)?.value + ); + }} + options={warehouseOptions} + onInputChange={setWarehouseSelectInputValue} + isLoading={isLoadingWarehouses} + isError={ + formik.touched.warehouse_tujuan_id && + Boolean(formik.errors.warehouse_tujuan_id) + } + errorMessage={formik.errors.warehouse_tujuan_id as string} + isDisabled={type === 'detail'} + isClearable + /> +
+
+
+ + {/* Products table */} +
+
+

Produk

+
+ + + + + + {type !== 'detail' && } + + + + {formik.values.product?.map((product, idx) => ( + + + + {type !== 'detail' && ( + + )} + + ))} + +
ProdukQtyAksi
+ { + formik.setFieldValue( + `product.${idx}.product`, + val + ); + formik.setFieldValue( + `product.${idx}.product_id`, + (val as OptionType)?.value + ); + }} + options={productOptions} + onInputChange={setProductSelectInputValue} + isLoading={isLoadingProducts} + isDisabled={type === 'detail'} + isClearable + isError={isRepeaterInputError( + 'product', + 'product', + idx + )} + /> + + + + +
+
+ {type !== 'detail' && ( + + )} +
+
+ + {/* Ekspedisi table */} +
+
+

Ekspedisi

+
+ + + + + + + + + + + + {type !== 'detail' && } + + + + {formik.values.ekspedisi?.map((ekspedisi, idx) => ( + + + + + + + + + + {type !== 'detail' && ( + + )} + + ))} + +
ProdukQtySupplierPlat NomorNo Surat JalanDokumenBiaya EkspedisiNama SopirAksi
+ { + formik.setFieldValue( + `ekspedisi.${idx}.product`, + val + ); + formik.setFieldValue( + `ekspedisi.${idx}.product_id`, + (val as OptionType)?.value + ); + }} + options={productOptions} + onInputChange={setProductSelectInputValue} + isLoading={isLoadingProducts} + isDisabled={type === 'detail'} + isClearable + isError={isRepeaterInputError( + 'ekspedisi', + 'product', + idx + )} + /> + + + + { + formik.setFieldValue( + `ekspedisi.${idx}.supplier`, + val + ); + formik.setFieldValue( + `ekspedisi.${idx}.supplier_id`, + (val as OptionType)?.value + ); + }} + options={supplierOptions} + onInputChange={setSupplierSelectInputValue} + isLoading={isLoadingSuppliers} + isDisabled={type === 'detail'} + isClearable + isError={isRepeaterInputError( + 'ekspedisi', + 'supplier', + idx + )} + /> + + + + + + { + const file = e.target.files?.[0]; + if (file) { + formik.setFieldValue( + `ekspedisi.${idx}.dokumen`, + file + ); + } + }} + isError={isRepeaterInputError( + 'ekspedisi', + 'dokumen', + idx + )} + readOnly={type === 'detail'} + className={{ + wrapper: 'w-full min-w-24', + }} + /> + + + + + + +
+
+ {type !== 'detail' && ( + + )} +
+
+ + {/* Action buttons */} + + type={type} + formik={formik} + editUrl={ + initialValues + ? `/inventory/movement/detail/edit/?movementId=${initialValues.id}` + : undefined + } + onDelete={deleteMovementClickHandler} + /> + + {movementFormErrorMessage && ( +
+ + {movementFormErrorMessage} +
+ )} +
{type !== 'add' && ( From 57831646d950eeb5b7ddb0b9561083673c93bb29 Mon Sep 17 00:00:00 2001 From: rstubryan Date: Fri, 10 Oct 2025 11:14:59 +0700 Subject: [PATCH 09/55] refactor(FE-62): optimize product and ekspedisi removal logic in MovementForm --- .../inventory/movement/form/MovementForm.tsx | 53 +++++++++++++------ 1 file changed, 37 insertions(+), 16 deletions(-) diff --git a/src/components/pages/inventory/movement/form/MovementForm.tsx b/src/components/pages/inventory/movement/form/MovementForm.tsx index bca69015..d9e1102e 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.tsx +++ b/src/components/pages/inventory/movement/form/MovementForm.tsx @@ -1,6 +1,6 @@ 'use client'; -import { useEffect, useMemo, useState } from 'react'; +import { useCallback, useEffect, useMemo, useState } from 'react'; import { useFormik } from 'formik'; import useSWR from 'swr'; @@ -105,12 +105,20 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { formik.setFieldValue('product', newProducts); }; - const removeProduct = (index: number) => { - const newProducts = formik.values.product?.filter( - (_, idx) => idx !== index - ); - formik.setFieldValue('product', newProducts); - }; + const removeProduct = useCallback( + (i: number) => { + const updatedProducts = + formik.values.product?.reduce((acc: ProductSchema[], item, index) => { + if (index !== i) { + acc.push(item); + } + return acc; + }, []) ?? []; + + formik.setFieldValue('product', updatedProducts); + }, + [formik] + ); const addEkspedisi = () => { const newEkspedisi = [ @@ -131,12 +139,23 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { formik.setFieldValue('ekspedisi', newEkspedisi); }; - const removeEkspedisi = (index: number) => { - const newEkspedisi = formik.values.ekspedisi?.filter( - (_, idx) => idx !== index - ); - formik.setFieldValue('ekspedisi', newEkspedisi); - }; + const removeEkspedisi = useCallback( + (i: number) => { + const updatedEkspedisi = + formik.values.ekspedisi?.reduce( + (acc: EkspedisiSchema[], item, index) => { + if (index !== i) { + acc.push(item); + } + return acc; + }, + [] + ) ?? []; + + formik.setFieldValue('ekspedisi', updatedEkspedisi); + }, + [formik] + ); const isRepeaterInputError = ( arrayName: T, @@ -201,7 +220,7 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { return ( <> -
+
{ {formik.values.product?.map((product, idx) => ( - + { {formik.values.ekspedisi?.map((ekspedisi, idx) => ( - + Date: Fri, 10 Oct 2025 13:14:39 +0700 Subject: [PATCH 10/55] feat(FE-62): implement bulk removal functionality for selected products and ekspedisi in MovementForm --- .../inventory/movement/form/MovementForm.tsx | 180 ++++++++++++++++-- 1 file changed, 162 insertions(+), 18 deletions(-) diff --git a/src/components/pages/inventory/movement/form/MovementForm.tsx b/src/components/pages/inventory/movement/form/MovementForm.tsx index d9e1102e..14c6fc45 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.tsx +++ b/src/components/pages/inventory/movement/form/MovementForm.tsx @@ -38,6 +38,8 @@ interface MovementFormProps { const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { const [, setMovementFormErrorMessage] = useState(''); + const [selectedProducts, setSelectedProducts] = useState([]); + const [selectedEkspedisi, setSelectedEkspedisi] = useState([]); const { deleteModal, @@ -120,6 +122,15 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { [formik] ); + const bulkRemoveProduct = useCallback(() => { + const updatedProducts = + formik.values.product?.filter( + (_, idx) => !selectedProducts.includes(idx) + ) ?? []; + formik.setFieldValue('product', updatedProducts); + setSelectedProducts([]); + }, [formik, selectedProducts]); + const addEkspedisi = () => { const newEkspedisi = [ ...(formik.values.ekspedisi || []), @@ -157,6 +168,15 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { [formik] ); + const bulkRemoveEkspedisi = useCallback(() => { + const updatedEkspedisi = + formik.values.ekspedisi?.filter( + (_, idx) => !selectedEkspedisi.includes(idx) + ) ?? []; + formik.setFieldValue('ekspedisi', updatedEkspedisi); + setSelectedEkspedisi([]); + }, [formik, selectedEkspedisi]); + const isRepeaterInputError = ( arrayName: T, column: T extends 'product' ? keyof ProductSchema : keyof EkspedisiSchema, @@ -333,6 +353,29 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { + {type !== 'detail' && ( + + )} {type !== 'detail' && } @@ -341,6 +384,27 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { {formik.values.product?.map((product, idx) => ( + {type !== 'detail' && ( + + )}
+ 0 + } + onChange={(e) => { + if (e.target.checked) { + setSelectedProducts( + formik.values.product?.map((_, idx) => idx) ?? + [] + ); + } else { + setSelectedProducts([]); + } + }} + /> + Produk QtyAksi
+ { + if (e.target.checked) { + setSelectedProducts([ + ...selectedProducts, + idx, + ]); + } else { + setSelectedProducts( + selectedProducts.filter((i) => i !== idx) + ); + } + }} + /> + {
{type !== 'detail' && ( - +
+ {selectedProducts.length > 0 && ( + + )} + +
)} @@ -428,6 +509,30 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { + {type !== 'detail' && ( + + )} @@ -444,6 +549,27 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { + {type !== 'detail' && ( + + )}
+ 0 + } + onChange={(e) => { + if (e.target.checked) { + setSelectedEkspedisi( + formik.values.ekspedisi?.map( + (_, idx) => idx + ) ?? [] + ); + } else { + setSelectedEkspedisi([]); + } + }} + /> + Produk Qty Supplier
+ { + if (e.target.checked) { + setSelectedEkspedisi([ + ...selectedEkspedisi, + idx, + ]); + } else { + setSelectedEkspedisi( + selectedEkspedisi.filter((i) => i !== idx) + ); + } + }} + /> + {
{type !== 'detail' && ( - +
+ {selectedEkspedisi.length > 0 && ( + + )} + +
)} From a1dc13ceb42e9543ad335a7041f630f4ce5df19a Mon Sep 17 00:00:00 2001 From: rstubryan Date: Fri, 10 Oct 2025 13:36:22 +0700 Subject: [PATCH 11/55] feat(FE-62): enhance MovementForm with area and location display for warehouse selection --- .../inventory/movement/form/MovementForm.tsx | 90 +++++++++++++++++-- 1 file changed, 82 insertions(+), 8 deletions(-) diff --git a/src/components/pages/inventory/movement/form/MovementForm.tsx b/src/components/pages/inventory/movement/form/MovementForm.tsx index 14c6fc45..22e39642 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.tsx +++ b/src/components/pages/inventory/movement/form/MovementForm.tsx @@ -198,6 +198,11 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { return touchedField && Boolean(errorField?.[column as string]); }; + interface WarehouseOptionType extends OptionType { + area?: string; + location?: string; + } + // Warehouse selection const [warehouseSelectInputValue, setWarehouseSelectInputValue] = useState(''); @@ -207,7 +212,15 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { WarehouseApi.getAllFetcher ); const warehouseOptions = isResponseSuccess(warehouses) - ? warehouses?.data.map((w) => ({ value: w.id, label: w.name })) + ? warehouses?.data.map((w) => ({ + value: w.id, + label: w.name, + area: w.area?.name, + location: + 'type' in w && (w.type === 'LOKASI' || w.type === 'KANDANG') + ? w.location?.name + : undefined, + })) : []; // Product selection @@ -291,16 +304,17 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { {/* Warehouse cards */}
-
+
+

Gudang Asal

{ formik.setFieldValue('warehouse_asal', val); formik.setFieldValue( 'warehouse_asal_id', - (val as OptionType)?.value + (val as WarehouseOptionType)?.value ); }} options={warehouseOptions} @@ -314,20 +328,51 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { isDisabled={type === 'detail'} isClearable /> + + {/* Area and Location Info */} +
+ + +
-
+
+

Gudang Tujuan

{ formik.setFieldValue('warehouse_tujuan', val); formik.setFieldValue( 'warehouse_tujuan_id', - (val as OptionType)?.value + (val as WarehouseOptionType)?.value ); }} options={warehouseOptions} @@ -341,10 +386,39 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { isDisabled={type === 'detail'} isClearable /> + + {/* Area and Location Info */} +
+ + +
- {/* Products table */}
From 757893c75789b09ce1d422771d2b71c53885ca48 Mon Sep 17 00:00:00 2001 From: rstubryan Date: Fri, 10 Oct 2025 13:43:30 +0700 Subject: [PATCH 12/55] feat(FE-62): add quantity validation for ekspedisi in MovementForm and filter product options --- .../inventory/movement/form/MovementForm.tsx | 49 +++++++++++++++++-- 1 file changed, 45 insertions(+), 4 deletions(-) diff --git a/src/components/pages/inventory/movement/form/MovementForm.tsx b/src/components/pages/inventory/movement/form/MovementForm.tsx index 22e39642..a2dd43c8 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.tsx +++ b/src/components/pages/inventory/movement/form/MovementForm.tsx @@ -30,6 +30,7 @@ import { SupplierApi, WarehouseApi, } from '@/services/api/master-data'; +import { toast } from 'react-hot-toast'; interface MovementFormProps { type?: 'add' | 'edit' | 'detail'; @@ -251,6 +252,37 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { formikSetValues(formikInitialValues); }, [formikSetValues, formikInitialValues]); + const getFilteredProductOptions = useCallback(() => { + return ( + formik.values.product + ?.filter((p) => p.product) + .map((p) => ({ + value: p.product_id, + label: (p.product as OptionType)?.label, + })) ?? [] + ); + }, [formik.values.product]); + + const validateEkspedisiQty = (ekspedisiIdx: number, qty: number) => { + const productId = formik.values.ekspedisi?.[ekspedisiIdx]?.product_id; + if (!productId) return true; + + const relatedProduct = formik.values.product?.find( + (p) => p.product_id === productId + ); + if (!relatedProduct) return true; + + const totalQtyUsed = + formik.values.ekspedisi?.reduce((total, eks, i) => { + if (eks.product_id === productId && i !== ekspedisiIdx) { + return total + (Number(eks.qty) || 0); + } + return total; + }, 0) || 0; + + return totalQtyUsed + qty <= Number(relatedProduct.qty_product); + }; + return ( <>
@@ -419,6 +451,7 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => {
+ {/* Products table */}
@@ -657,10 +690,9 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { `ekspedisi.${idx}.product_id`, (val as OptionType)?.value ); + formik.setFieldValue(`ekspedisi.${idx}.qty`, ''); }} - options={productOptions} - onInputChange={setProductSelectInputValue} - isLoading={isLoadingProducts} + options={getFilteredProductOptions()} isDisabled={type === 'detail'} isClearable isError={isRepeaterInputError( @@ -676,7 +708,16 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { type='number' name={`ekspedisi.${idx}.qty`} value={ekspedisi.qty ?? ''} - onChange={formik.handleChange} + onChange={(e) => { + const newQty = Number(e.target.value); + if (validateEkspedisiQty(idx, newQty)) { + formik.handleChange(e); + } else { + toast.error( + 'Quantity exceeds available product quantity' + ); + } + }} onBlur={formik.handleBlur} isError={isRepeaterInputError( 'ekspedisi', From 478f52c94b5b6068f0a71a7ff5714fe99589d289 Mon Sep 17 00:00:00 2001 From: rstubryan Date: Sat, 11 Oct 2025 08:33:48 +0700 Subject: [PATCH 13/55] feat(FE-62,65): add biaya_ekspedisi_per_item field and calculation in MovementForm --- .../movement/form/MovementForm.schema.ts | 9 +++- .../inventory/movement/form/MovementForm.tsx | 44 ++++++++++++++++++- src/types/api/inventory/movement.d.ts | 1 + 3 files changed, 52 insertions(+), 2 deletions(-) diff --git a/src/components/pages/inventory/movement/form/MovementForm.schema.ts b/src/components/pages/inventory/movement/form/MovementForm.schema.ts index 230244be..0bc0397c 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.schema.ts +++ b/src/components/pages/inventory/movement/form/MovementForm.schema.ts @@ -26,10 +26,10 @@ export type EkspedisiSchema = { no_surat_jalan: string; dokumen: string | File; biaya_ekspedisi: number; + biaya_ekspedisi_per_item?: number | undefined; nama_sopir: string; }; -// Define schemas for nested objects const ProductObjectSchema: Yup.ObjectSchema = Yup.object({ product: Yup.object({ value: Yup.number().min(1).required(), @@ -64,6 +64,12 @@ const EkspedisiObjectSchema: Yup.ObjectSchema = Yup.object({ .required('Biaya ekspedisi wajib diisi!') .min(0, 'Biaya minimal 0!') .typeError('Biaya harus berupa angka!'), + biaya_ekspedisi_per_item: Yup.number() + .transform((value) => (isNaN(value) ? undefined : value)) + .min(0, 'Biaya per item minimal 0!') + .typeError('Biaya per item harus berupa angka!') + .optional() + .default(undefined), nama_sopir: Yup.string().required('Nama sopir wajib diisi!'), }); @@ -130,6 +136,7 @@ export const getMovementFormInitialValues = ( no_surat_jalan: e.no_surat_jalan, dokumen: e.dokumen, biaya_ekspedisi: e.biaya_ekspedisi, + biaya_ekspedisi_per_item: e.biaya_ekspedisi, nama_sopir: e.nama_sopir, })) ?? [], }); diff --git a/src/components/pages/inventory/movement/form/MovementForm.tsx b/src/components/pages/inventory/movement/form/MovementForm.tsx index a2dd43c8..db432a4d 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.tsx +++ b/src/components/pages/inventory/movement/form/MovementForm.tsx @@ -81,6 +81,9 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { dokumen: e.dokumen instanceof File ? e.dokumen : (e.dokumen as string), biaya_ekspedisi: e.biaya_ekspedisi, + biaya_ekspedisi_per_item: e.qty + ? e.biaya_ekspedisi / e.qty + : e.biaya_ekspedisi, nama_sopir: e.nama_sopir, })), }; @@ -145,6 +148,7 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { no_surat_jalan: '', dokumen: '', biaya_ekspedisi: 0, + biaya_ekspedisi_per_item: 0, nama_sopir: '', }, ]; @@ -252,6 +256,18 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { formikSetValues(formikInitialValues); }, [formikSetValues, formikInitialValues]); + useEffect(() => { + formik.values.ekspedisi?.forEach((eks, idx) => { + if (eks.qty && eks.biaya_ekspedisi) { + const perItem = eks.biaya_ekspedisi / eks.qty; + formik.setFieldValue( + `ekspedisi.${idx}.biaya_ekspedisi_per_item`, + perItem + ); + } + }); + }, [formik.values.ekspedisi]); + const getFilteredProductOptions = useCallback(() => { return ( formik.values.product @@ -646,7 +662,8 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { Plat Nomor No Surat Jalan Dokumen - Biaya Ekspedisi + Biaya Ekspedisi (Rp.) + Biaya Ekspedisi / Item (Rp.) Nama Sopir {type !== 'detail' && Aksi} @@ -836,6 +853,31 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { }} /> + + + Date: Sun, 12 Oct 2025 19:15:14 +0700 Subject: [PATCH 14/55] feat(FE-64): add hatchery and npwp fields to MovementTable data structure --- .../inventory/movement/MovementTable.tsx | 91 ++----------------- 1 file changed, 10 insertions(+), 81 deletions(-) diff --git a/src/components/pages/inventory/movement/MovementTable.tsx b/src/components/pages/inventory/movement/MovementTable.tsx index 288cb2a6..c863a1a9 100644 --- a/src/components/pages/inventory/movement/MovementTable.tsx +++ b/src/components/pages/inventory/movement/MovementTable.tsx @@ -89,9 +89,11 @@ const dummyMovements: Movement[] = [ category: 'General', pic: 'PIC 1', type: 'Type 1', + hatchery: 'Hatchery 1', phone: '08123456789', email: 'supplier1@example.com', address: 'Address 1', + npwp: '1234567890123456', account_number: '1234567890', balance: 0, due_date: 30, @@ -103,7 +105,6 @@ const dummyMovements: Movement[] = [ nama_sopir: 'Andi', }, ], - name: 'Movement 1', }, { ...baseMetadata, @@ -165,9 +166,11 @@ const dummyMovements: Movement[] = [ category: 'Special', pic: 'PIC 2', type: 'Type 2', + hatchery: 'Hatchery 2', phone: '08123456780', email: 'supplier2@example.com', address: 'Address 2', + npwp: '1234567890123457', account_number: '1234567891', balance: 1000, due_date: 15, @@ -179,7 +182,6 @@ const dummyMovements: Movement[] = [ nama_sopir: 'Budi', }, ], - name: 'Movement 2', }, { ...baseMetadata, @@ -241,9 +243,11 @@ const dummyMovements: Movement[] = [ category: 'Return', pic: 'PIC 3', type: 'Type 3', + hatchery: 'Hatchery 3', phone: '08123456781', email: 'supplier3@example.com', address: 'Address 3', + npwp: '1234567890123458', account_number: '1234567892', balance: 500, due_date: 10, @@ -255,7 +259,6 @@ const dummyMovements: Movement[] = [ nama_sopir: 'Cici', }, ], - name: 'Movement 3', }, { ...baseMetadata, @@ -317,9 +320,11 @@ const dummyMovements: Movement[] = [ category: 'Internal', pic: 'PIC 4', type: 'Type 4', + hatchery: 'Hatchery 4', phone: '08123456782', email: 'supplier4@example.com', address: 'Address 4', + npwp: '1234567890123459', account_number: '1234567893', balance: 200, due_date: 20, @@ -331,86 +336,10 @@ const dummyMovements: Movement[] = [ nama_sopir: 'Dedi', }, ], - name: 'Movement 4', - }, - { - ...baseMetadata, - id: 5, - alasan_transfer: 'Distribusi', - tanggal_transfer: '2024-06-05', - warehouse_asal: { - ...baseMetadata, - id: 5, - name: 'Warehouse E', - type: 'AREA', - area: { id: 5, name: 'Area 5' }, - }, - warehouse_tujuan: { - ...baseMetadata, - id: 1, - name: 'Warehouse A', - type: 'AREA', - area: { id: 1, name: 'Area 1' }, - }, - product: [ - { - product: { - ...baseMetadata, - id: 5, - name: 'Product B', - brand: 'Brand B', - sku: 'SKU-B', - product_price: 8000, - selling_price: 9500, - tax: 2, - expiry_period: 120, - uom: { - ...baseMetadata, - id: 5, - name: 'PAK', - }, - product_category: { - ...baseMetadata, - id: 5, - code: 'CAT-5', - name: 'Category 5', - }, - suppliers: [], - flags: [], - }, - qty_product: 15, - }, - ], - ekspedisi: [ - { - product_id: 5, - qty: 15, - supplier: { - ...baseMetadata, - id: 5, - name: 'Supplier 5', - alias: 'S5', - category: 'Distribusi', - pic: 'PIC 5', - type: 'Type 5', - phone: '08123456783', - email: 'supplier5@example.com', - address: 'Address 5', - account_number: '1234567894', - balance: 300, - due_date: 25, - }, - plat_nomor: 'K 6789 KL', - no_surat_jalan: 'SJ-005', - dokumen: 'doc5.pdf', - biaya_ekspedisi: 70000, - nama_sopir: 'Eka', - }, - ], - name: 'Movement 5', }, ]; + const RowOptionsMenu = ({ type = 'dropdown', props, @@ -633,7 +562,7 @@ const MovementTable = () => { Date: Sun, 12 Oct 2025 20:32:02 +0700 Subject: [PATCH 15/55] feat: add Layout component to wrap children with SuspenseHelper --- src/app/master-data/customer/detail/layout.tsx | 11 +++++++++++ src/app/master-data/supplier/detail/layout.tsx | 11 +++++++++++ 2 files changed, 22 insertions(+) create mode 100644 src/app/master-data/customer/detail/layout.tsx create mode 100644 src/app/master-data/supplier/detail/layout.tsx diff --git a/src/app/master-data/customer/detail/layout.tsx b/src/app/master-data/customer/detail/layout.tsx new file mode 100644 index 00000000..7220dfa1 --- /dev/null +++ b/src/app/master-data/customer/detail/layout.tsx @@ -0,0 +1,11 @@ +import SuspenseHelper from '@/components/helper/SuspenseHelper'; + +const Layout = ({ + children, +}: Readonly<{ + children: React.ReactNode; +}>) => { + return {children}; +}; + +export default Layout; diff --git a/src/app/master-data/supplier/detail/layout.tsx b/src/app/master-data/supplier/detail/layout.tsx new file mode 100644 index 00000000..7220dfa1 --- /dev/null +++ b/src/app/master-data/supplier/detail/layout.tsx @@ -0,0 +1,11 @@ +import SuspenseHelper from '@/components/helper/SuspenseHelper'; + +const Layout = ({ + children, +}: Readonly<{ + children: React.ReactNode; +}>) => { + return {children}; +}; + +export default Layout; From 44e07ddc501335c568e928df72a4a6be3ef3d8e0 Mon Sep 17 00:00:00 2001 From: rstubryan Date: Tue, 14 Oct 2025 09:26:21 +0700 Subject: [PATCH 16/55] feat(FE-64): refactor MovementTable with new TableToolbar and TableRowSizeSelector components --- .../inventory/movement/MovementTable.tsx | 325 +++++++----------- src/components/table/TableRowOptions.tsx | 62 ++++ src/components/table/TableRowSizeSelector.tsx | 33 ++ src/components/table/TableToolbar.tsx | 37 ++ 4 files changed, 249 insertions(+), 208 deletions(-) create mode 100644 src/components/table/TableRowOptions.tsx create mode 100644 src/components/table/TableRowSizeSelector.tsx create mode 100644 src/components/table/TableToolbar.tsx diff --git a/src/components/pages/inventory/movement/MovementTable.tsx b/src/components/pages/inventory/movement/MovementTable.tsx index c863a1a9..b39906d3 100644 --- a/src/components/pages/inventory/movement/MovementTable.tsx +++ b/src/components/pages/inventory/movement/MovementTable.tsx @@ -1,23 +1,22 @@ 'use client'; import { useState, useMemo } from 'react'; -import { CellContext, ColumnDef, SortingState } from '@tanstack/react-table'; +import { SortingState } from '@tanstack/react-table'; import { Icon } from '@iconify/react'; import Table from '@/components/Table'; -import DebouncedTextInput from '@/components/input/DebouncedTextInput'; -import Button from '@/components/Button'; import { useModal } from '@/components/Modal'; import ConfirmationModal from '@/components/modal/ConfirmationModal'; -import SelectInput, { OptionType } from '@/components/input/SelectInput'; -import RowDropdownOptions from '@/components/table/RowDropdownOptions'; -import RowCollapseOptions from '@/components/table/RowCollapseOptions'; -import { cn } from '@/lib/helper'; import { ROWS_OPTIONS } from '@/config/constant'; import { Movement } from '@/types/api/inventory/movement'; -import { BaseMetadata } from '@/types/api/api-general'; +import { TableToolbar } from '@/components/table/TableToolbar'; +import { TableRowSizeSelector } from '@/components/table/TableRowSizeSelector'; +import { TableRowOptions } from '@/components/table/TableRowOptions'; +import { OptionType } from '@/components/input/SelectInput'; +import Button from '@/components/Button'; +import { cn } from '@/lib/helper'; // Dummy data -const baseMetadata: BaseMetadata = { +const baseMetadata = { created_user: { id: 1, id_user: 1, @@ -339,159 +338,17 @@ const dummyMovements: Movement[] = [ }, ]; - -const RowOptionsMenu = ({ - type = 'dropdown', - props, - deleteClickHandler, -}: { - type: 'dropdown' | 'collapse'; - props: CellContext; - deleteClickHandler: () => void; -}) => ( -
- - - -
-); - const MovementTable = () => { const [search, setSearch] = useState(''); const [page, setPage] = useState(1); const [pageSize, setPageSize] = useState(10); const [sorting, setSorting] = useState([]); - const [selectedMovement, setSelectedMovement] = useState< - Movement | undefined - >(undefined); + const [, setSelectedMovement] = useState(undefined); const [isDeleteLoading, setIsDeleteLoading] = useState(false); - const paginatedData = useMemo(() => { - const start = (page - 1) * pageSize; - return dummyMovements.slice(start, start + pageSize); - }, [page, pageSize]); - - const movementsColumns: ColumnDef[] = [ - { - header: '#', - cell: (props) => pageSize * (page - 1) + props.row.index + 1, - }, - { - accessorKey: 'warehouse_asal', - header: 'Gudang Asal', - cell: (props) => props.row.original.warehouse_asal.name, - }, - { - accessorKey: 'warehouse_tujuan', - header: 'Gudang Tujuan', - cell: (props) => props.row.original.warehouse_tujuan.name, - }, - { - accessorKey: 'product', - header: 'Nama Produk', - cell: (props) => props.row.original.product.map((p) => p.product.name), - }, - { - accessorKey: 'alasan_transfer', - header: 'Catatan', - }, - { - accessorKey: 'biaya_ekspedisi', - header: 'Biaya Ekspedisi', - cell: (props) => - props.row.original.ekspedisi.map((e) => e.biaya_ekspedisi), - }, - { - header: 'Aksi', - cell: (props) => { - const currentPageSize = props.table.getPaginationRowModel().rows.length; - const currentPageRows = props.table.getPaginationRowModel().flatRows; - const currentRowRelativeIndex = - currentPageRows.findIndex((r) => r.id === props.row.id) + 1; - - const isLast2Rows = currentRowRelativeIndex > currentPageSize - 2; - - const deleteClickHandler = () => { - setSelectedMovement(props.row.original); - deleteModal.openModal(); - }; - - return ( - <> - {currentPageSize > 2 && ( - - - - )} - {currentPageSize <= 2 && ( - - - - )} - - ); - }, - }, - ]; - const deleteModal = useModal(); - const confirmationModalDeleteClickHandler = async () => { - setIsDeleteLoading(true); - setTimeout(() => { - setIsDeleteLoading(false); - deleteModal.closeModal(); - }, 1000); - }; - const searchChangeHandler: React.ChangeEventHandler = ( - e - ) => { + const searchChangeHandler = (e: React.ChangeEvent) => { setSearch(e.target.value); setPage(1); }; @@ -502,67 +359,119 @@ const MovementTable = () => { setPage(1); }; + const confirmationModalDeleteClickHandler = async () => { + setIsDeleteLoading(true); + setTimeout(() => { + setIsDeleteLoading(false); + deleteModal.closeModal(); + }, 1000); + }; + + const paginatedData = useMemo(() => { + const start = (page - 1) * pageSize; + return dummyMovements.slice(start, start + pageSize); + }, [page, pageSize]); + return ( - <> -
-
-
-
- -
- -
-
- -
-
- - data={paginatedData} - columns={movementsColumns} - pageSize={pageSize} - page={page} - totalItems={dummyMovements.length} - onPageChange={setPage} - isLoading={false} - sorting={sorting} - setSorting={setSorting} - className={{ - containerClassName: cn({ - 'mb-20': paginatedData.length === 0, - }), - tableWrapperClassName: 'overflow-x-auto min-h-full!', - tableClassName: 'font-inter w-full table-auto min-h-full!', - headerRowClassName: 'border-b border-b-gray-200', - headerColumnClassName: - 'px-6 py-3 text-xs font-semibold text-gray-500 last:flex last:flex-row last:justify-end', - bodyRowClassName: 'border-b border-b-gray-200', - bodyColumnClassName: - 'px-6 py-3 last:flex last:flex-row last:justify-end', +
+
+ +
+ + pageSize * (page - 1) + props.row.index + 1, + }, + { + accessorKey: 'warehouse_asal', + header: 'Gudang Asal', + cell: (props) => props.row.original.warehouse_asal.name, + }, + { + accessorKey: 'warehouse_tujuan', + header: 'Gudang Tujuan', + cell: (props) => props.row.original.warehouse_tujuan.name, + }, + { + accessorKey: 'product', + header: 'Nama Produk', + cell: (props) => + props.row.original.product.map((p) => p.product.name), + }, + { + accessorKey: 'alasan_transfer', + header: 'Catatan', + }, + { + accessorKey: 'biaya_ekspedisi', + header: 'Biaya Ekspedisi', + cell: (props) => + props.row.original.ekspedisi.map((e) => e.biaya_ekspedisi), + }, + { + id: 'actions', + cell: (props) => ( +
+ + { + setSelectedMovement(props.row.original); + deleteModal.openModal(); + }} + /> +
+ ), + }, + ]} + pageSize={pageSize} + page={page} + totalItems={dummyMovements.length} + onPageChange={setPage} + isLoading={false} + sorting={sorting} + setSorting={setSorting} + className={{ + containerClassName: cn({ + 'mb-20': paginatedData.length === 0, + }), + tableWrapperClassName: 'overflow-x-auto min-h-full!', + tableClassName: 'font-inter w-full table-auto min-h-full!', + headerRowClassName: 'border-b border-b-gray-200', + headerColumnClassName: + 'px-6 py-3 text-xs font-semibold text-gray-500 last:flex last:flex-row last:justify-end', + bodyRowClassName: 'border-b border-b-gray-200', + bodyColumnClassName: + 'px-6 py-3 last:flex last:flex-row last:justify-end', + }} + /> + { onClick: confirmationModalDeleteClickHandler, }} /> - + ); }; diff --git a/src/components/table/TableRowOptions.tsx b/src/components/table/TableRowOptions.tsx new file mode 100644 index 00000000..61332b4f --- /dev/null +++ b/src/components/table/TableRowOptions.tsx @@ -0,0 +1,62 @@ +import { Icon } from '@iconify/react'; +import Button from '../Button'; +import { cn } from '@/lib/helper'; + +interface TableRowOptionsProps { + type?: 'dropdown' | 'collapse'; + recordId: string | number; + basePath: string; + onDelete?: () => void; +} + +export const TableRowOptions = ({ + type = 'dropdown', + recordId, + basePath, + onDelete, +}: TableRowOptionsProps) => ( +
+ + + {onDelete && ( + + )} +
+); diff --git a/src/components/table/TableRowSizeSelector.tsx b/src/components/table/TableRowSizeSelector.tsx new file mode 100644 index 00000000..a6fd039d --- /dev/null +++ b/src/components/table/TableRowSizeSelector.tsx @@ -0,0 +1,33 @@ +import SelectInput from '../input/SelectInput'; + +export interface OptionType { + label: string; + value: string | number; +} + +interface TableRowSizeSelectorProps { + value: number; + onChange: (val: OptionType | OptionType[] | null) => void; + options: OptionType[]; +} + +export const TableRowSizeSelector = ({ + value, + onChange, + options, +}: TableRowSizeSelectorProps) => { + return ( +
+ +
+ ); +}; diff --git a/src/components/table/TableToolbar.tsx b/src/components/table/TableToolbar.tsx new file mode 100644 index 00000000..e3b385b1 --- /dev/null +++ b/src/components/table/TableToolbar.tsx @@ -0,0 +1,37 @@ +import { Icon } from '@iconify/react'; +import Button from '../Button'; +import DebouncedTextInput from '../input/DebouncedTextInput'; + +interface TableToolbarProps { + addButton?: { + href: string; + label: string; + }; + search: { + value: string; + onChange: (e: React.ChangeEvent) => void; + placeholder?: string; + }; +} + +export const TableToolbar = ({ addButton, search }: TableToolbarProps) => { + return ( +
+ {addButton && ( +
+ +
+ )} + +
+ ); +}; From e7085ab4ff31b3e368a47d12f1b36a1cdbbab967 Mon Sep 17 00:00:00 2001 From: rstubryan Date: Tue, 14 Oct 2025 10:02:56 +0700 Subject: [PATCH 17/55] feat(FE-65): add file size validation for dokumen in MovementForm --- .../inventory/movement/form/MovementForm.schema.ts | 10 +++++++++- .../pages/inventory/movement/form/MovementForm.tsx | 4 ++++ 2 files changed, 13 insertions(+), 1 deletion(-) diff --git a/src/components/pages/inventory/movement/form/MovementForm.schema.ts b/src/components/pages/inventory/movement/form/MovementForm.schema.ts index 0bc0397c..f606b0c5 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.schema.ts +++ b/src/components/pages/inventory/movement/form/MovementForm.schema.ts @@ -59,7 +59,15 @@ const EkspedisiObjectSchema: Yup.ObjectSchema = Yup.object({ supplier_id: Yup.number().required('Supplier wajib diisi!'), plat_nomor: Yup.string().required('Plat nomor wajib diisi!'), no_surat_jalan: Yup.string().required('No surat jalan wajib diisi!'), - dokumen: Yup.mixed().required('Dokumen wajib diisi!'), + dokumen: Yup.mixed() + .required('Dokumen wajib diisi!') + .test( + 'fileSize', + 'Ukuran dokumen maksimal 2 MB!', + (value) => + typeof value === 'string' || + (value instanceof File && value.size <= 2 * 1024 * 1024) + ), biaya_ekspedisi: Yup.number() .required('Biaya ekspedisi wajib diisi!') .min(0, 'Biaya minimal 0!') diff --git a/src/components/pages/inventory/movement/form/MovementForm.tsx b/src/components/pages/inventory/movement/form/MovementForm.tsx index db432a4d..772f70d5 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.tsx +++ b/src/components/pages/inventory/movement/form/MovementForm.tsx @@ -817,6 +817,10 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { onChange={(e) => { const file = e.target.files?.[0]; if (file) { + if (file.size > 2 * 1024 * 1024) { + toast.error('Ukuran dokumen maksimal 2 MB!'); + return; + } formik.setFieldValue( `ekspedisi.${idx}.dokumen`, file From b2f0bd6698e57fc19a3ed9c41dd28594fc6c1d5a Mon Sep 17 00:00:00 2001 From: rstubryan Date: Tue, 14 Oct 2025 10:31:34 +0700 Subject: [PATCH 18/55] feat(FE-65): add file type validation for dokumen in MovementForm --- .../inventory/movement/form/MovementForm.schema.ts | 8 ++++++++ .../pages/inventory/movement/form/MovementForm.tsx | 11 +++++++++++ 2 files changed, 19 insertions(+) diff --git a/src/components/pages/inventory/movement/form/MovementForm.schema.ts b/src/components/pages/inventory/movement/form/MovementForm.schema.ts index f606b0c5..aa138cac 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.schema.ts +++ b/src/components/pages/inventory/movement/form/MovementForm.schema.ts @@ -61,6 +61,14 @@ const EkspedisiObjectSchema: Yup.ObjectSchema = Yup.object({ no_surat_jalan: Yup.string().required('No surat jalan wajib diisi!'), dokumen: Yup.mixed() .required('Dokumen wajib diisi!') + .test( + 'fileType', + 'Mohon upload file berformat PDF atau JPEG/JPG.', + (value) => + typeof value === 'string' || + (value instanceof File && + ['application/pdf', 'image/jpeg', 'image/jpg'].includes(value.type)) + ) .test( 'fileSize', 'Ukuran dokumen maksimal 2 MB!', diff --git a/src/components/pages/inventory/movement/form/MovementForm.tsx b/src/components/pages/inventory/movement/form/MovementForm.tsx index 772f70d5..cd81b8a0 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.tsx +++ b/src/components/pages/inventory/movement/form/MovementForm.tsx @@ -817,6 +817,17 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { onChange={(e) => { const file = e.target.files?.[0]; if (file) { + const allowedTypes = [ + 'application/pdf', + 'image/jpeg', + 'image/jpg', + ]; + if (!allowedTypes.includes(file.type)) { + toast.error( + 'Mohon upload file berformat PDF atau JPEG/JPG.' + ); + return; + } if (file.size > 2 * 1024 * 1024) { toast.error('Ukuran dokumen maksimal 2 MB!'); return; From 6facfd3d3c2d89e706db63b815e29f66b5a03f20 Mon Sep 17 00:00:00 2001 From: rstubryan Date: Tue, 14 Oct 2025 14:00:58 +0700 Subject: [PATCH 19/55] feat(FE-65): enhance MovementForm to support file uploads with FormData conversion --- .../inventory/movement/form/MovementForm.tsx | 17 +++++-- .../movement/form/useMovementFormHandlers.ts | 13 +++++- src/lib/form-data.ts | 45 +++++++++++++++++++ src/services/api/base.ts | 26 +++++++---- 4 files changed, 88 insertions(+), 13 deletions(-) create mode 100644 src/lib/form-data.ts diff --git a/src/components/pages/inventory/movement/form/MovementForm.tsx b/src/components/pages/inventory/movement/form/MovementForm.tsx index cd81b8a0..e912a453 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.tsx +++ b/src/components/pages/inventory/movement/form/MovementForm.tsx @@ -24,13 +24,17 @@ import { ProductSchema, EkspedisiSchema, } from '@/components/pages/inventory/movement/form/MovementForm.schema'; -import { useMovementFormHandlers } from './useMovementFormHandlers'; +import { + useMovementFormHandlers, + containsFile, +} from './useMovementFormHandlers'; import { ProductApi, SupplierApi, WarehouseApi, } from '@/services/api/master-data'; import { toast } from 'react-hot-toast'; +import FileInput from '@/components/input/FileInput'; interface MovementFormProps { type?: 'add' | 'edit' | 'detail'; @@ -62,6 +66,11 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { validationSchema: type === 'edit' ? UpdateMovementFormSchema : MovementFormSchema, onSubmit: async (values) => { + console.log( + 'Dokumen:', + values.ekspedisi?.map((e) => e.dokumen) + ); + setMovementFormErrorMessage(''); const payload: CreateMovementPayload = { alasan_transfer: values.alasan_transfer, @@ -88,6 +97,9 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { })), }; + console.log('containsFile:', containsFile(payload)); + console.log('payload:', payload); + switch (type) { case 'add': await createMovementHandler(payload); @@ -810,9 +822,8 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { />
- {formik.values.product?.map((product, idx) => ( + {formik.values.products?.map((product, idx) => ( {type !== 'detail' && (
- { const file = e.target.files?.[0]; diff --git a/src/components/pages/inventory/movement/form/useMovementFormHandlers.ts b/src/components/pages/inventory/movement/form/useMovementFormHandlers.ts index f6531bf4..0b6b0962 100644 --- a/src/components/pages/inventory/movement/form/useMovementFormHandlers.ts +++ b/src/components/pages/inventory/movement/form/useMovementFormHandlers.ts @@ -8,6 +8,7 @@ import { UpdateMovementPayload, } from '@/types/api/inventory/movement'; import { isResponseError } from '@/lib/api-helper'; +import { containsFile, toFormData } from '@/lib/form-data'; export const useMovementFormHandlers = (initialValuesId?: number) => { const router = useRouter(); @@ -17,7 +18,11 @@ export const useMovementFormHandlers = (initialValuesId?: number) => { const createMovementHandler = useCallback( async (payload: CreateMovementPayload) => { - const res = await MovementApi.create(payload); + const finalPayload = containsFile(payload) + ? (toFormData(payload) as unknown as CreateMovementPayload) + : payload; + + const res = await MovementApi.create(finalPayload); if (isResponseError(res)) { setMovementFormErrorMessage(res.message); return; @@ -30,7 +35,11 @@ export const useMovementFormHandlers = (initialValuesId?: number) => { const updateMovementHandler = useCallback( async (movementId: number, payload: UpdateMovementPayload) => { - const res = await MovementApi.update(movementId, payload); + const finalPayload = containsFile(payload) + ? (toFormData(payload) as unknown as UpdateMovementPayload) + : payload; + + const res = await MovementApi.update(movementId, finalPayload); if (res?.status === 'error') { setMovementFormErrorMessage(res.message); return; diff --git a/src/lib/form-data.ts b/src/lib/form-data.ts new file mode 100644 index 00000000..d94e0724 --- /dev/null +++ b/src/lib/form-data.ts @@ -0,0 +1,45 @@ +export function toFormData( + value: unknown, + form = new FormData(), + parentKey?: string +) { + if (value === undefined || value === null) { + if (parentKey) form.append(parentKey, ''); + return form; + } + + if (value instanceof File) { + if (!parentKey) throw new Error('File must have a key'); + form.append(parentKey, value); + return form; + } + + if (Array.isArray(value)) { + value.forEach((v, i) => { + const key = parentKey ? `${parentKey}[${i}]` : `${i}`; + toFormData(v, form, key); + }); + return form; + } + + if (typeof value === 'object') { + Object.entries(value as Record).forEach(([k, v]) => { + const key = parentKey ? `${parentKey}[${k}]` : k; + toFormData(v, form, key); + }); + return form; + } + + if (parentKey) form.append(parentKey, String(value)); + return form; +} + +export function containsFile(obj: unknown): boolean { + if (!obj) return false; + if (obj instanceof File) return true; + if (Array.isArray(obj)) return obj.some(containsFile); + if (typeof obj === 'object') { + return Object.values(obj as Record).some(containsFile); + } + return false; +} diff --git a/src/services/api/base.ts b/src/services/api/base.ts index d1ac4729..c4dd826e 100644 --- a/src/services/api/base.ts +++ b/src/services/api/base.ts @@ -4,9 +4,11 @@ import { BaseApiResponse } from '@/types/api/api-general'; export class BaseApiService { basePath: string; + header?: Record; - constructor(basePath: string) { + constructor(basePath: string, header?: Record) { this.basePath = basePath; + this.header = header; } async getAllFetcher(endpoint: string): Promise> { @@ -23,42 +25,52 @@ export class BaseApiService { if (axios.isAxiosError>(error)) { return error.response?.data; } - return undefined; } } async create(payload: CreatePayloadGeneric) { + const isFormData = + typeof FormData !== 'undefined' && payload instanceof FormData; try { + const headers = isFormData + ? { ...(this.header ?? {}) } + : { 'Content-Type': 'application/json', ...(this.header ?? {}) }; + const createRes = await httpClient>(this.basePath, { method: 'POST', body: payload, + headers, }); - return createRes; } catch (error: unknown) { if (axios.isAxiosError>(error)) { return error.response?.data; } - return undefined; } } async update(id: number, payload: UpdatePayloadGeneric) { + const isFormData = + typeof FormData !== 'undefined' && payload instanceof FormData; try { const updatePath = `${this.basePath}/${id}`; + + const headers = isFormData + ? { ...(this.header ?? {}) } + : { 'Content-Type': 'application/json', ...(this.header ?? {}) }; + const updateRes = await httpClient>(updatePath, { method: 'PATCH', body: payload, + headers, }); - return updateRes; } catch (error: unknown) { if (axios.isAxiosError>(error)) { return error.response?.data; } - return undefined; } } @@ -69,13 +81,11 @@ export class BaseApiService { const deleteRes = await httpClient(deletePath, { method: 'DELETE', }); - return deleteRes; } catch (error) { if (axios.isAxiosError(error)) { return error.response?.data; } - return undefined; } } From 19bca9ec730438e8831c100ac0d6d56daaa829d3 Mon Sep 17 00:00:00 2001 From: rstubryan Date: Tue, 14 Oct 2025 14:00:58 +0700 Subject: [PATCH 20/55] feat(FE-65): enhance MovementForm to support file uploads with FormData conversion --- .../inventory/movement/form/MovementForm.tsx | 13 +++++- .../movement/form/useMovementFormHandlers.ts | 13 +++++- src/lib/form-data.ts | 45 +++++++++++++++++++ src/services/api/base.ts | 26 +++++++---- 4 files changed, 85 insertions(+), 12 deletions(-) create mode 100644 src/lib/form-data.ts diff --git a/src/components/pages/inventory/movement/form/MovementForm.tsx b/src/components/pages/inventory/movement/form/MovementForm.tsx index cd81b8a0..89546442 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.tsx +++ b/src/components/pages/inventory/movement/form/MovementForm.tsx @@ -31,6 +31,8 @@ import { WarehouseApi, } from '@/services/api/master-data'; import { toast } from 'react-hot-toast'; +import FileInput from '@/components/input/FileInput'; +import { containsFile } from '@/lib/form-data'; interface MovementFormProps { type?: 'add' | 'edit' | 'detail'; @@ -62,6 +64,11 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { validationSchema: type === 'edit' ? UpdateMovementFormSchema : MovementFormSchema, onSubmit: async (values) => { + console.log( + 'Dokumen:', + values.ekspedisi?.map((e) => e.dokumen) + ); + setMovementFormErrorMessage(''); const payload: CreateMovementPayload = { alasan_transfer: values.alasan_transfer, @@ -88,6 +95,9 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { })), }; + console.log('containsFile:', containsFile(payload)); + console.log('payload:', payload); + switch (type) { case 'add': await createMovementHandler(payload); @@ -810,9 +820,8 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { /> - { const file = e.target.files?.[0]; diff --git a/src/components/pages/inventory/movement/form/useMovementFormHandlers.ts b/src/components/pages/inventory/movement/form/useMovementFormHandlers.ts index f6531bf4..0b6b0962 100644 --- a/src/components/pages/inventory/movement/form/useMovementFormHandlers.ts +++ b/src/components/pages/inventory/movement/form/useMovementFormHandlers.ts @@ -8,6 +8,7 @@ import { UpdateMovementPayload, } from '@/types/api/inventory/movement'; import { isResponseError } from '@/lib/api-helper'; +import { containsFile, toFormData } from '@/lib/form-data'; export const useMovementFormHandlers = (initialValuesId?: number) => { const router = useRouter(); @@ -17,7 +18,11 @@ export const useMovementFormHandlers = (initialValuesId?: number) => { const createMovementHandler = useCallback( async (payload: CreateMovementPayload) => { - const res = await MovementApi.create(payload); + const finalPayload = containsFile(payload) + ? (toFormData(payload) as unknown as CreateMovementPayload) + : payload; + + const res = await MovementApi.create(finalPayload); if (isResponseError(res)) { setMovementFormErrorMessage(res.message); return; @@ -30,7 +35,11 @@ export const useMovementFormHandlers = (initialValuesId?: number) => { const updateMovementHandler = useCallback( async (movementId: number, payload: UpdateMovementPayload) => { - const res = await MovementApi.update(movementId, payload); + const finalPayload = containsFile(payload) + ? (toFormData(payload) as unknown as UpdateMovementPayload) + : payload; + + const res = await MovementApi.update(movementId, finalPayload); if (res?.status === 'error') { setMovementFormErrorMessage(res.message); return; diff --git a/src/lib/form-data.ts b/src/lib/form-data.ts new file mode 100644 index 00000000..d94e0724 --- /dev/null +++ b/src/lib/form-data.ts @@ -0,0 +1,45 @@ +export function toFormData( + value: unknown, + form = new FormData(), + parentKey?: string +) { + if (value === undefined || value === null) { + if (parentKey) form.append(parentKey, ''); + return form; + } + + if (value instanceof File) { + if (!parentKey) throw new Error('File must have a key'); + form.append(parentKey, value); + return form; + } + + if (Array.isArray(value)) { + value.forEach((v, i) => { + const key = parentKey ? `${parentKey}[${i}]` : `${i}`; + toFormData(v, form, key); + }); + return form; + } + + if (typeof value === 'object') { + Object.entries(value as Record).forEach(([k, v]) => { + const key = parentKey ? `${parentKey}[${k}]` : k; + toFormData(v, form, key); + }); + return form; + } + + if (parentKey) form.append(parentKey, String(value)); + return form; +} + +export function containsFile(obj: unknown): boolean { + if (!obj) return false; + if (obj instanceof File) return true; + if (Array.isArray(obj)) return obj.some(containsFile); + if (typeof obj === 'object') { + return Object.values(obj as Record).some(containsFile); + } + return false; +} diff --git a/src/services/api/base.ts b/src/services/api/base.ts index d1ac4729..c4dd826e 100644 --- a/src/services/api/base.ts +++ b/src/services/api/base.ts @@ -4,9 +4,11 @@ import { BaseApiResponse } from '@/types/api/api-general'; export class BaseApiService { basePath: string; + header?: Record; - constructor(basePath: string) { + constructor(basePath: string, header?: Record) { this.basePath = basePath; + this.header = header; } async getAllFetcher(endpoint: string): Promise> { @@ -23,42 +25,52 @@ export class BaseApiService { if (axios.isAxiosError>(error)) { return error.response?.data; } - return undefined; } } async create(payload: CreatePayloadGeneric) { + const isFormData = + typeof FormData !== 'undefined' && payload instanceof FormData; try { + const headers = isFormData + ? { ...(this.header ?? {}) } + : { 'Content-Type': 'application/json', ...(this.header ?? {}) }; + const createRes = await httpClient>(this.basePath, { method: 'POST', body: payload, + headers, }); - return createRes; } catch (error: unknown) { if (axios.isAxiosError>(error)) { return error.response?.data; } - return undefined; } } async update(id: number, payload: UpdatePayloadGeneric) { + const isFormData = + typeof FormData !== 'undefined' && payload instanceof FormData; try { const updatePath = `${this.basePath}/${id}`; + + const headers = isFormData + ? { ...(this.header ?? {}) } + : { 'Content-Type': 'application/json', ...(this.header ?? {}) }; + const updateRes = await httpClient>(updatePath, { method: 'PATCH', body: payload, + headers, }); - return updateRes; } catch (error: unknown) { if (axios.isAxiosError>(error)) { return error.response?.data; } - return undefined; } } @@ -69,13 +81,11 @@ export class BaseApiService { const deleteRes = await httpClient(deletePath, { method: 'DELETE', }); - return deleteRes; } catch (error) { if (axios.isAxiosError(error)) { return error.response?.data; } - return undefined; } } From 4b4b74d07cb23cb223df6020a25fc9e97231f6ad Mon Sep 17 00:00:00 2001 From: rstubryan Date: Tue, 14 Oct 2025 18:00:34 +0700 Subject: [PATCH 21/55] feat(FE-65): add validation for quantity and required fields in MovementForm --- src/components/helper/form/FormActions.tsx | 4 +- .../movement/form/MovementForm.schema.ts | 20 +++- .../inventory/movement/form/MovementForm.tsx | 98 ++++++++++--------- 3 files changed, 70 insertions(+), 52 deletions(-) diff --git a/src/components/helper/form/FormActions.tsx b/src/components/helper/form/FormActions.tsx index 25b586c3..7ced46cd 100644 --- a/src/components/helper/form/FormActions.tsx +++ b/src/components/helper/form/FormActions.tsx @@ -8,6 +8,7 @@ interface FormActionsProps { formik: FormikContextType; editUrl?: string; onDelete?: () => void; + disableSubmit?: boolean; } export const FormActions = ({ @@ -15,6 +16,7 @@ export const FormActions = ({ formik, editUrl, onDelete, + disableSubmit = false, }: FormActionsProps) => { return (
@@ -71,7 +73,7 @@ export const FormActions = ({ color='primary' className='px-4' isLoading={formik.isSubmitting} - disabled={!formik.isValid || formik.isSubmitting} + disabled={disableSubmit || !formik.isValid || formik.isSubmitting} > Submit diff --git a/src/components/pages/inventory/movement/form/MovementForm.schema.ts b/src/components/pages/inventory/movement/form/MovementForm.schema.ts index aa138cac..453ca40b 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.schema.ts +++ b/src/components/pages/inventory/movement/form/MovementForm.schema.ts @@ -51,7 +51,17 @@ const EkspedisiObjectSchema: Yup.ObjectSchema = Yup.object({ qty: Yup.number() .required('Qty wajib diisi!') .min(1, 'Qty minimal 1!') - .typeError('Qty harus berupa angka!'), + .typeError('Qty harus berupa angka!') + .test('max-product-qty', 'Qty melebihi stok produk!', function (value) { + const { product_id } = this.parent; + const products = (this.options.context?.product ?? []) as { + product_id: number; + qty_product: number; + }[]; + const product = products.find((p) => p.product_id === product_id); + if (!product) return true; + return (value ?? 0) <= Number(product.qty_product); + }), supplier: Yup.object({ value: Yup.number().min(1).required(), label: Yup.string().required(), @@ -108,8 +118,12 @@ export const MovementFormSchema = Yup.object({ .typeError('Gudang tujuan wajib diisi!'), product: Yup.array() .of(ProductObjectSchema) - .min(1, 'Minimal harus ada 1 produk!'), - ekspedisi: Yup.array().of(EkspedisiObjectSchema).optional().default([]), + .min(1, 'Minimal harus ada 1 produk!') + .required('Produk wajib diisi!'), + ekspedisi: Yup.array() + .of(EkspedisiObjectSchema) + .min(1, 'Minimal harus ada 1 ekspedisi!') + .required('Ekspedisi wajib diisi!'), }); export const UpdateMovementFormSchema = MovementFormSchema; diff --git a/src/components/pages/inventory/movement/form/MovementForm.tsx b/src/components/pages/inventory/movement/form/MovementForm.tsx index 89546442..e5c90f4a 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.tsx +++ b/src/components/pages/inventory/movement/form/MovementForm.tsx @@ -1,7 +1,7 @@ 'use client'; import { useCallback, useEffect, useMemo, useState } from 'react'; -import { useFormik } from 'formik'; +import { FormikProps, useFormik } from 'formik'; import useSWR from 'swr'; import { Icon } from '@iconify/react'; @@ -32,13 +32,30 @@ import { } from '@/services/api/master-data'; import { toast } from 'react-hot-toast'; import FileInput from '@/components/input/FileInput'; -import { containsFile } from '@/lib/form-data'; interface MovementFormProps { type?: 'add' | 'edit' | 'detail'; initialValues?: Movement; } +function getEkspedisiFieldError( + formik: FormikProps, + idx: number, + field: keyof EkspedisiSchema +) { + const errorObj = formik.errors.ekspedisi?.[idx]; + const touched = formik.touched.ekspedisi?.[idx]?.[field]; + const isError = + touched && + typeof errorObj === 'object' && + !!(errorObj as Record)?.[field]; + const errorMessage = + typeof errorObj === 'object' + ? (errorObj as Record)?.[field] + : undefined; + return { isError, errorMessage }; +} + const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { const [, setMovementFormErrorMessage] = useState(''); const [selectedProducts, setSelectedProducts] = useState([]); @@ -63,12 +80,9 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { initialValues: formikInitialValues, validationSchema: type === 'edit' ? UpdateMovementFormSchema : MovementFormSchema, + validateOnChange: true, + validateOnBlur: true, onSubmit: async (values) => { - console.log( - 'Dokumen:', - values.ekspedisi?.map((e) => e.dokumen) - ); - setMovementFormErrorMessage(''); const payload: CreateMovementPayload = { alasan_transfer: values.alasan_transfer, @@ -95,9 +109,6 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { })), }; - console.log('containsFile:', containsFile(payload)); - console.log('payload:', payload); - switch (type) { case 'add': await createMovementHandler(payload); @@ -292,12 +303,10 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { const validateEkspedisiQty = (ekspedisiIdx: number, qty: number) => { const productId = formik.values.ekspedisi?.[ekspedisiIdx]?.product_id; if (!productId) return true; - const relatedProduct = formik.values.product?.find( (p) => p.product_id === productId ); if (!relatedProduct) return true; - const totalQtyUsed = formik.values.ekspedisi?.reduce((total, eks, i) => { if (eks.product_id === productId && i !== ekspedisiIdx) { @@ -305,10 +314,15 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { } return total; }, 0) || 0; - return totalQtyUsed + qty <= Number(relatedProduct.qty_product); }; + const invalidQtyRows = + formik.values.ekspedisi?.map((eks, idx) => { + const qty = Number(eks.qty) || 0; + return !validateEkspedisiQty(idx, qty); + }) ?? []; + return ( <>
@@ -735,22 +749,9 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { type='number' name={`ekspedisi.${idx}.qty`} value={ekspedisi.qty ?? ''} - onChange={(e) => { - const newQty = Number(e.target.value); - if (validateEkspedisiQty(idx, newQty)) { - formik.handleChange(e); - } else { - toast.error( - 'Quantity exceeds available product quantity' - ); - } - }} + onChange={formik.handleChange} onBlur={formik.handleBlur} - isError={isRepeaterInputError( - 'ekspedisi', - 'qty', - idx - )} + {...getEkspedisiFieldError(formik, idx, 'qty')} readOnly={type === 'detail'} className={{ wrapper: 'w-full min-w-24', @@ -790,10 +791,10 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { value={ekspedisi.plat_nomor ?? ''} onChange={formik.handleChange} onBlur={formik.handleBlur} - isError={isRepeaterInputError( - 'ekspedisi', - 'plat_nomor', - idx + {...getEkspedisiFieldError( + formik, + idx, + 'plat_nomor' )} readOnly={type === 'detail'} className={{ @@ -808,10 +809,10 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { value={ekspedisi.no_surat_jalan ?? ''} onChange={formik.handleChange} onBlur={formik.handleBlur} - isError={isRepeaterInputError( - 'ekspedisi', - 'no_surat_jalan', - idx + {...getEkspedisiFieldError( + formik, + idx, + 'no_surat_jalan' )} readOnly={type === 'detail'} className={{ @@ -866,10 +867,10 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { value={ekspedisi.biaya_ekspedisi ?? ''} onChange={formik.handleChange} onBlur={formik.handleBlur} - isError={isRepeaterInputError( - 'ekspedisi', - 'biaya_ekspedisi', - idx + {...getEkspedisiFieldError( + formik, + idx, + 'biaya_ekspedisi' )} readOnly={type === 'detail'} className={{ @@ -882,10 +883,10 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { disabled onChange={formik.handleChange} onBlur={formik.handleBlur} - isError={isRepeaterInputError( - 'ekspedisi', - 'biaya_ekspedisi_per_item', - idx + {...getEkspedisiFieldError( + formik, + idx, + 'biaya_ekspedisi_per_item' )} name={`ekspedisi.${idx}.biaya_ekspedisi_per_item`} value={ @@ -909,10 +910,10 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { value={ekspedisi.nama_sopir ?? ''} onChange={formik.handleChange} onBlur={formik.handleBlur} - isError={isRepeaterInputError( - 'ekspedisi', - 'nama_sopir', - idx + {...getEkspedisiFieldError( + formik, + idx, + 'nama_sopir' )} readOnly={type === 'detail'} className={{ @@ -982,6 +983,7 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { : undefined } onDelete={deleteMovementClickHandler} + disableSubmit={invalidQtyRows.some(Boolean)} /> {movementFormErrorMessage && ( From 56a9fc2349f8aa20ba56aa6c56e3ef433a516e3b Mon Sep 17 00:00:00 2001 From: rstubryan Date: Wed, 15 Oct 2025 10:51:06 +0700 Subject: [PATCH 22/55] refactor(FE-62,65): simplify error handling in MovementForm by consolidating error checks --- .../inventory/movement/form/MovementForm.tsx | 90 ++++++++----------- 1 file changed, 37 insertions(+), 53 deletions(-) diff --git a/src/components/pages/inventory/movement/form/MovementForm.tsx b/src/components/pages/inventory/movement/form/MovementForm.tsx index e5c90f4a..5d69f03e 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.tsx +++ b/src/components/pages/inventory/movement/form/MovementForm.tsx @@ -38,24 +38,6 @@ interface MovementFormProps { initialValues?: Movement; } -function getEkspedisiFieldError( - formik: FormikProps, - idx: number, - field: keyof EkspedisiSchema -) { - const errorObj = formik.errors.ekspedisi?.[idx]; - const touched = formik.touched.ekspedisi?.[idx]?.[field]; - const isError = - touched && - typeof errorObj === 'object' && - !!(errorObj as Record)?.[field]; - const errorMessage = - typeof errorObj === 'object' - ? (errorObj as Record)?.[field] - : undefined; - return { isError, errorMessage }; -} - const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { const [, setMovementFormErrorMessage] = useState(''); const [selectedProducts, setSelectedProducts] = useState([]); @@ -212,16 +194,22 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { !formik.touched[arrayName] || !Array.isArray(formik.touched[arrayName]) ) { - return false; + return { + isError: false, + errorMessage: undefined, + }; } const touchedField = formik.touched[arrayName]?.[idx]?.[column as string]; const errorField = formik.errors[arrayName]?.[idx] as Record< string, - unknown + string >; - return touchedField && Boolean(errorField?.[column as string]); + return { + isError: touchedField && Boolean(errorField?.[column as string]), + errorMessage: touchedField ? errorField?.[column as string] : undefined, + }; }; interface WarehouseOptionType extends OptionType { @@ -571,11 +559,7 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { isLoading={isLoadingProducts} isDisabled={type === 'detail'} isClearable - isError={isRepeaterInputError( - 'product', - 'product', - idx - )} + {...isRepeaterInputError('product', 'product', idx)} />
@@ -586,7 +570,7 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { value={product.qty_product ?? ''} onChange={formik.handleChange} onBlur={formik.handleBlur} - isError={isRepeaterInputError( + {...isRepeaterInputError( 'product', 'qty_product', idx @@ -734,13 +718,13 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { formik.setFieldValue(`ekspedisi.${idx}.qty`, ''); }} options={getFilteredProductOptions()} - isDisabled={type === 'detail'} - isClearable - isError={isRepeaterInputError( + {...isRepeaterInputError( 'ekspedisi', 'product', idx )} + isDisabled={type === 'detail'} + isClearable /> @@ -751,7 +735,7 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { value={ekspedisi.qty ?? ''} onChange={formik.handleChange} onBlur={formik.handleBlur} - {...getEkspedisiFieldError(formik, idx, 'qty')} + {...isRepeaterInputError('ekspedisi', 'qty', idx)} readOnly={type === 'detail'} className={{ wrapper: 'w-full min-w-24', @@ -777,7 +761,7 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { isLoading={isLoadingSuppliers} isDisabled={type === 'detail'} isClearable - isError={isRepeaterInputError( + {...isRepeaterInputError( 'ekspedisi', 'supplier', idx @@ -791,10 +775,10 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { value={ekspedisi.plat_nomor ?? ''} onChange={formik.handleChange} onBlur={formik.handleBlur} - {...getEkspedisiFieldError( - formik, - idx, - 'plat_nomor' + {...isRepeaterInputError( + 'ekspedisi', + 'plat_nomor', + idx )} readOnly={type === 'detail'} className={{ @@ -809,10 +793,10 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { value={ekspedisi.no_surat_jalan ?? ''} onChange={formik.handleChange} onBlur={formik.handleBlur} - {...getEkspedisiFieldError( - formik, - idx, - 'no_surat_jalan' + {...isRepeaterInputError( + 'ekspedisi', + 'no_surat_jalan', + idx )} readOnly={type === 'detail'} className={{ @@ -848,7 +832,7 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { ); } }} - isError={isRepeaterInputError( + {...isRepeaterInputError( 'ekspedisi', 'dokumen', idx @@ -867,10 +851,10 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { value={ekspedisi.biaya_ekspedisi ?? ''} onChange={formik.handleChange} onBlur={formik.handleBlur} - {...getEkspedisiFieldError( - formik, - idx, - 'biaya_ekspedisi' + {...isRepeaterInputError( + 'ekspedisi', + 'biaya_ekspedisi', + idx )} readOnly={type === 'detail'} className={{ @@ -883,10 +867,10 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { disabled onChange={formik.handleChange} onBlur={formik.handleBlur} - {...getEkspedisiFieldError( - formik, - idx, - 'biaya_ekspedisi_per_item' + {...isRepeaterInputError( + 'ekspedisi', + 'biaya_ekspedisi_per_item', + idx )} name={`ekspedisi.${idx}.biaya_ekspedisi_per_item`} value={ @@ -910,10 +894,10 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { value={ekspedisi.nama_sopir ?? ''} onChange={formik.handleChange} onBlur={formik.handleBlur} - {...getEkspedisiFieldError( - formik, - idx, - 'nama_sopir' + {...isRepeaterInputError( + 'ekspedisi', + 'nama_sopir', + idx )} readOnly={type === 'detail'} className={{ From 3c4333021fb88ffb04130e4742ef272e71a84ee8 Mon Sep 17 00:00:00 2001 From: rstubryan Date: Wed, 15 Oct 2025 10:54:38 +0700 Subject: [PATCH 23/55] feat(FE-62,65): enhance MovementForm and FormActions to improve form validation and reset behavior --- src/components/helper/form/FormActions.tsx | 12 ++++++++++-- .../pages/inventory/movement/form/MovementForm.tsx | 1 + 2 files changed, 11 insertions(+), 2 deletions(-) diff --git a/src/components/helper/form/FormActions.tsx b/src/components/helper/form/FormActions.tsx index 7ced46cd..54600d00 100644 --- a/src/components/helper/form/FormActions.tsx +++ b/src/components/helper/form/FormActions.tsx @@ -64,7 +64,10 @@ export const FormActions = ({ type='reset' color='warning' className='px-4' - onClick={formik.handleReset} + onClick={() => { + formik.handleReset(); + formik.validateForm(); + }} > Reset @@ -73,7 +76,12 @@ export const FormActions = ({ color='primary' className='px-4' isLoading={formik.isSubmitting} - disabled={disableSubmit || !formik.isValid || formik.isSubmitting} + disabled={ + disableSubmit || + !formik.isValid || + !formik.dirty || + formik.isSubmitting + } > Submit diff --git a/src/components/pages/inventory/movement/form/MovementForm.tsx b/src/components/pages/inventory/movement/form/MovementForm.tsx index 5d69f03e..f196b242 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.tsx +++ b/src/components/pages/inventory/movement/form/MovementForm.tsx @@ -64,6 +64,7 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { type === 'edit' ? UpdateMovementFormSchema : MovementFormSchema, validateOnChange: true, validateOnBlur: true, + validateOnMount: true, onSubmit: async (values) => { setMovementFormErrorMessage(''); const payload: CreateMovementPayload = { From dcd5d2692fdb12baa9eb9a43a53f788ce70ec951 Mon Sep 17 00:00:00 2001 From: rstubryan Date: Wed, 15 Oct 2025 10:54:38 +0700 Subject: [PATCH 24/55] feat(FE-62,65): enhance MovementForm and FormActions to improve form validation and reset behavior --- src/components/helper/form/FormActions.tsx | 12 ++++++++++-- .../pages/inventory/movement/form/MovementForm.tsx | 3 ++- 2 files changed, 12 insertions(+), 3 deletions(-) diff --git a/src/components/helper/form/FormActions.tsx b/src/components/helper/form/FormActions.tsx index 7ced46cd..54600d00 100644 --- a/src/components/helper/form/FormActions.tsx +++ b/src/components/helper/form/FormActions.tsx @@ -64,7 +64,10 @@ export const FormActions = ({ type='reset' color='warning' className='px-4' - onClick={formik.handleReset} + onClick={() => { + formik.handleReset(); + formik.validateForm(); + }} > Reset @@ -73,7 +76,12 @@ export const FormActions = ({ color='primary' className='px-4' isLoading={formik.isSubmitting} - disabled={disableSubmit || !formik.isValid || formik.isSubmitting} + disabled={ + disableSubmit || + !formik.isValid || + !formik.dirty || + formik.isSubmitting + } > Submit diff --git a/src/components/pages/inventory/movement/form/MovementForm.tsx b/src/components/pages/inventory/movement/form/MovementForm.tsx index 5d69f03e..dd927665 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.tsx +++ b/src/components/pages/inventory/movement/form/MovementForm.tsx @@ -1,7 +1,7 @@ 'use client'; import { useCallback, useEffect, useMemo, useState } from 'react'; -import { FormikProps, useFormik } from 'formik'; +import { useFormik } from 'formik'; import useSWR from 'swr'; import { Icon } from '@iconify/react'; @@ -64,6 +64,7 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { type === 'edit' ? UpdateMovementFormSchema : MovementFormSchema, validateOnChange: true, validateOnBlur: true, + validateOnMount: true, onSubmit: async (values) => { setMovementFormErrorMessage(''); const payload: CreateMovementPayload = { From df73ee1fdfc19bdcf935445b5f6d799c108de191 Mon Sep 17 00:00:00 2001 From: rstubryan Date: Wed, 15 Oct 2025 12:00:17 +0700 Subject: [PATCH 25/55] feat(FE-62,63,65): refactor MovementForm and related types for improved clarity and consistency --- src/components/helper/form/FormActions.tsx | 7 +- .../movement/form/MovementForm.schema.ts | 164 +++--- .../inventory/movement/form/MovementForm.tsx | 476 ++++++++---------- src/types/api/inventory/movement.d.ts | 61 +-- 4 files changed, 335 insertions(+), 373 deletions(-) diff --git a/src/components/helper/form/FormActions.tsx b/src/components/helper/form/FormActions.tsx index 54600d00..92c2a92c 100644 --- a/src/components/helper/form/FormActions.tsx +++ b/src/components/helper/form/FormActions.tsx @@ -76,12 +76,7 @@ export const FormActions = ({ color='primary' className='px-4' isLoading={formik.isSubmitting} - disabled={ - disableSubmit || - !formik.isValid || - !formik.dirty || - formik.isSubmitting - } + disabled={disableSubmit || !formik.isValid || formik.isSubmitting} > Submit diff --git a/src/components/pages/inventory/movement/form/MovementForm.schema.ts b/src/components/pages/inventory/movement/form/MovementForm.schema.ts index 453ca40b..cb0d228d 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.schema.ts +++ b/src/components/pages/inventory/movement/form/MovementForm.schema.ts @@ -7,27 +7,28 @@ export type ProductSchema = { label: string; } | null; product_id: number; - qty_product: number; + product_qty: number; }; -export type EkspedisiSchema = { - product: { - value: number; - label: string; - } | null; - product_id: number; - qty: number; +export type DeliverySchema = { + delivery_cost: number; + delivery_cost_per_item?: number | undefined; + document: string | File; + driver_name: string; + vehicle_plate: string; supplier: { value: number; label: string; } | null; supplier_id: number; - plat_nomor: string; - no_surat_jalan: string; - dokumen: string | File; - biaya_ekspedisi: number; - biaya_ekspedisi_per_item?: number | undefined; - nama_sopir: string; + products: { + product: { + value: number; + label: string; + } | null; + product_id: number; + product_qty: number; + }[]; }; const ProductObjectSchema: Yup.ObjectSchema = Yup.object({ @@ -36,40 +37,34 @@ const ProductObjectSchema: Yup.ObjectSchema = Yup.object({ label: Yup.string().required(), }).nullable(), product_id: Yup.number().required('Produk wajib diisi!'), - qty_product: Yup.number() + product_qty: Yup.number() .required('Qty wajib diisi!') .min(1, 'Qty minimal 1!') .typeError('Qty harus berupa angka!'), }); -const EkspedisiObjectSchema: Yup.ObjectSchema = Yup.object({ +const DeliveryProductObjectSchema = Yup.object({ product: Yup.object({ value: Yup.number().min(1).required(), label: Yup.string().required(), }).nullable(), product_id: Yup.number().required('Produk wajib diisi!'), - qty: Yup.number() + product_qty: Yup.number() .required('Qty wajib diisi!') .min(1, 'Qty minimal 1!') - .typeError('Qty harus berupa angka!') - .test('max-product-qty', 'Qty melebihi stok produk!', function (value) { - const { product_id } = this.parent; - const products = (this.options.context?.product ?? []) as { - product_id: number; - qty_product: number; - }[]; - const product = products.find((p) => p.product_id === product_id); - if (!product) return true; - return (value ?? 0) <= Number(product.qty_product); - }), - supplier: Yup.object({ - value: Yup.number().min(1).required(), - label: Yup.string().required(), - }).nullable(), - supplier_id: Yup.number().required('Supplier wajib diisi!'), - plat_nomor: Yup.string().required('Plat nomor wajib diisi!'), - no_surat_jalan: Yup.string().required('No surat jalan wajib diisi!'), - dokumen: Yup.mixed() + .typeError('Qty harus berupa angka!'), +}); + +const DeliveryObjectSchema: Yup.ObjectSchema = Yup.object({ + delivery_cost: Yup.number() + .required('Biaya pengiriman wajib diisi!') + .min(0, 'Biaya minimal 0!') + .typeError('Biaya harus berupa angka!'), + delivery_cost_per_item: Yup.number() + .transform((value) => (isNaN(value) ? undefined : value)) + .min(0, 'Biaya per item minimal 0!') + .typeError('Biaya per item harus berupa angka!'), + document: Yup.mixed() .required('Dokumen wajib diisi!') .test( 'fileType', @@ -86,44 +81,44 @@ const EkspedisiObjectSchema: Yup.ObjectSchema = Yup.object({ typeof value === 'string' || (value instanceof File && value.size <= 2 * 1024 * 1024) ), - biaya_ekspedisi: Yup.number() - .required('Biaya ekspedisi wajib diisi!') - .min(0, 'Biaya minimal 0!') - .typeError('Biaya harus berupa angka!'), - biaya_ekspedisi_per_item: Yup.number() - .transform((value) => (isNaN(value) ? undefined : value)) - .min(0, 'Biaya per item minimal 0!') - .typeError('Biaya per item harus berupa angka!') - .optional() - .default(undefined), - nama_sopir: Yup.string().required('Nama sopir wajib diisi!'), + driver_name: Yup.string().required('Nama sopir wajib diisi!'), + vehicle_plate: Yup.string().required('Plat nomor wajib diisi!'), + supplier: Yup.object({ + value: Yup.number().min(1).required(), + label: Yup.string().required(), + }).nullable(), + supplier_id: Yup.number().required('Supplier wajib diisi!'), + products: Yup.array() + .of(DeliveryProductObjectSchema) + .min(1, 'Minimal harus ada 1 produk!') + .required('Produk wajib diisi!'), }); export const MovementFormSchema = Yup.object({ - alasan_transfer: Yup.string().required('Alasan transfer wajib diisi!'), - tanggal_transfer: Yup.string().required('Tanggal transfer wajib diisi!'), - warehouse_asal: Yup.object({ + transfer_reason: Yup.string().required('Alasan transfer wajib diisi!'), + transfer_date: Yup.string().required('Tanggal transfer wajib diisi!'), + source_warehouse: Yup.object({ value: Yup.number().min(1).required(), label: Yup.string().required(), }).nullable(), - warehouse_asal_id: Yup.number() + source_warehouse_id: Yup.number() .required('Gudang asal wajib diisi!') .typeError('Gudang asal wajib diisi!'), - warehouse_tujuan: Yup.object({ + destination_warehouse: Yup.object({ value: Yup.number().min(1).required(), label: Yup.string().required(), }).nullable(), - warehouse_tujuan_id: Yup.number() + destination_warehouse_id: Yup.number() .required('Gudang tujuan wajib diisi!') .typeError('Gudang tujuan wajib diisi!'), - product: Yup.array() + products: Yup.array() .of(ProductObjectSchema) .min(1, 'Minimal harus ada 1 produk!') .required('Produk wajib diisi!'), - ekspedisi: Yup.array() - .of(EkspedisiObjectSchema) - .min(1, 'Minimal harus ada 1 ekspedisi!') - .required('Ekspedisi wajib diisi!'), + deliveries: Yup.array() + .of(DeliveryObjectSchema) + .min(1, 'Minimal harus ada 1 pengiriman!') + .required('Pengiriman wajib diisi!'), }); export const UpdateMovementFormSchema = MovementFormSchema; @@ -133,40 +128,41 @@ export type MovementFormValues = Yup.InferType; export const getMovementFormInitialValues = ( initialValues?: Movement ): MovementFormValues => ({ - alasan_transfer: initialValues?.alasan_transfer ?? '', - tanggal_transfer: initialValues?.tanggal_transfer ?? '', - warehouse_asal: initialValues?.warehouse_asal + transfer_reason: initialValues?.transfer_reason ?? '', + transfer_date: initialValues?.transfer_date ?? '', + source_warehouse: initialValues?.source_warehouse ? { - value: initialValues.warehouse_asal.id, - label: initialValues.warehouse_asal.name, + value: initialValues.source_warehouse.id, + label: initialValues.source_warehouse.name, } : null, - warehouse_asal_id: initialValues?.warehouse_asal?.id ?? 0, - warehouse_tujuan: initialValues?.warehouse_tujuan + source_warehouse_id: initialValues?.source_warehouse?.id ?? 0, + destination_warehouse: initialValues?.destination_warehouse ? { - value: initialValues.warehouse_tujuan.id, - label: initialValues.warehouse_tujuan.name, + value: initialValues.destination_warehouse.id, + label: initialValues.destination_warehouse.name, } : null, - warehouse_tujuan_id: initialValues?.warehouse_tujuan?.id ?? 0, - product: - initialValues?.product?.map((p) => ({ + destination_warehouse_id: initialValues?.destination_warehouse?.id ?? 0, + products: + initialValues?.products?.map((p) => ({ product: { value: p.product.id, label: p.product.name }, product_id: p.product.id, - qty_product: p.qty_product, + product_qty: p.product_qty, })) ?? [], - ekspedisi: - initialValues?.ekspedisi?.map((e) => ({ - product: { value: e.product_id, label: '' }, - product_id: e.product_id, - qty: e.qty, - supplier: { value: e.supplier.id, label: e.supplier.name }, - supplier_id: e.supplier.id, - plat_nomor: e.plat_nomor, - no_surat_jalan: e.no_surat_jalan, - dokumen: e.dokumen, - biaya_ekspedisi: e.biaya_ekspedisi, - biaya_ekspedisi_per_item: e.biaya_ekspedisi, - nama_sopir: e.nama_sopir, + deliveries: + initialValues?.deliveries?.map((d) => ({ + delivery_cost: d.delivery_cost, + delivery_cost_per_item: d.delivery_cost_per_item, + document: d.document, + driver_name: d.driver_name, + vehicle_plate: d.vehicle_plate, + supplier: { value: d.supplier.id, label: d.supplier.name }, + supplier_id: d.supplier.id, + products: d.products.map((p) => ({ + product: { value: p.product.id, label: p.product.name }, + product_id: p.product.id, + product_qty: p.product_qty, + })), })) ?? [], }); diff --git a/src/components/pages/inventory/movement/form/MovementForm.tsx b/src/components/pages/inventory/movement/form/MovementForm.tsx index dd927665..183fe760 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.tsx +++ b/src/components/pages/inventory/movement/form/MovementForm.tsx @@ -22,7 +22,7 @@ import { UpdateMovementFormSchema, getMovementFormInitialValues, ProductSchema, - EkspedisiSchema, + DeliverySchema, } from '@/components/pages/inventory/movement/form/MovementForm.schema'; import { useMovementFormHandlers } from './useMovementFormHandlers'; import { @@ -41,7 +41,7 @@ interface MovementFormProps { const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { const [, setMovementFormErrorMessage] = useState(''); const [selectedProducts, setSelectedProducts] = useState([]); - const [selectedEkspedisi, setSelectedEkspedisi] = useState([]); + const [selectedDeliveries, setSelectedDeliveries] = useState([]); const { deleteModal, @@ -64,31 +64,30 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { type === 'edit' ? UpdateMovementFormSchema : MovementFormSchema, validateOnChange: true, validateOnBlur: true, - validateOnMount: true, + validateOnMount: false, + enableReinitialize: true, onSubmit: async (values) => { setMovementFormErrorMessage(''); const payload: CreateMovementPayload = { - alasan_transfer: values.alasan_transfer, - tanggal_transfer: values.tanggal_transfer, - warehouse_asal_id: values.warehouse_asal_id, - warehouse_tujuan_id: values.warehouse_tujuan_id, - product: (values.product ?? []).map((p) => ({ + transfer_reason: values.transfer_reason, + transfer_date: values.transfer_date, + source_warehouse_id: values.source_warehouse_id, + destination_warehouse_id: values.destination_warehouse_id, + products: values.products.map((p) => ({ product_id: p.product_id, - qty_product: p.qty_product, + product_qty: p.product_qty, })), - ekspedisi: (values.ekspedisi ?? []).map((e) => ({ - product_id: e.product_id, - qty: e.qty, - supplier_id: e.supplier_id, - plat_nomor: e.plat_nomor, - no_surat_jalan: e.no_surat_jalan, - dokumen: - e.dokumen instanceof File ? e.dokumen : (e.dokumen as string), - biaya_ekspedisi: e.biaya_ekspedisi, - biaya_ekspedisi_per_item: e.qty - ? e.biaya_ekspedisi / e.qty - : e.biaya_ekspedisi, - nama_sopir: e.nama_sopir, + deliveries: values.deliveries.map((d) => ({ + delivery_cost: d.delivery_cost, + delivery_cost_per_item: d.delivery_cost_per_item ?? 0, + document: d.document instanceof File ? d.document : d.document, + driver_name: d.driver_name, + vehicle_plate: d.vehicle_plate, + supplier_id: d.supplier_id, + products: d.products.map((p) => ({ + product_id: p.product_id, + product_qty: p.product_qty, + })), })), }; @@ -105,65 +104,67 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { const addProduct = () => { const newProducts = [ - ...(formik.values.product || []), + ...(formik.values.products || []), { product: null, product_id: 0, - qty_product: 0, + product_qty: 0, }, ]; - formik.setFieldValue('product', newProducts); + formik.setFieldValue('products', newProducts); }; const removeProduct = useCallback( (i: number) => { const updatedProducts = - formik.values.product?.reduce((acc: ProductSchema[], item, index) => { + formik.values.products?.reduce((acc: ProductSchema[], item, index) => { if (index !== i) { acc.push(item); } return acc; }, []) ?? []; - formik.setFieldValue('product', updatedProducts); + formik.setFieldValue('products', updatedProducts); }, [formik] ); const bulkRemoveProduct = useCallback(() => { const updatedProducts = - formik.values.product?.filter( + formik.values.products?.filter( (_, idx) => !selectedProducts.includes(idx) ) ?? []; - formik.setFieldValue('product', updatedProducts); + formik.setFieldValue('products', updatedProducts); setSelectedProducts([]); }, [formik, selectedProducts]); - const addEkspedisi = () => { - const newEkspedisi = [ - ...(formik.values.ekspedisi || []), + const addDelivery = () => { + formik.setFieldValue('deliveries', [ + ...(formik.values.deliveries || []), { - product: null, - product_id: 0, - qty: 0, + delivery_cost: 0, + delivery_cost_per_item: 0, + document: '', + driver_name: '', + vehicle_plate: '', supplier: null, supplier_id: 0, - plat_nomor: '', - no_surat_jalan: '', - dokumen: '', - biaya_ekspedisi: 0, - biaya_ekspedisi_per_item: 0, - nama_sopir: '', + products: [ + { + product: null, + product_id: 0, + product_qty: 0, + }, + ], }, - ]; - formik.setFieldValue('ekspedisi', newEkspedisi); + ]); }; - const removeEkspedisi = useCallback( + const removeDelivery = useCallback( (i: number) => { - const updatedEkspedisi = - formik.values.ekspedisi?.reduce( - (acc: EkspedisiSchema[], item, index) => { + const updatedDeliveries = + formik.values.deliveries?.reduce( + (acc: DeliverySchema[], item, index) => { if (index !== i) { acc.push(item); } @@ -172,23 +173,23 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { [] ) ?? []; - formik.setFieldValue('ekspedisi', updatedEkspedisi); + formik.setFieldValue('deliveries', updatedDeliveries); }, [formik] ); - const bulkRemoveEkspedisi = useCallback(() => { - const updatedEkspedisi = - formik.values.ekspedisi?.filter( - (_, idx) => !selectedEkspedisi.includes(idx) + const bulkRemoveDelivery = useCallback(() => { + const updatedDeliveries = + formik.values.deliveries?.filter( + (_, idx) => !selectedDeliveries.includes(idx) ) ?? []; - formik.setFieldValue('ekspedisi', updatedEkspedisi); - setSelectedEkspedisi([]); - }, [formik, selectedEkspedisi]); + formik.setFieldValue('deliveries', updatedDeliveries); + setSelectedDeliveries([]); + }, [formik, selectedDeliveries]); - const isRepeaterInputError = ( + const isRepeaterInputError = ( arrayName: T, - column: T extends 'product' ? keyof ProductSchema : keyof EkspedisiSchema, + column: T extends 'products' ? keyof ProductSchema : keyof DeliverySchema, idx: number ) => { if ( @@ -260,57 +261,80 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { ? suppliers?.data.map((s) => ({ value: s.id, label: s.name })) : []; - const { setValues: formikSetValues } = formik; - useEffect(() => { - formikSetValues(formikInitialValues); - }, [formikSetValues, formikInitialValues]); - - useEffect(() => { - formik.values.ekspedisi?.forEach((eks, idx) => { - if (eks.qty && eks.biaya_ekspedisi) { - const perItem = eks.biaya_ekspedisi / eks.qty; + formik.values.deliveries?.forEach((delivery, idx) => { + const productQty = delivery.products.reduce( + (sum, p) => sum + p.product_qty, + 0 + ); + if (productQty && delivery.delivery_cost) { + const perItem = delivery.delivery_cost / productQty; formik.setFieldValue( - `ekspedisi.${idx}.biaya_ekspedisi_per_item`, + `deliveries.${idx}.delivery_cost_per_item`, perItem ); } }); - }, [formik.values.ekspedisi]); + }, [formik.values.deliveries]); const getFilteredProductOptions = useCallback(() => { return ( - formik.values.product + formik.values.products ?.filter((p) => p.product) .map((p) => ({ value: p.product_id, label: (p.product as OptionType)?.label, })) ?? [] ); - }, [formik.values.product]); + }, [formik.values.products]); - const validateEkspedisiQty = (ekspedisiIdx: number, qty: number) => { - const productId = formik.values.ekspedisi?.[ekspedisiIdx]?.product_id; - if (!productId) return true; - const relatedProduct = formik.values.product?.find( - (p) => p.product_id === productId - ); - if (!relatedProduct) return true; - const totalQtyUsed = - formik.values.ekspedisi?.reduce((total, eks, i) => { - if (eks.product_id === productId && i !== ekspedisiIdx) { - return total + (Number(eks.qty) || 0); - } - return total; - }, 0) || 0; - return totalQtyUsed + qty <= Number(relatedProduct.qty_product); - }; + const validateDeliveryQty = useCallback( + (deliveryIdx: number, deliveryProductIdx: number, qty: number) => { + const delivery = formik.values.deliveries?.[deliveryIdx]; + if (!delivery) return true; - const invalidQtyRows = - formik.values.ekspedisi?.map((eks, idx) => { - const qty = Number(eks.qty) || 0; - return !validateEkspedisiQty(idx, qty); - }) ?? []; + const deliveryProduct = delivery.products[deliveryProductIdx]; + if (!deliveryProduct) return true; + + const productId = deliveryProduct.product_id; + if (!productId) return true; + + const relatedProduct = formik.values.products?.find( + (p) => p.product_id === productId + ); + if (!relatedProduct) return true; + + const totalQtyUsed = + formik.values.deliveries?.reduce((total, d) => { + const productQty = d.products.reduce((sum, p, pIdx) => { + if ( + p.product_id === productId && + !(d === delivery && pIdx === deliveryProductIdx) + ) { + return sum + (Number(p.product_qty) || 0); + } + return sum; + }, 0); + return total + productQty; + }, 0) || 0; + + return totalQtyUsed + qty <= Number(relatedProduct.product_qty); + }, + [formik.values.deliveries, formik.values.products] + ); + + const invalidQtyRows = useMemo( + () => + formik.values.deliveries?.flatMap((delivery, deliveryIdx) => + delivery.products.map((product, productIdx) => { + const qty = Number(product.product_qty) || 0; + return !validateDeliveryQty(deliveryIdx, productIdx, qty); + }) + ) ?? [], + [formik.values.deliveries, formik.values.products, validateDeliveryQty] + ); + + const hasInvalidQty = invalidQtyRows.some(Boolean); return ( <> @@ -332,30 +356,30 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { @@ -370,11 +394,11 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { { - formik.setFieldValue('warehouse_asal', val); + formik.setFieldValue('source_warehouse', val); formik.setFieldValue( - 'warehouse_asal_id', + 'source_warehouse_id', (val as WarehouseOptionType)?.value ); }} @@ -382,10 +406,10 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { onInputChange={setWarehouseSelectInputValue} isLoading={isLoadingWarehouses} isError={ - formik.touched.warehouse_asal_id && - Boolean(formik.errors.warehouse_asal_id) + formik.touched.source_warehouse_id && + Boolean(formik.errors.source_warehouse_id) } - errorMessage={formik.errors.warehouse_asal_id as string} + errorMessage={formik.errors.source_warehouse_id as string} isDisabled={type === 'detail'} isClearable /> @@ -394,9 +418,9 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => {
{ /> { { - formik.setFieldValue('warehouse_tujuan', val); + formik.setFieldValue('destination_warehouse', val); formik.setFieldValue( - 'warehouse_tujuan_id', + 'destination_warehouse_id', (val as WarehouseOptionType)?.value ); }} @@ -440,10 +464,12 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { onInputChange={setWarehouseSelectInputValue} isLoading={isLoadingWarehouses} isError={ - formik.touched.warehouse_tujuan_id && - Boolean(formik.errors.warehouse_tujuan_id) + formik.touched.destination_warehouse_id && + Boolean(formik.errors.destination_warehouse_id) + } + errorMessage={ + formik.errors.destination_warehouse_id as string } - errorMessage={formik.errors.warehouse_tujuan_id as string} isDisabled={type === 'detail'} isClearable /> @@ -452,10 +478,12 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => {
{ /> { type='checkbox' className='checkbox' checked={ - formik.values.product?.length === + formik.values.products?.length === selectedProducts.length && - formik.values.product?.length > 0 + formik.values.products?.length > 0 } onChange={(e) => { if (e.target.checked) { setSelectedProducts( - formik.values.product?.map((_, idx) => idx) ?? - [] + formik.values.products?.map( + (_, idx) => idx + ) ?? [] ); } else { setSelectedProducts([]); @@ -518,7 +549,7 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => {
@@ -547,11 +578,11 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { value={product.product ?? undefined} onChange={(val) => { formik.setFieldValue( - `product.${idx}.product`, + `products.${idx}.product`, val ); formik.setFieldValue( - `product.${idx}.product_id`, + `products.${idx}.product_id`, (val as OptionType)?.value ); }} @@ -560,20 +591,24 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { isLoading={isLoadingProducts} isDisabled={type === 'detail'} isClearable - {...isRepeaterInputError('product', 'product', idx)} + {...isRepeaterInputError( + 'products', + 'product', + idx + )} /> { - {/* Ekspedisi table */} + {/* Deliveries table */}
-

Ekspedisi

+

Pengiriman

@@ -647,19 +682,19 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { type='checkbox' className='checkbox' checked={ - formik.values.ekspedisi?.length === - selectedEkspedisi.length && - formik.values.ekspedisi?.length > 0 + formik.values.deliveries?.length === + selectedDeliveries.length && + formik.values.deliveries?.length > 0 } onChange={(e) => { if (e.target.checked) { - setSelectedEkspedisi( - formik.values.ekspedisi?.map( + setSelectedDeliveries( + formik.values.deliveries?.map( (_, idx) => idx ) ?? [] ); } else { - setSelectedEkspedisi([]); + setSelectedDeliveries([]); } }} /> @@ -669,34 +704,31 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { - - - + + {type !== 'detail' && } - {formik.values.ekspedisi?.map((ekspedisi, idx) => ( - + {formik.values.deliveries?.map((delivery, idx) => ( + {type !== 'detail' && ( - {type !== 'detail' && ( @@ -911,7 +880,7 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { )} )} @@ -968,7 +936,7 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { : undefined } onDelete={deleteMovementClickHandler} - disableSubmit={invalidQtyRows.some(Boolean)} + disableSubmit={hasInvalidQty} /> {movementFormErrorMessage && ( diff --git a/src/types/api/inventory/movement.d.ts b/src/types/api/inventory/movement.d.ts index d7f2776a..11da41a5 100644 --- a/src/types/api/inventory/movement.d.ts +++ b/src/types/api/inventory/movement.d.ts @@ -5,47 +5,50 @@ import { Warehouse } from '@/types/api/master-data/warehouse'; export type BaseMovement = { id: number; - alasan_transfer: string; - tanggal_transfer: string; - warehouse_asal: Warehouse; - warehouse_tujuan: Warehouse; - product: { + transfer_reason: string; + transfer_date: string; + source_warehouse: Warehouse; + destination_warehouse: Warehouse; + products: { product: Product; - qty_product: number; + product_qty: number; }[]; - ekspedisi: { - product_id: number; - qty: number; + deliveries: { + delivery_cost: number; + delivery_cost_per_item: number; + document: string; + driver_name: string; + vehicle_plate: string; supplier: Supplier; - plat_nomor: string; - no_surat_jalan: string; - dokumen: string; - biaya_ekspedisi: number; - nama_sopir: string; + products: { + product: Product; + product_qty: number; + }[]; }[]; }; export type Movement = BaseMetadata & BaseMovement; export type CreateMovementPayload = { - alasan_transfer: string; - tanggal_transfer: string; - warehouse_asal_id: number; - warehouse_tujuan_id: number; - product: { + transfer_reason: string; + transfer_date: string; + source_warehouse_id: number; + destination_warehouse_id: number; + products: { product_id: number; - qty_product: number; + product_qty: number; }[]; - ekspedisi: { - product_id: number; - qty: number; + deliveries: { + delivery_cost: number; + delivery_cost_per_item: number; + document: string | File; + driver_name: string; + vehicle_plate: string; supplier_id: number; - plat_nomor: string; - no_surat_jalan: string; - dokumen: string | File; - biaya_ekspedisi: number; - biaya_ekspedisi_per_item?: number; - nama_sopir: string; + products: { + product_id: number; + product_qty: number; + }[]; }[]; }; From 06dc869b846c396d568a58aa1d9cdb695c8be3d5 Mon Sep 17 00:00:00 2001 From: rstubryan Date: Wed, 15 Oct 2025 12:08:52 +0700 Subject: [PATCH 26/55] feat(FE-64): update MovementTable structure for improved data clarity and consistency --- .../inventory/movement/MovementTable.tsx | 244 +++++++++++++----- 1 file changed, 178 insertions(+), 66 deletions(-) diff --git a/src/components/pages/inventory/movement/MovementTable.tsx b/src/components/pages/inventory/movement/MovementTable.tsx index b39906d3..a00f7111 100644 --- a/src/components/pages/inventory/movement/MovementTable.tsx +++ b/src/components/pages/inventory/movement/MovementTable.tsx @@ -31,23 +31,23 @@ const dummyMovements: Movement[] = [ { ...baseMetadata, id: 1, - alasan_transfer: 'Restock', - tanggal_transfer: '2024-06-01', - warehouse_asal: { + transfer_reason: 'Restock', + transfer_date: '2024-06-01', + source_warehouse: { ...baseMetadata, id: 1, name: 'Warehouse A', type: 'AREA', area: { id: 1, name: 'Area 1' }, }, - warehouse_tujuan: { + destination_warehouse: { ...baseMetadata, id: 2, name: 'Warehouse B', type: 'AREA', area: { id: 2, name: 'Area 2' }, }, - product: [ + products: [ { product: { ...baseMetadata, @@ -73,13 +73,16 @@ const dummyMovements: Movement[] = [ suppliers: [], flags: [], }, - qty_product: 10, + product_qty: 10, }, ], - ekspedisi: [ + deliveries: [ { - product_id: 1, - qty: 10, + delivery_cost: 50000, + delivery_cost_per_item: 5000, + document: 'doc1.pdf', + driver_name: 'Andi', + vehicle_plate: 'B 1234 CD', supplier: { ...baseMetadata, id: 1, @@ -97,34 +100,58 @@ const dummyMovements: Movement[] = [ balance: 0, due_date: 30, }, - plat_nomor: 'B 1234 CD', - no_surat_jalan: 'SJ-001', - dokumen: 'doc1.pdf', - biaya_ekspedisi: 50000, - nama_sopir: 'Andi', + products: [ + { + product: { + ...baseMetadata, + id: 1, + name: 'Product X', + brand: 'Brand X', + sku: 'SKU-X', + product_price: 10000, + selling_price: 12000, + tax: 10, + expiry_period: 365, + uom: { + ...baseMetadata, + id: 1, + name: 'PCS', + }, + product_category: { + ...baseMetadata, + id: 1, + code: 'CAT-1', + name: 'Category 1', + }, + suppliers: [], + flags: [], + }, + product_qty: 10, + }, + ], }, ], }, { ...baseMetadata, id: 2, - alasan_transfer: 'Mutasi Stok', - tanggal_transfer: '2024-06-02', - warehouse_asal: { + transfer_reason: 'Mutasi Stok', + transfer_date: '2024-06-02', + source_warehouse: { ...baseMetadata, id: 2, name: 'Warehouse B', type: 'AREA', area: { id: 2, name: 'Area 2' }, }, - warehouse_tujuan: { + destination_warehouse: { ...baseMetadata, id: 3, name: 'Warehouse C', type: 'AREA', area: { id: 3, name: 'Area 3' }, }, - product: [ + products: [ { product: { ...baseMetadata, @@ -150,13 +177,16 @@ const dummyMovements: Movement[] = [ suppliers: [], flags: [], }, - qty_product: 5, + product_qty: 5, }, ], - ekspedisi: [ + deliveries: [ { - product_id: 2, - qty: 5, + delivery_cost: 60000, + delivery_cost_per_item: 12000, + document: 'doc2.pdf', + driver_name: 'Budi', + vehicle_plate: 'D 5678 EF', supplier: { ...baseMetadata, id: 2, @@ -174,34 +204,58 @@ const dummyMovements: Movement[] = [ balance: 1000, due_date: 15, }, - plat_nomor: 'D 5678 EF', - no_surat_jalan: 'SJ-002', - dokumen: 'doc2.pdf', - biaya_ekspedisi: 60000, - nama_sopir: 'Budi', + products: [ + { + product: { + ...baseMetadata, + id: 2, + name: 'Product Y', + brand: 'Brand Y', + sku: 'SKU-Y', + product_price: 20000, + selling_price: 25000, + tax: 5, + expiry_period: 180, + uom: { + ...baseMetadata, + id: 2, + name: 'BOX', + }, + product_category: { + ...baseMetadata, + id: 2, + code: 'CAT-2', + name: 'Category 2', + }, + suppliers: [], + flags: [], + }, + product_qty: 5, + }, + ], }, ], }, { ...baseMetadata, id: 3, - alasan_transfer: 'Pengembalian', - tanggal_transfer: '2024-06-03', - warehouse_asal: { + transfer_reason: 'Pengembalian', + transfer_date: '2024-06-03', + source_warehouse: { ...baseMetadata, id: 3, name: 'Warehouse C', type: 'AREA', area: { id: 3, name: 'Area 3' }, }, - warehouse_tujuan: { + destination_warehouse: { ...baseMetadata, id: 1, name: 'Warehouse A', type: 'AREA', area: { id: 1, name: 'Area 1' }, }, - product: [ + products: [ { product: { ...baseMetadata, @@ -227,13 +281,16 @@ const dummyMovements: Movement[] = [ suppliers: [], flags: [], }, - qty_product: 8, + product_qty: 8, }, ], - ekspedisi: [ + deliveries: [ { - product_id: 3, - qty: 8, + delivery_cost: 40000, + delivery_cost_per_item: 5000, + document: 'doc3.pdf', + driver_name: 'Cici', + vehicle_plate: 'F 9101 GH', supplier: { ...baseMetadata, id: 3, @@ -251,34 +308,58 @@ const dummyMovements: Movement[] = [ balance: 500, due_date: 10, }, - plat_nomor: 'F 9101 GH', - no_surat_jalan: 'SJ-003', - dokumen: 'doc3.pdf', - biaya_ekspedisi: 40000, - nama_sopir: 'Cici', + products: [ + { + product: { + ...baseMetadata, + id: 3, + name: 'Product Z', + brand: 'Brand Z', + sku: 'SKU-Z', + product_price: 15000, + selling_price: 18000, + tax: 8, + expiry_period: 90, + uom: { + ...baseMetadata, + id: 3, + name: 'KG', + }, + product_category: { + ...baseMetadata, + id: 3, + code: 'CAT-3', + name: 'Category 3', + }, + suppliers: [], + flags: [], + }, + product_qty: 8, + }, + ], }, ], }, { ...baseMetadata, id: 4, - alasan_transfer: 'Transfer Internal', - tanggal_transfer: '2024-06-04', - warehouse_asal: { + transfer_reason: 'Transfer Internal', + transfer_date: '2024-06-04', + source_warehouse: { ...baseMetadata, id: 4, name: 'Warehouse D', type: 'AREA', area: { id: 4, name: 'Area 4' }, }, - warehouse_tujuan: { + destination_warehouse: { ...baseMetadata, id: 5, name: 'Warehouse E', type: 'AREA', area: { id: 5, name: 'Area 5' }, }, - product: [ + products: [ { product: { ...baseMetadata, @@ -304,13 +385,16 @@ const dummyMovements: Movement[] = [ suppliers: [], flags: [], }, - qty_product: 20, + product_qty: 20, }, ], - ekspedisi: [ + deliveries: [ { - product_id: 4, - qty: 20, + delivery_cost: 30000, + delivery_cost_per_item: 1500, + document: 'doc4.pdf', + driver_name: 'Dedi', + vehicle_plate: 'H 2345 IJ', supplier: { ...baseMetadata, id: 4, @@ -328,11 +412,35 @@ const dummyMovements: Movement[] = [ balance: 200, due_date: 20, }, - plat_nomor: 'H 2345 IJ', - no_surat_jalan: 'SJ-004', - dokumen: 'doc4.pdf', - biaya_ekspedisi: 30000, - nama_sopir: 'Dedi', + products: [ + { + product: { + ...baseMetadata, + id: 4, + name: 'Product A', + brand: 'Brand A', + sku: 'SKU-A', + product_price: 5000, + selling_price: 7000, + tax: 0, + expiry_period: 60, + uom: { + ...baseMetadata, + id: 4, + name: 'LITER', + }, + product_category: { + ...baseMetadata, + id: 4, + code: 'CAT-4', + name: 'Category 4', + }, + suppliers: [], + flags: [], + }, + product_qty: 20, + }, + ], }, ], }, @@ -401,30 +509,34 @@ const MovementTable = () => { cell: (props) => pageSize * (page - 1) + props.row.index + 1, }, { - accessorKey: 'warehouse_asal', + accessorKey: 'source_warehouse', header: 'Gudang Asal', - cell: (props) => props.row.original.warehouse_asal.name, + cell: (props) => props.row.original.source_warehouse.name, }, { - accessorKey: 'warehouse_tujuan', + accessorKey: 'destination_warehouse', header: 'Gudang Tujuan', - cell: (props) => props.row.original.warehouse_tujuan.name, + cell: (props) => props.row.original.destination_warehouse.name, }, { - accessorKey: 'product', + accessorKey: 'products', header: 'Nama Produk', cell: (props) => - props.row.original.product.map((p) => p.product.name), + props.row.original.products + .map((p) => p.product.name) + .join(', '), }, { - accessorKey: 'alasan_transfer', + accessorKey: 'transfer_reason', header: 'Catatan', }, { - accessorKey: 'biaya_ekspedisi', - header: 'Biaya Ekspedisi', + accessorKey: 'delivery_cost', + header: 'Biaya Pengiriman', cell: (props) => - props.row.original.ekspedisi.map((e) => e.biaya_ekspedisi), + props.row.original.deliveries + .reduce((sum, d) => sum + d.delivery_cost, 0) + .toLocaleString('id-ID'), }, { id: 'actions', From aa21088e99cf25be75961872e1d4d6c795b16af8 Mon Sep 17 00:00:00 2001 From: rstubryan Date: Wed, 15 Oct 2025 12:30:59 +0700 Subject: [PATCH 27/55] feat(FE-62): enhance MovementForm with delivery product input error handling and validation --- .../inventory/movement/form/MovementForm.tsx | 86 ++++++++++++++++++- 1 file changed, 83 insertions(+), 3 deletions(-) diff --git a/src/components/pages/inventory/movement/form/MovementForm.tsx b/src/components/pages/inventory/movement/form/MovementForm.tsx index 183fe760..1af77cf7 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.tsx +++ b/src/components/pages/inventory/movement/form/MovementForm.tsx @@ -214,6 +214,32 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { }; }; + const isDeliveryProductInputError = ( + deliveryIdx: number, + productIdx: number, + column: keyof DeliverySchema['products'][number] + ) => { + const touchedDelivery = formik.touched.deliveries?.[deliveryIdx]; + const errorDelivery = formik.errors.deliveries?.[deliveryIdx] as + | { products: Array> } + | undefined; + + if (!touchedDelivery?.products || !errorDelivery?.products) { + return { + isError: false, + errorMessage: undefined, + }; + } + + const touchedField = touchedDelivery.products[productIdx]?.[column]; + const errorField = errorDelivery.products[productIdx]?.[column]; + + return { + isError: Boolean(touchedField && errorField), + errorMessage: touchedField ? errorField : undefined, + }; + }; + interface WarehouseOptionType extends OptionType { area?: string; location?: string; @@ -305,11 +331,11 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { if (!relatedProduct) return true; const totalQtyUsed = - formik.values.deliveries?.reduce((total, d) => { + formik.values.deliveries?.reduce((total, d, dIdx) => { const productQty = d.products.reduce((sum, p, pIdx) => { if ( p.product_id === productId && - !(d === delivery && pIdx === deliveryProductIdx) + !(dIdx === deliveryIdx && pIdx === deliveryProductIdx) ) { return sum + (Number(p.product_qty) || 0); } @@ -323,6 +349,47 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { [formik.values.deliveries, formik.values.products] ); + const getDeliveryQtyError = useCallback( + (deliveryIdx: number, deliveryProductIdx: number) => { + const delivery = formik.values.deliveries?.[deliveryIdx]; + if (!delivery) return null; + + const deliveryProduct = delivery.products[deliveryProductIdx]; + if (!deliveryProduct || !deliveryProduct.product_id) return null; + + const qty = Number(deliveryProduct.product_qty) || 0; + const productId = deliveryProduct.product_id; + + const relatedProduct = formik.values.products?.find( + (p) => p.product_id === productId + ); + if (!relatedProduct) return null; + + const totalQtyUsed = + formik.values.deliveries?.reduce((total, d, dIdx) => { + const productQty = d.products.reduce((sum, p, pIdx) => { + if ( + p.product_id === productId && + !(dIdx === deliveryIdx && pIdx === deliveryProductIdx) + ) { + return sum + (Number(p.product_qty) || 0); + } + return sum; + }, 0); + return total + productQty; + }, 0) || 0; + + const availableQty = Number(relatedProduct.product_qty) - totalQtyUsed; + + if (totalQtyUsed + qty > Number(relatedProduct.product_qty)) { + return `Qty melebihi stok produk! Tersedia: ${availableQty}, Total digunakan: ${totalQtyUsed + qty}`; + } + + return null; + }, + [formik.values.deliveries, formik.values.products] + ); + const invalidQtyRows = useMemo( () => formik.values.deliveries?.flatMap((delivery, deliveryIdx) => @@ -334,7 +401,10 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { [formik.values.deliveries, formik.values.products, validateDeliveryQty] ); - const hasInvalidQty = invalidQtyRows.some(Boolean); + const hasInvalidQty = useMemo( + () => invalidQtyRows.some(Boolean), + [invalidQtyRows] + ); return ( <> @@ -762,6 +832,16 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { value={delivery.products[0]?.product_qty ?? ''} onChange={formik.handleChange} onBlur={formik.handleBlur} + isError={ + isDeliveryProductInputError(idx, 0, 'product_qty') + .isError || Boolean(getDeliveryQtyError(idx, 0)) + } + errorMessage={ + isDeliveryProductInputError(idx, 0, 'product_qty') + .errorMessage || + getDeliveryQtyError(idx, 0) || + undefined + } readOnly={type === 'detail'} className={{ wrapper: 'w-full min-w-24', From eb0f04310ebe8f655f192b4d17dae7f17d4abf32 Mon Sep 17 00:00:00 2001 From: ValdiANS Date: Thu, 16 Oct 2025 10:01:00 +0700 Subject: [PATCH 28/55] chore(FE-91): create daisyui.css file for extending daisyUI style --- src/app/globals.css | 1 + src/styles/daisyui.css | 11 +++++++++++ 2 files changed, 12 insertions(+) create mode 100644 src/styles/daisyui.css diff --git a/src/app/globals.css b/src/app/globals.css index 386e7620..79af241b 100644 --- a/src/app/globals.css +++ b/src/app/globals.css @@ -1,5 +1,6 @@ @import 'tailwindcss'; @plugin "daisyui"; +@import '../styles/daisyui.css'; :root { --color-primary: #1f74bf; diff --git a/src/styles/daisyui.css b/src/styles/daisyui.css new file mode 100644 index 00000000..9a148fb4 --- /dev/null +++ b/src/styles/daisyui.css @@ -0,0 +1,11 @@ +@layer utilities { + .step.step-success::before { + --step-bg: var(--color-success); + --step-fg: var(--color-success-content); + } + + .step.step-error::before { + --step-bg: var(--color-error); + --step-fg: var(--color-error-content); + } +} From 156de6112bb88d62cf41054b738a764ed024c94a Mon Sep 17 00:00:00 2001 From: ValdiANS Date: Thu, 16 Oct 2025 10:01:14 +0700 Subject: [PATCH 29/55] feat(FE-91): create Tooltip component --- src/components/Tooltip.tsx | 60 ++++++++++++++++++++++++++++++++++++++ 1 file changed, 60 insertions(+) create mode 100644 src/components/Tooltip.tsx diff --git a/src/components/Tooltip.tsx b/src/components/Tooltip.tsx new file mode 100644 index 00000000..02f86dca --- /dev/null +++ b/src/components/Tooltip.tsx @@ -0,0 +1,60 @@ +import { ReactNode } from 'react'; + +import { cn } from '@/lib/helper'; +import { Color } from '@/types/theme'; + +interface TooltipProps { + children?: ReactNode; + content?: ReactNode; + className?: { + wrapper?: string; + content?: string; + }; + open?: boolean; + color?: Color; + position?: 'top' | 'bottom' | 'left' | 'right'; +} + +const Tooltip = ({ + children, + content, + className, + open, + color, + position, +}: TooltipProps) => { + const tooltipBaseClassName = cn('tooltip', { + 'tooltip-open': typeof open === 'boolean' && open, + + 'tooltip-top': position === 'top', + 'tooltip-bottom': position === 'bottom', + 'tooltip-left': position === 'left', + 'tooltip-right': position === 'right', + + 'tooltip-primary': color === 'primary', + 'tooltip-secondary': color === 'secondary', + 'tooltip-accent': color === 'accent', + 'tooltip-neutral': color === 'neutral', + 'tooltip-info': color === 'info', + 'tooltip-success': color === 'success', + 'tooltip-warning': color === 'warning', + 'tooltip-error': color === 'error', + }); + return ( +
+
+ {content} +
+ + {children} +
+ ); +}; + +export default Tooltip; From 76dd2e4c54a929103625a20dcc76ceb76d0263ea Mon Sep 17 00:00:00 2001 From: ValdiANS Date: Thu, 16 Oct 2025 10:01:23 +0700 Subject: [PATCH 30/55] feat(FE-91): create Steps component --- src/components/steps/Steps.tsx | 23 +++++++++++++++++++++++ 1 file changed, 23 insertions(+) create mode 100644 src/components/steps/Steps.tsx diff --git a/src/components/steps/Steps.tsx b/src/components/steps/Steps.tsx new file mode 100644 index 00000000..29d307e1 --- /dev/null +++ b/src/components/steps/Steps.tsx @@ -0,0 +1,23 @@ +import { ReactNode } from 'react'; +import { cn } from '@/lib/helper'; + +interface StepsProps { + children?: ReactNode; + className?: string; + direction?: 'horizontal' | 'vertical'; +} + +const Steps = ({ children, className, direction }: StepsProps) => { + const stepsBaseClassName = cn('steps gap-2', { + 'steps-horizontal': direction === 'horizontal', + 'steps-vertical': direction === 'vertical', + }); + + return ( +
    + {children} +
+ ); +}; + +export default Steps; From 0577f6ce1d6875d33af23a691f0d61bf122fef2f Mon Sep 17 00:00:00 2001 From: ValdiANS Date: Thu, 16 Oct 2025 10:01:29 +0700 Subject: [PATCH 31/55] feat(FE-91): create StepItem component --- src/components/steps/StepItem.tsx | 34 +++++++++++++++++++++++++++++++ 1 file changed, 34 insertions(+) create mode 100644 src/components/steps/StepItem.tsx diff --git a/src/components/steps/StepItem.tsx b/src/components/steps/StepItem.tsx new file mode 100644 index 00000000..85ec4f3e --- /dev/null +++ b/src/components/steps/StepItem.tsx @@ -0,0 +1,34 @@ +import { ReactNode } from 'react'; + +import { cn } from '@/lib/helper'; +import { Color } from '@/types/theme'; + +interface StepItemProps { + children?: ReactNode; + icon?: ReactNode; + className?: string; + color?: Color; +} + +const StepItem = ({ children, icon, className, color }: StepItemProps) => { + const stepItemBaseClassName = cn('step', { + 'step-primary': color === 'primary', + 'step-secondary': color === 'secondary', + 'step-accent': color === 'accent', + 'step-neutral': color === 'neutral', + 'step-info': color === 'info', + 'step-success': color === 'success', + 'step-warning': color === 'warning', + 'step-error': color === 'error', + }); + + return ( +
  • + {icon} + +
    {children}
    +
  • + ); +}; + +export default StepItem; From 93beb86f910fe0bcadecb3c4ded5498893c79af1 Mon Sep 17 00:00:00 2001 From: ValdiANS Date: Thu, 16 Oct 2025 10:01:40 +0700 Subject: [PATCH 32/55] feat(FE-91): create ApprovalSteps component --- src/components/pages/ApprovalSteps.tsx | 64 ++++++++++++++++++++++++++ 1 file changed, 64 insertions(+) create mode 100644 src/components/pages/ApprovalSteps.tsx diff --git a/src/components/pages/ApprovalSteps.tsx b/src/components/pages/ApprovalSteps.tsx new file mode 100644 index 00000000..4022e254 --- /dev/null +++ b/src/components/pages/ApprovalSteps.tsx @@ -0,0 +1,64 @@ +import { Icon } from '@iconify/react'; +import Steps from '@/components/steps/Steps'; +import StepItem from '@/components/steps/StepItem'; +import Tooltip from '@/components/Tooltip'; + +import { formatDate } from '@/lib/helper'; +import { ApprovalsLine } from '@/types/api/api-general'; + +interface ApprovalStepsProps { + approvals: ApprovalsLine; +} + +const ApprovalSteps = ({ approvals }: ApprovalStepsProps) => { + return ( + + {approvals.map((approval, idx) => { + const stepItemColor = + approval.status === 'approved' + ? 'success' + : approval.status === 'rejected' + ? 'error' + : undefined; + + const stepItemIcon = + approval.status === 'approved' + ? 'material-symbols:check-rounded' + : approval.status === 'rejected' + ? 'material-symbols:close-rounded' + : 'bxs:hourglass'; + + return ( + + {formatDate(approval.date, 'YYYY-MM-DD')} + Oleh: {approval.action_by} + Catatan: {approval.notes} + + } + > + + + ) + } + > + {approval.role} + + ); + })} + + ); +}; + +export default ApprovalSteps; From b7a30cc73adb2f9c874f8d5539cbf5cfe8d8e8b8 Mon Sep 17 00:00:00 2001 From: ValdiANS Date: Thu, 16 Oct 2025 10:01:50 +0700 Subject: [PATCH 33/55] chore(FE-91): create ApprovalsLine type --- src/types/api/api-general.d.ts | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/src/types/api/api-general.d.ts b/src/types/api/api-general.d.ts index 6a3fc6be..cf3e57f7 100644 --- a/src/types/api/api-general.d.ts +++ b/src/types/api/api-general.d.ts @@ -66,3 +66,11 @@ export type flags = | 'STARTER' | 'FINISHER' | 'OVK'; + +export type ApprovalsLine = { + action_by?: string; + date?: string; + notes?: string; + role?: string; + status: 'approved' | 'rejected' | 'waiting'; +}[]; From c6a0c542aad1dcc2c4dc8c5361bfa95a6d9b01cd Mon Sep 17 00:00:00 2001 From: rstubryan Date: Thu, 16 Oct 2025 14:33:49 +0700 Subject: [PATCH 34/55] refactor(FE-62,63,65): refactor Movement and ProductWarehouse APIs, update MovementForm schema, and enhance MovementTable functionality --- .../inventory/movement/MovementTable.tsx | 622 ++++-------------- .../movement/form/MovementForm.schema.ts | 53 +- .../movement/form/useMovementFormHandlers.ts | 80 ++- src/components/table/TableRowOptions.tsx | 11 +- src/services/api/inventory.ts | 13 +- src/types/api/inventory/movement.d.ts | 32 +- .../api/inventory/product-warehouse.d.ts | 22 + 7 files changed, 277 insertions(+), 556 deletions(-) create mode 100644 src/types/api/inventory/product-warehouse.d.ts diff --git a/src/components/pages/inventory/movement/MovementTable.tsx b/src/components/pages/inventory/movement/MovementTable.tsx index a00f7111..e0bc9541 100644 --- a/src/components/pages/inventory/movement/MovementTable.tsx +++ b/src/components/pages/inventory/movement/MovementTable.tsx @@ -1,463 +1,56 @@ 'use client'; -import { useState, useMemo } from 'react'; +import { useState } from 'react'; +import useSWR from 'swr'; import { SortingState } from '@tanstack/react-table'; -import { Icon } from '@iconify/react'; + import Table from '@/components/Table'; import { useModal } from '@/components/Modal'; import ConfirmationModal from '@/components/modal/ConfirmationModal'; -import { ROWS_OPTIONS } from '@/config/constant'; import { Movement } from '@/types/api/inventory/movement'; +import { MovementApi } from '@/services/api/inventory'; +import { cn } from '@/lib/helper'; +import { isResponseSuccess } from '@/lib/api-helper'; +import { useTableFilter } from '@/services/hooks/useTableFilter'; +import { ROWS_OPTIONS } from '@/config/constant'; import { TableToolbar } from '@/components/table/TableToolbar'; import { TableRowSizeSelector } from '@/components/table/TableRowSizeSelector'; -import { TableRowOptions } from '@/components/table/TableRowOptions'; import { OptionType } from '@/components/input/SelectInput'; -import Button from '@/components/Button'; -import { cn } from '@/lib/helper'; - -// Dummy data -const baseMetadata = { - created_user: { - id: 1, - id_user: 1, - email: 'user@example.com', - name: 'User', - }, - created_at: '2024-06-01T00:00:00Z', - updated_at: '2024-06-01T00:00:00Z', -}; - -const dummyMovements: Movement[] = [ - { - ...baseMetadata, - id: 1, - transfer_reason: 'Restock', - transfer_date: '2024-06-01', - source_warehouse: { - ...baseMetadata, - id: 1, - name: 'Warehouse A', - type: 'AREA', - area: { id: 1, name: 'Area 1' }, - }, - destination_warehouse: { - ...baseMetadata, - id: 2, - name: 'Warehouse B', - type: 'AREA', - area: { id: 2, name: 'Area 2' }, - }, - products: [ - { - product: { - ...baseMetadata, - id: 1, - name: 'Product X', - brand: 'Brand X', - sku: 'SKU-X', - product_price: 10000, - selling_price: 12000, - tax: 10, - expiry_period: 365, - uom: { - ...baseMetadata, - id: 1, - name: 'PCS', - }, - product_category: { - ...baseMetadata, - id: 1, - code: 'CAT-1', - name: 'Category 1', - }, - suppliers: [], - flags: [], - }, - product_qty: 10, - }, - ], - deliveries: [ - { - delivery_cost: 50000, - delivery_cost_per_item: 5000, - document: 'doc1.pdf', - driver_name: 'Andi', - vehicle_plate: 'B 1234 CD', - supplier: { - ...baseMetadata, - id: 1, - name: 'Supplier 1', - alias: 'S1', - category: 'General', - pic: 'PIC 1', - type: 'Type 1', - hatchery: 'Hatchery 1', - phone: '08123456789', - email: 'supplier1@example.com', - address: 'Address 1', - npwp: '1234567890123456', - account_number: '1234567890', - balance: 0, - due_date: 30, - }, - products: [ - { - product: { - ...baseMetadata, - id: 1, - name: 'Product X', - brand: 'Brand X', - sku: 'SKU-X', - product_price: 10000, - selling_price: 12000, - tax: 10, - expiry_period: 365, - uom: { - ...baseMetadata, - id: 1, - name: 'PCS', - }, - product_category: { - ...baseMetadata, - id: 1, - code: 'CAT-1', - name: 'Category 1', - }, - suppliers: [], - flags: [], - }, - product_qty: 10, - }, - ], - }, - ], - }, - { - ...baseMetadata, - id: 2, - transfer_reason: 'Mutasi Stok', - transfer_date: '2024-06-02', - source_warehouse: { - ...baseMetadata, - id: 2, - name: 'Warehouse B', - type: 'AREA', - area: { id: 2, name: 'Area 2' }, - }, - destination_warehouse: { - ...baseMetadata, - id: 3, - name: 'Warehouse C', - type: 'AREA', - area: { id: 3, name: 'Area 3' }, - }, - products: [ - { - product: { - ...baseMetadata, - id: 2, - name: 'Product Y', - brand: 'Brand Y', - sku: 'SKU-Y', - product_price: 20000, - selling_price: 25000, - tax: 5, - expiry_period: 180, - uom: { - ...baseMetadata, - id: 2, - name: 'BOX', - }, - product_category: { - ...baseMetadata, - id: 2, - code: 'CAT-2', - name: 'Category 2', - }, - suppliers: [], - flags: [], - }, - product_qty: 5, - }, - ], - deliveries: [ - { - delivery_cost: 60000, - delivery_cost_per_item: 12000, - document: 'doc2.pdf', - driver_name: 'Budi', - vehicle_plate: 'D 5678 EF', - supplier: { - ...baseMetadata, - id: 2, - name: 'Supplier 2', - alias: 'S2', - category: 'Special', - pic: 'PIC 2', - type: 'Type 2', - hatchery: 'Hatchery 2', - phone: '08123456780', - email: 'supplier2@example.com', - address: 'Address 2', - npwp: '1234567890123457', - account_number: '1234567891', - balance: 1000, - due_date: 15, - }, - products: [ - { - product: { - ...baseMetadata, - id: 2, - name: 'Product Y', - brand: 'Brand Y', - sku: 'SKU-Y', - product_price: 20000, - selling_price: 25000, - tax: 5, - expiry_period: 180, - uom: { - ...baseMetadata, - id: 2, - name: 'BOX', - }, - product_category: { - ...baseMetadata, - id: 2, - code: 'CAT-2', - name: 'Category 2', - }, - suppliers: [], - flags: [], - }, - product_qty: 5, - }, - ], - }, - ], - }, - { - ...baseMetadata, - id: 3, - transfer_reason: 'Pengembalian', - transfer_date: '2024-06-03', - source_warehouse: { - ...baseMetadata, - id: 3, - name: 'Warehouse C', - type: 'AREA', - area: { id: 3, name: 'Area 3' }, - }, - destination_warehouse: { - ...baseMetadata, - id: 1, - name: 'Warehouse A', - type: 'AREA', - area: { id: 1, name: 'Area 1' }, - }, - products: [ - { - product: { - ...baseMetadata, - id: 3, - name: 'Product Z', - brand: 'Brand Z', - sku: 'SKU-Z', - product_price: 15000, - selling_price: 18000, - tax: 8, - expiry_period: 90, - uom: { - ...baseMetadata, - id: 3, - name: 'KG', - }, - product_category: { - ...baseMetadata, - id: 3, - code: 'CAT-3', - name: 'Category 3', - }, - suppliers: [], - flags: [], - }, - product_qty: 8, - }, - ], - deliveries: [ - { - delivery_cost: 40000, - delivery_cost_per_item: 5000, - document: 'doc3.pdf', - driver_name: 'Cici', - vehicle_plate: 'F 9101 GH', - supplier: { - ...baseMetadata, - id: 3, - name: 'Supplier 3', - alias: 'S3', - category: 'Return', - pic: 'PIC 3', - type: 'Type 3', - hatchery: 'Hatchery 3', - phone: '08123456781', - email: 'supplier3@example.com', - address: 'Address 3', - npwp: '1234567890123458', - account_number: '1234567892', - balance: 500, - due_date: 10, - }, - products: [ - { - product: { - ...baseMetadata, - id: 3, - name: 'Product Z', - brand: 'Brand Z', - sku: 'SKU-Z', - product_price: 15000, - selling_price: 18000, - tax: 8, - expiry_period: 90, - uom: { - ...baseMetadata, - id: 3, - name: 'KG', - }, - product_category: { - ...baseMetadata, - id: 3, - code: 'CAT-3', - name: 'Category 3', - }, - suppliers: [], - flags: [], - }, - product_qty: 8, - }, - ], - }, - ], - }, - { - ...baseMetadata, - id: 4, - transfer_reason: 'Transfer Internal', - transfer_date: '2024-06-04', - source_warehouse: { - ...baseMetadata, - id: 4, - name: 'Warehouse D', - type: 'AREA', - area: { id: 4, name: 'Area 4' }, - }, - destination_warehouse: { - ...baseMetadata, - id: 5, - name: 'Warehouse E', - type: 'AREA', - area: { id: 5, name: 'Area 5' }, - }, - products: [ - { - product: { - ...baseMetadata, - id: 4, - name: 'Product A', - brand: 'Brand A', - sku: 'SKU-A', - product_price: 5000, - selling_price: 7000, - tax: 0, - expiry_period: 60, - uom: { - ...baseMetadata, - id: 4, - name: 'LITER', - }, - product_category: { - ...baseMetadata, - id: 4, - code: 'CAT-4', - name: 'Category 4', - }, - suppliers: [], - flags: [], - }, - product_qty: 20, - }, - ], - deliveries: [ - { - delivery_cost: 30000, - delivery_cost_per_item: 1500, - document: 'doc4.pdf', - driver_name: 'Dedi', - vehicle_plate: 'H 2345 IJ', - supplier: { - ...baseMetadata, - id: 4, - name: 'Supplier 4', - alias: 'S4', - category: 'Internal', - pic: 'PIC 4', - type: 'Type 4', - hatchery: 'Hatchery 4', - phone: '08123456782', - email: 'supplier4@example.com', - address: 'Address 4', - npwp: '1234567890123459', - account_number: '1234567893', - balance: 200, - due_date: 20, - }, - products: [ - { - product: { - ...baseMetadata, - id: 4, - name: 'Product A', - brand: 'Brand A', - sku: 'SKU-A', - product_price: 5000, - selling_price: 7000, - tax: 0, - expiry_period: 60, - uom: { - ...baseMetadata, - id: 4, - name: 'LITER', - }, - product_category: { - ...baseMetadata, - id: 4, - code: 'CAT-4', - name: 'Category 4', - }, - suppliers: [], - flags: [], - }, - product_qty: 20, - }, - ], - }, - ], - }, -]; +import RowDropdownOptions from '@/components/table/RowDropdownOptions'; +import RowCollapseOptions from '@/components/table/RowCollapseOptions'; +import { TableRowOptions } from '@/components/table/TableRowOptions'; const MovementTable = () => { - const [search, setSearch] = useState(''); - const [page, setPage] = useState(1); - const [pageSize, setPageSize] = useState(10); + const { + state: tableFilterState, + updateFilter, + setPage, + setPageSize, + toQueryString: getTableFilterQueryString, + } = useTableFilter({ + initial: { search: '' }, + paramMap: { page: 'page', pageSize: 'limit' }, + }); + const [sorting, setSorting] = useState([]); - const [, setSelectedMovement] = useState(undefined); + const [selectedMovement, setSelectedMovement] = useState< + Movement | undefined + >(undefined); const [isDeleteLoading, setIsDeleteLoading] = useState(false); const deleteModal = useModal(); + const { + data: movements, + isLoading, + mutate: refreshMovements, + } = useSWR( + `${MovementApi.basePath}${getTableFilterQueryString()}`, + MovementApi.getAllFetcher + ); + const searchChangeHandler = (e: React.ChangeEvent) => { - setSearch(e.target.value); + updateFilter('search', e.target.value); setPage(1); }; @@ -469,17 +62,15 @@ const MovementTable = () => { const confirmationModalDeleteClickHandler = async () => { setIsDeleteLoading(true); - setTimeout(() => { - setIsDeleteLoading(false); + try { + await MovementApi.delete(selectedMovement?.id as number); + refreshMovements(); deleteModal.closeModal(); - }, 1000); + } finally { + setIsDeleteLoading(false); + } }; - const paginatedData = useMemo(() => { - const start = (page - 1) * pageSize; - return dummyMovements.slice(start, start + pageSize); - }, [page, pageSize]); - return (
    @@ -489,85 +80,118 @@ const MovementTable = () => { label: 'Tambah Movement', }} search={{ - value: search, + value: tableFilterState.search, onChange: searchChangeHandler, placeholder: 'Cari Movement', }} />
    -
    Qty Supplier Plat NomorNo Surat Jalan DokumenBiaya Ekspedisi (Rp.)Biaya Ekspedisi / Item (Rp.)Biaya Pengiriman (Rp.)Biaya Per Item (Rp.) Nama SopirAksi
    { if (e.target.checked) { - setSelectedEkspedisi([ - ...selectedEkspedisi, + setSelectedDeliveries([ + ...selectedDeliveries, idx, ]); } else { - setSelectedEkspedisi( - selectedEkspedisi.filter((i) => i !== idx) + setSelectedDeliveries( + selectedDeliveries.filter((i) => i !== idx) ); } }} @@ -706,24 +738,18 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { { formik.setFieldValue( - `ekspedisi.${idx}.product`, + `deliveries.${idx}.products.0.product`, val ); formik.setFieldValue( - `ekspedisi.${idx}.product_id`, + `deliveries.${idx}.products.0.product_id`, (val as OptionType)?.value ); - formik.setFieldValue(`ekspedisi.${idx}.qty`, ''); }} options={getFilteredProductOptions()} - {...isRepeaterInputError( - 'ekspedisi', - 'product', - idx - )} isDisabled={type === 'detail'} isClearable /> @@ -732,11 +758,10 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { { { formik.setFieldValue( - `ekspedisi.${idx}.supplier`, + `deliveries.${idx}.supplier`, val ); formik.setFieldValue( - `ekspedisi.${idx}.supplier_id`, + `deliveries.${idx}.supplier_id`, (val as OptionType)?.value ); }} @@ -762,128 +787,75 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { isLoading={isLoadingSuppliers} isDisabled={type === 'detail'} isClearable - {...isRepeaterInputError( - 'ekspedisi', - 'supplier', - idx - )} /> - - { const file = e.target.files?.[0]; if (file) { - const allowedTypes = [ - 'application/pdf', - 'image/jpeg', - 'image/jpg', - ]; - if (!allowedTypes.includes(file.type)) { - toast.error( - 'Mohon upload file berformat PDF atau JPEG/JPG.' - ); - return; - } if (file.size > 2 * 1024 * 1024) { toast.error('Ukuran dokumen maksimal 2 MB!'); return; } formik.setFieldValue( - `ekspedisi.${idx}.dokumen`, + `deliveries.${idx}.document`, file ); } }} {...isRepeaterInputError( - 'ekspedisi', - 'dokumen', + 'deliveries', + 'document', idx )} readOnly={type === 'detail'} - className={{ - wrapper: 'w-full min-w-24', - }} /> @@ -891,19 +863,16 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => {
    + data={isResponseSuccess(movements) ? movements?.data : []} columns={[ { header: '#', - cell: (props) => pageSize * (page - 1) + props.row.index + 1, - }, - { - accessorKey: 'source_warehouse', - header: 'Gudang Asal', - cell: (props) => props.row.original.source_warehouse.name, - }, - { - accessorKey: 'destination_warehouse', - header: 'Gudang Tujuan', - cell: (props) => props.row.original.destination_warehouse.name, - }, - { - accessorKey: 'products', - header: 'Nama Produk', cell: (props) => - props.row.original.products - .map((p) => p.product.name) - .join(', '), + tableFilterState.pageSize * (tableFilterState.page - 1) + + props.row.index + + 1, + }, + { + accessorFn: (row) => row.source_warehouse?.name, + header: 'Gudang Asal', + }, + { + accessorFn: (row) => row.destination_warehouse?.name, + header: 'Gudang Tujuan', }, { accessorKey: 'transfer_reason', header: 'Catatan', }, { - accessorKey: 'delivery_cost', - header: 'Biaya Pengiriman', + accessorKey: 'transfer_date', + header: 'Tanggal', cell: (props) => - props.row.original.deliveries - .reduce((sum, d) => sum + d.delivery_cost, 0) - .toLocaleString('id-ID'), + new Date(props.row.original.transfer_date).toLocaleDateString( + 'id-ID' + ), }, { - id: 'actions', - cell: (props) => ( -
    - - { - setSelectedMovement(props.row.original); - deleteModal.openModal(); - }} - /> -
    - ), + accessorFn: (row) => { + const totalCost = row.deliveries?.reduce( + (sum, d) => sum + (d.shipping_cost_total || 0), + 0 + ); + return totalCost?.toLocaleString('id-ID'); + }, + header: 'Biaya Pengiriman', + }, + { + header: 'Aksi', + cell: (props) => { + const currentPageSize = + props.table.getPaginationRowModel().rows.length; + const currentPageRows = + props.table.getPaginationRowModel().flatRows; + const currentRowRelativeIndex = + currentPageRows.findIndex((r) => r.id === props.row.id) + 1; + + const isLast2Rows = currentRowRelativeIndex > currentPageSize - 2; + + const deleteClickHandler = () => { + setSelectedMovement(props.row.original); + deleteModal.openModal(); + }; + + return ( + <> + {currentPageSize > 2 && ( + + + + )} + + {currentPageSize <= 2 && ( + + + + )} + + ); + }, }, ]} - pageSize={pageSize} - page={page} - totalItems={dummyMovements.length} + pageSize={tableFilterState.pageSize} + page={isResponseSuccess(movements) ? movements?.meta?.page : 0} + totalItems={ + isResponseSuccess(movements) ? movements?.meta?.total_results : 0 + } onPageChange={setPage} - isLoading={false} + isLoading={isLoading} sorting={sorting} setSorting={setSorting} className={{ containerClassName: cn({ - 'mb-20': paginatedData.length === 0, + 'mb-20': + isResponseSuccess(movements) && movements?.data?.length === 0, }), tableWrapperClassName: 'overflow-x-auto min-h-full!', tableClassName: 'font-inter w-full table-auto min-h-full!', diff --git a/src/components/pages/inventory/movement/form/MovementForm.schema.ts b/src/components/pages/inventory/movement/form/MovementForm.schema.ts index cb0d228d..148a7dce 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.schema.ts +++ b/src/components/pages/inventory/movement/form/MovementForm.schema.ts @@ -13,7 +13,7 @@ export type ProductSchema = { export type DeliverySchema = { delivery_cost: number; delivery_cost_per_item?: number | undefined; - document: string | File; + document?: File | string | null; driver_name: string; vehicle_plate: string; supplier: { @@ -64,23 +64,15 @@ const DeliveryObjectSchema: Yup.ObjectSchema = Yup.object({ .transform((value) => (isNaN(value) ? undefined : value)) .min(0, 'Biaya per item minimal 0!') .typeError('Biaya per item harus berupa angka!'), - document: Yup.mixed() - .required('Dokumen wajib diisi!') - .test( - 'fileType', - 'Mohon upload file berformat PDF atau JPEG/JPG.', - (value) => - typeof value === 'string' || - (value instanceof File && - ['application/pdf', 'image/jpeg', 'image/jpg'].includes(value.type)) - ) - .test( - 'fileSize', - 'Ukuran dokumen maksimal 2 MB!', - (value) => - typeof value === 'string' || - (value instanceof File && value.size <= 2 * 1024 * 1024) - ), + document_index: Yup.number().optional(), + document: Yup.mixed() + .nullable() + .test('fileSize', 'Ukuran dokumen maksimal 2 MB', (value) => { + if (!value) return true; + if (typeof value === 'string') return true; + if (value instanceof File) return value.size <= 2 * 1024 * 1024; + return false; + }), driver_name: Yup.string().required('Nama sopir wajib diisi!'), vehicle_plate: Yup.string().required('Plat nomor wajib diisi!'), supplier: Yup.object({ @@ -145,24 +137,25 @@ export const getMovementFormInitialValues = ( : null, destination_warehouse_id: initialValues?.destination_warehouse?.id ?? 0, products: - initialValues?.products?.map((p) => ({ - product: { value: p.product.id, label: p.product.name }, - product_id: p.product.id, - product_qty: p.product_qty, + initialValues?.details?.map((p) => ({ + product: { value: p.product_id, label: '' }, + product_id: p.product_id, + product_qty: p.quantity, })) ?? [], deliveries: initialValues?.deliveries?.map((d) => ({ - delivery_cost: d.delivery_cost, - delivery_cost_per_item: d.delivery_cost_per_item, - document: d.document, + delivery_cost: d.shipping_cost_total, + delivery_cost_per_item: d.shipping_cost_item, + document_index: 0, + document: d.document_path || null, driver_name: d.driver_name, vehicle_plate: d.vehicle_plate, supplier: { value: d.supplier.id, label: d.supplier.name }, - supplier_id: d.supplier.id, - products: d.products.map((p) => ({ - product: { value: p.product.id, label: p.product.name }, - product_id: p.product.id, - product_qty: p.product_qty, + supplier_id: d.supplier_id, + products: d.items.map((p) => ({ + product: { value: 0, label: '' }, + product_id: 0, + product_qty: p.quantity, })), })) ?? [], }); diff --git a/src/components/pages/inventory/movement/form/useMovementFormHandlers.ts b/src/components/pages/inventory/movement/form/useMovementFormHandlers.ts index 0b6b0962..1894b1a7 100644 --- a/src/components/pages/inventory/movement/form/useMovementFormHandlers.ts +++ b/src/components/pages/inventory/movement/form/useMovementFormHandlers.ts @@ -8,7 +8,6 @@ import { UpdateMovementPayload, } from '@/types/api/inventory/movement'; import { isResponseError } from '@/lib/api-helper'; -import { containsFile, toFormData } from '@/lib/form-data'; export const useMovementFormHandlers = (initialValuesId?: number) => { const router = useRouter(); @@ -17,13 +16,44 @@ export const useMovementFormHandlers = (initialValuesId?: number) => { const [isDeleteLoading, setIsDeleteLoading] = useState(false); const createMovementHandler = useCallback( - async (payload: CreateMovementPayload) => { - const finalPayload = containsFile(payload) - ? (toFormData(payload) as unknown as CreateMovementPayload) - : payload; + async (payload: CreateMovementPayload, documents: File[] = []) => { + console.log('=== CREATE HANDLER DEBUG ==='); + console.log('1. Received payload:', payload); + console.log('2. Documents count:', documents.length); + + let finalPayload: CreateMovementPayload | FormData; + + if (documents.length > 0) { + // Ada dokumen: kirim sebagai FormData dengan "data" field + console.log('3. Creating FormData (has documents)'); + const formData = new FormData(); + formData.append('data', JSON.stringify(payload)); + documents.forEach((file, index) => { + formData.append(`documents[${index}]`, file); + }); + + console.log('4. FormData entries:'); + for (const [key, value] of formData.entries()) { + if (value instanceof File) { + console.log(` ${key}: [File] ${value.name} (${value.size} bytes)`); + } else { + console.log(` ${key}: ${value}`); + } + } + + finalPayload = formData as unknown as CreateMovementPayload; + } else { + // Tidak ada dokumen: kirim sebagai JSON biasa + console.log('3. Sending as JSON (no documents)'); + console.log('4. Payload:', JSON.stringify(payload, null, 2)); + finalPayload = payload; + } + + console.log('=== END CREATE HANDLER DEBUG ==='); const res = await MovementApi.create(finalPayload); if (isResponseError(res)) { + console.error('API Error:', res); setMovementFormErrorMessage(res.message); return; } @@ -34,13 +64,45 @@ export const useMovementFormHandlers = (initialValuesId?: number) => { ); const updateMovementHandler = useCallback( - async (movementId: number, payload: UpdateMovementPayload) => { - const finalPayload = containsFile(payload) - ? (toFormData(payload) as unknown as UpdateMovementPayload) - : payload; + async (movementId: number, payload: UpdateMovementPayload, documents: File[] = []) => { + console.log('=== UPDATE HANDLER DEBUG ==='); + console.log('1. Received payload:', payload); + console.log('2. Movement ID:', movementId); + console.log('3. Documents count:', documents.length); + + let finalPayload: UpdateMovementPayload | FormData; + + if (documents.length > 0) { + // Ada dokumen: kirim sebagai FormData dengan "data" field + console.log('4. Creating FormData (has documents)'); + const formData = new FormData(); + formData.append('data', JSON.stringify(payload)); + documents.forEach((file, index) => { + formData.append(`documents[${index}]`, file); + }); + + console.log('5. FormData entries:'); + for (const [key, value] of formData.entries()) { + if (value instanceof File) { + console.log(` ${key}: [File] ${value.name} (${value.size} bytes)`); + } else { + console.log(` ${key}: ${value}`); + } + } + + finalPayload = formData as unknown as UpdateMovementPayload; + } else { + // Tidak ada dokumen: kirim sebagai JSON biasa + console.log('4. Sending as JSON (no documents)'); + console.log('5. Payload:', JSON.stringify(payload, null, 2)); + finalPayload = payload; + } + + console.log('=== END UPDATE HANDLER DEBUG ==='); const res = await MovementApi.update(movementId, finalPayload); if (res?.status === 'error') { + console.error('API Error:', res); setMovementFormErrorMessage(res.message); return; } diff --git a/src/components/table/TableRowOptions.tsx b/src/components/table/TableRowOptions.tsx index 61332b4f..e34f2ad4 100644 --- a/src/components/table/TableRowOptions.tsx +++ b/src/components/table/TableRowOptions.tsx @@ -7,6 +7,7 @@ interface TableRowOptionsProps { recordId: string | number; basePath: string; onDelete?: () => void; + queryParam?: string; } export const TableRowOptions = ({ @@ -14,6 +15,7 @@ export const TableRowOptions = ({ recordId, basePath, onDelete, + queryParam = 'id', }: TableRowOptionsProps) => (
    {onDelete && ( @@ -51,9 +53,10 @@ export const TableRowOptions = ({ className='text-error hover:text-inherit justify-start text-sm' > Delete diff --git a/src/services/api/inventory.ts b/src/services/api/inventory.ts index cf799442..ec58f6f2 100644 --- a/src/services/api/inventory.ts +++ b/src/services/api/inventory.ts @@ -1,4 +1,9 @@ import { BaseApiService } from '@/services/api/base'; +import { + CreateProductWarehousePayload, + ProductWarehouse, + UpdateProductWarehousePayload, +} from '@/types/api/inventory/product-warehouse'; import { CreateMovementPayload, Movement, @@ -9,11 +14,17 @@ import { InventoryAdjustment, } from '@/types/api/inventory/adjustment'; +export const ProductWarehouseApi = new BaseApiService< + ProductWarehouse, + CreateProductWarehousePayload, + UpdateProductWarehousePayload +>('/inventory/product-warehouses'); + export const MovementApi = new BaseApiService< Movement, CreateMovementPayload, UpdateMovementPayload ->('/inventory/movements'); +>('/inventory/transfers'); export const inventoryAdjustmentApi = new BaseApiService< InventoryAdjustment, diff --git a/src/types/api/inventory/movement.d.ts b/src/types/api/inventory/movement.d.ts index 11da41a5..9e156a1e 100644 --- a/src/types/api/inventory/movement.d.ts +++ b/src/types/api/inventory/movement.d.ts @@ -1,5 +1,4 @@ import { BaseMetadata } from '@/types/api/api-general'; -import { Product } from '@/types/api/master-data/product'; import { Supplier } from '@/types/api/master-data/supplier'; import { Warehouse } from '@/types/api/master-data/warehouse'; @@ -9,20 +8,27 @@ export type BaseMovement = { transfer_date: string; source_warehouse: Warehouse; destination_warehouse: Warehouse; - products: { - product: Product; - product_qty: number; + details: { + id: number; + product_id: number; + quantity: number; + before_quantity: number; + after_quantity: number; }[]; deliveries: { - delivery_cost: number; - delivery_cost_per_item: number; - document: string; - driver_name: string; - vehicle_plate: string; + id: number; + supplier_id: number; supplier: Supplier; - products: { - product: Product; - product_qty: number; + vehicle_plate: string; + driver_name: string; + document_number: string; + document_path: string; + shipping_cost_item: number; + shipping_cost_total: number; + items: { + id: number; + stock_transfer_detail_id: number; + quantity: number; }[]; }[]; }; @@ -41,7 +47,7 @@ export type CreateMovementPayload = { deliveries: { delivery_cost: number; delivery_cost_per_item: number; - document: string | File; + document_index?: number; driver_name: string; vehicle_plate: string; supplier_id: number; diff --git a/src/types/api/inventory/product-warehouse.d.ts b/src/types/api/inventory/product-warehouse.d.ts new file mode 100644 index 00000000..eda8d1b8 --- /dev/null +++ b/src/types/api/inventory/product-warehouse.d.ts @@ -0,0 +1,22 @@ +import { BaseMetadata } from '@/types/api/api-general'; +import { Warehouse } from '@/types/api/master-data/warehouse'; +import { Product } from '@/types/api/master-data/product'; + +export type BaseProductWarehouse = { + id: number; + product_id: number; + warehouse_id: number; + quantity: number; + product: Product; + warehouse: Warehouse; +}; + +export type ProductWarehouse = BaseMetadata & BaseProductWarehouse; + +export type CreateProductWarehousePayload = { + product_id: number; + warehouse_id: number; + quantity: number; +}; + +export type UpdateProductWarehousePayload = CreateProductWarehousePayload; From f5ce898bd2f7ee07e0f12573d7f0fa514cead015 Mon Sep 17 00:00:00 2001 From: rstubryan Date: Thu, 16 Oct 2025 15:29:26 +0700 Subject: [PATCH 35/55] feat(FE-62,63,65): enhance MovementForm with product warehouse selection, delivery document handling, and stock validation --- .../inventory/movement/form/MovementForm.tsx | 239 ++++++++++++++---- .../movement/form/useMovementFormHandlers.ts | 18 +- 2 files changed, 197 insertions(+), 60 deletions(-) diff --git a/src/components/pages/inventory/movement/form/MovementForm.tsx b/src/components/pages/inventory/movement/form/MovementForm.tsx index 1af77cf7..898a1d56 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.tsx +++ b/src/components/pages/inventory/movement/form/MovementForm.tsx @@ -25,11 +25,8 @@ import { DeliverySchema, } from '@/components/pages/inventory/movement/form/MovementForm.schema'; import { useMovementFormHandlers } from './useMovementFormHandlers'; -import { - ProductApi, - SupplierApi, - WarehouseApi, -} from '@/services/api/master-data'; +import { SupplierApi, WarehouseApi } from '@/services/api/master-data'; +import { ProductWarehouseApi } from '@/services/api/inventory'; import { toast } from 'react-hot-toast'; import FileInput from '@/components/input/FileInput'; @@ -40,6 +37,10 @@ interface MovementFormProps { const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { const [, setMovementFormErrorMessage] = useState(''); + const [ + productWarehouseSelectInputValue, + setProductWarehouseSelectInputValue, + ] = useState(''); const [selectedProducts, setSelectedProducts] = useState([]); const [selectedDeliveries, setSelectedDeliveries] = useState([]); @@ -67,7 +68,43 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { validateOnMount: false, enableReinitialize: true, onSubmit: async (values) => { + console.log('=== FORM SUBMIT DEBUG ==='); + console.log('1. Form values received:', values); + setMovementFormErrorMessage(''); + const documents: File[] = []; + const deliveriesPayload = values.deliveries.map((d, idx) => { + let documentIndex = 0; + + console.log(`2. Processing delivery ${idx}:`, { + driver_name: d.driver_name, + document: d.document, + documentType: d.document instanceof File ? 'File' : typeof d.document, + documentSize: d.document instanceof File ? d.document.size : 'N/A', + }); + + if (d.document && d.document instanceof File) { + documents.push(d.document); + documentIndex = documents.length - 1; + console.log(` → Document added at index ${documentIndex}`); + } else { + console.log(` → No document for delivery ${idx}, using index 0`); + } + + return { + delivery_cost: d.delivery_cost, + delivery_cost_per_item: d.delivery_cost_per_item ?? 0, + document_index: documentIndex, + driver_name: d.driver_name, + vehicle_plate: d.vehicle_plate, + supplier_id: d.supplier_id, + products: d.products.map((p) => ({ + product_id: p.product_id, + product_qty: p.product_qty, + })), + }; + }); + const payload: CreateMovementPayload = { transfer_reason: values.transfer_reason, transfer_date: values.transfer_date, @@ -77,26 +114,34 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { product_id: p.product_id, product_qty: p.product_qty, })), - deliveries: values.deliveries.map((d) => ({ - delivery_cost: d.delivery_cost, - delivery_cost_per_item: d.delivery_cost_per_item ?? 0, - document: d.document instanceof File ? d.document : d.document, - driver_name: d.driver_name, - vehicle_plate: d.vehicle_plate, - supplier_id: d.supplier_id, - products: d.products.map((p) => ({ - product_id: p.product_id, - product_qty: p.product_qty, - })), - })), + deliveries: deliveriesPayload, }; + console.log('3. Final payload structure:', { + ...payload, + }); + + console.log( + '4. Document indices in deliveries:', + deliveriesPayload.map((d, i) => ({ + delivery: i, + document_index: d.document_index, + })) + ); + + console.log('5. Total documents:', documents.length); + console.log('=== END SUBMIT DEBUG ==='); + switch (type) { case 'add': - await createMovementHandler(payload); + await createMovementHandler(payload, documents); break; case 'edit': - await updateMovementHandler(initialValues?.id as number, payload); + await updateMovementHandler( + initialValues?.id as number, + payload, + documents + ); break; } }, @@ -144,7 +189,7 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { { delivery_cost: 0, delivery_cost_per_item: 0, - document: '', + document: null, driver_name: '', vehicle_plate: '', supplier: null, @@ -265,15 +310,31 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { })) : []; - // Product selection - const [productSelectInputValue, setProductSelectInputValue] = useState(''); - const productsUrl = `${ProductApi.basePath}?${new URLSearchParams({ search: productSelectInputValue }).toString()}`; - const { data: products, isLoading: isLoadingProducts } = useSWR( - productsUrl, - ProductApi.getAllFetcher - ); - const productOptions = isResponseSuccess(products) - ? products?.data.map((p) => ({ value: p.id, label: p.name })) + // Product Warehouse selection - Filter by source warehouse + const productWarehouseParams = new URLSearchParams({ + search: productWarehouseSelectInputValue, + }); + if (formik.values.source_warehouse_id) { + productWarehouseParams.append( + 'warehouse_id', + formik.values.source_warehouse_id.toString() + ); + } + const productWarehousesUrl = `${ProductWarehouseApi.basePath}?${productWarehouseParams.toString()}`; + const { data: productWarehouses, isLoading: isLoadingProductWarehouses } = + useSWR( + formik.values.source_warehouse_id ? productWarehousesUrl : null, + ProductWarehouseApi.getAllFetcher + ); + const productWarehouseOptions = isResponseSuccess(productWarehouses) + ? productWarehouses?.data.map((pw) => ({ + value: pw.id, + label: pw.product.name, + product_id: pw.product.id, + warehouse_id: pw.warehouse.id, + warehouse_name: pw.warehouse.name, + quantity: pw.quantity, + })) : []; // Supplier selection @@ -303,7 +364,14 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { }); }, [formik.values.deliveries]); - const getFilteredProductOptions = useCallback(() => { + useEffect(() => { + if (formik.values.source_warehouse_id && type !== 'edit') { + formik.setFieldValue('products', []); + formik.setFieldValue('deliveries', []); + } + }, [formik.values.source_warehouse_id]); + + const getFilteredProductWarehouseOptions = useCallback(() => { return ( formik.values.products ?.filter((p) => p.product) @@ -314,6 +382,33 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { ); }, [formik.values.products]); + const getAvailableStock = useCallback( + (productId: number) => { + const productWarehouse = productWarehouseOptions.find( + (pw) => pw.product_id === productId + ); + return productWarehouse?.quantity ?? 0; + }, + [productWarehouseOptions] + ); + + const getProductQtyError = useCallback( + (productIdx: number) => { + const product = formik.values.products?.[productIdx]; + if (!product || !product.product_id) return null; + + const availableStock = getAvailableStock(product.product_id); + const requestedQty = Number(product.product_qty) || 0; + + if (requestedQty > availableStock) { + return `Qty melebihi stok tersedia! Maksimal: ${availableStock.toLocaleString('id-ID')}`; + } + + return null; + }, + [formik.values.products, getAvailableStock] + ); + const validateDeliveryQty = useCallback( (deliveryIdx: number, deliveryProductIdx: number, qty: number) => { const delivery = formik.values.deliveries?.[deliveryIdx]; @@ -406,6 +501,14 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { [invalidQtyRows] ); + const hasExceededStock = useMemo(() => { + return ( + formik.values.products?.some((product, idx) => { + return getProductQtyError(idx) !== null; + }) ?? false + ); + }, [formik.values.products, getProductQtyError]); + return ( <>
    @@ -656,10 +759,18 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { (val as OptionType)?.value ); }} - options={productOptions} - onInputChange={setProductSelectInputValue} - isLoading={isLoadingProducts} - isDisabled={type === 'detail'} + options={productWarehouseOptions} + onInputChange={setProductWarehouseSelectInputValue} + isLoading={isLoadingProductWarehouses} + isDisabled={ + type === 'detail' || + !formik.values.source_warehouse_id + } + placeholder={ + !formik.values.source_warehouse_id + ? 'Pilih gudang asal terlebih dahulu' + : 'Pilih produk' + } isClearable {...isRepeaterInputError( 'products', @@ -669,23 +780,46 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { />
    {type !== 'detail' && ( {type !== 'detail' && ( @@ -1126,6 +1130,10 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { isLoading={isLoadingSuppliers} isDisabled={type === 'detail'} isClearable + className={{ + wrapper: + 'w-full min-w-52 md:min-w-72 lg:min-w-80', + }} /> {type !== 'detail' && ( From c8db992b17a12b0dca4b667c28a0d1fd545ed2b3 Mon Sep 17 00:00:00 2001 From: rstubryan Date: Mon, 20 Oct 2025 11:50:19 +0700 Subject: [PATCH 53/55] feat(FE-62,63,65): add document_path field to deliveries in MovementForm --- .../movement/form/MovementForm.schema.ts | 43 +++++++------ .../inventory/movement/form/MovementForm.tsx | 63 +++++++++++-------- 2 files changed, 61 insertions(+), 45 deletions(-) diff --git a/src/components/pages/inventory/movement/form/MovementForm.schema.ts b/src/components/pages/inventory/movement/form/MovementForm.schema.ts index 2cc5d910..5df66930 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.schema.ts +++ b/src/components/pages/inventory/movement/form/MovementForm.schema.ts @@ -14,6 +14,7 @@ export type DeliverySchema = { delivery_cost?: number | undefined; delivery_cost_per_item?: number | undefined; document?: File | string | null; + document_path?: string | null; driver_name: string; vehicle_plate: string; supplier: { @@ -86,6 +87,7 @@ const DeliveryObjectSchema: Yup.ObjectSchema = Yup.object({ ); } ), + document_path: Yup.string().optional(), document_index: Yup.number().optional(), document: Yup.mixed() .nullable() @@ -161,8 +163,8 @@ export const getMovementFormInitialValues = ( ? { value: initialValues.source_warehouse.id, label: initialValues.source_warehouse.name, - area: initialValues.source_warehouse.area?.name, - location: initialValues.source_warehouse.location?.name, + area: initialValues.source_warehouse.area?.name ?? undefined, + location: initialValues.source_warehouse.location?.name ?? undefined, } : null, source_warehouse_id: initialValues?.source_warehouse?.id ?? 0, @@ -170,8 +172,9 @@ export const getMovementFormInitialValues = ( ? { value: initialValues.destination_warehouse.id, label: initialValues.destination_warehouse.name, - area: initialValues.destination_warehouse.area?.name, - location: initialValues.destination_warehouse.location?.name, + area: initialValues.destination_warehouse.area?.name ?? undefined, + location: + initialValues.destination_warehouse.location?.name ?? undefined, } : null, destination_warehouse_id: initialValues?.destination_warehouse?.id ?? 0, @@ -185,19 +188,20 @@ export const getMovementFormInitialValues = ( product_qty: detail.quantity, })) ?? [], deliveries: - initialValues?.deliveries?.map((d) => { - return { - delivery_cost: d.shipping_cost_total, - delivery_cost_per_item: d.shipping_cost_item, - document_index: 0, - document: d.document_path || null, - driver_name: d.driver_name, - vehicle_plate: d.vehicle_plate, - supplier: d.supplier - ? { value: d.supplier.id, label: d.supplier.name } - : null, - supplier_id: d.supplier?.id ?? 0, - products: d.items.map((item) => { + initialValues?.deliveries?.map((d) => ({ + delivery_cost: d.shipping_cost_total ?? undefined, + delivery_cost_per_item: d.shipping_cost_item ?? undefined, + document_number: d.document_number ?? '', + document: d.document_path ?? null, + document_path: d.document_path ?? null, + driver_name: d.driver_name ?? '', + vehicle_plate: d.vehicle_plate ?? '', + supplier: d.supplier + ? { value: d.supplier.id, label: d.supplier.name } + : null, + supplier_id: d.supplier?.id ?? 0, + products: + d.items?.map((item) => { const productData = detailIdToProductId.get( item.stock_transfer_detail_id ); @@ -208,8 +212,7 @@ export const getMovementFormInitialValues = ( product_id: productData?.id ?? 0, product_qty: item.quantity, }; - }), - }; - }) ?? [], + }) ?? [], + })) ?? [], }; }; diff --git a/src/components/pages/inventory/movement/form/MovementForm.tsx b/src/components/pages/inventory/movement/form/MovementForm.tsx index 034e7b91..671615d9 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.tsx +++ b/src/components/pages/inventory/movement/form/MovementForm.tsx @@ -83,6 +83,7 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { delivery_cost: d.delivery_cost ?? 0, delivery_cost_per_item: d.delivery_cost_per_item ?? 0, document_index: documentIndex, + document_path: d.document_path, driver_name: d.driver_name, vehicle_plate: d.vehicle_plate, supplier_id: d.supplier_id, @@ -1156,32 +1157,44 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { /> @@ -1315,12 +1320,6 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { type={type} formik={formik} - editUrl={ - initialValues - ? `/inventory/movement/detail/edit/?movementId=${initialValues.id}` - : undefined - } - onDelete={deleteMovementClickHandler} disableSubmit={hasInvalidQty || hasExceededStock} /> @@ -1336,23 +1335,6 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { )} - - {type !== 'add' && ( - - )} ); }; From c3338d3e05f8fd602fb88fcd3a6f359039f91b6c Mon Sep 17 00:00:00 2001 From: rstubryan Date: Mon, 20 Oct 2025 15:23:18 +0700 Subject: [PATCH 55/55] feat(FE-62): add button for document path in MovementForm with link functionality --- src/components/Button.tsx | 8 +++-- .../inventory/movement/form/MovementForm.tsx | 29 ++++++++++++++----- 2 files changed, 27 insertions(+), 10 deletions(-) diff --git a/src/components/Button.tsx b/src/components/Button.tsx index c67a29c2..e901b765 100644 --- a/src/components/Button.tsx +++ b/src/components/Button.tsx @@ -1,7 +1,5 @@ import react from 'react'; - import Link from 'next/link'; - import { cn } from '@/lib/helper'; import { Color } from '@/types/theme'; @@ -10,6 +8,8 @@ interface ButtonProps extends react.ComponentProps<'button'> { color?: Color; href?: string; isLoading?: boolean; + target?: string; + rel?: string; } const Button = ({ @@ -22,6 +22,8 @@ const Button = ({ className, disabled, onClick, + target, + rel, ...props }: ButtonProps) => { const btnBaseClassName = cn( @@ -68,6 +70,8 @@ const Button = ({ {href && ( {
    - + + {product.product_id && ( +
    + + Stok tersedia: + {' '} + {getAvailableStock( + product.product_id + ).toLocaleString('id-ID')} +
    )} - readOnly={type === 'detail'} - className={{ - wrapper: 'w-full min-w-24', - }} - /> +
    @@ -819,7 +953,7 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { (val as OptionType)?.value ); }} - options={getFilteredProductOptions()} + options={getFilteredProductWarehouseOptions()} isDisabled={type === 'detail'} isClearable /> @@ -886,7 +1020,6 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { { const file = e.target.files?.[0]; @@ -1016,7 +1149,7 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { : undefined } onDelete={deleteMovementClickHandler} - disableSubmit={hasInvalidQty} + disableSubmit={hasInvalidQty || hasExceededStock} /> {movementFormErrorMessage && ( diff --git a/src/components/pages/inventory/movement/form/useMovementFormHandlers.ts b/src/components/pages/inventory/movement/form/useMovementFormHandlers.ts index 1894b1a7..3f46b71a 100644 --- a/src/components/pages/inventory/movement/form/useMovementFormHandlers.ts +++ b/src/components/pages/inventory/movement/form/useMovementFormHandlers.ts @@ -24,7 +24,6 @@ export const useMovementFormHandlers = (initialValuesId?: number) => { let finalPayload: CreateMovementPayload | FormData; if (documents.length > 0) { - // Ada dokumen: kirim sebagai FormData dengan "data" field console.log('3. Creating FormData (has documents)'); const formData = new FormData(); formData.append('data', JSON.stringify(payload)); @@ -35,7 +34,9 @@ export const useMovementFormHandlers = (initialValuesId?: number) => { console.log('4. FormData entries:'); for (const [key, value] of formData.entries()) { if (value instanceof File) { - console.log(` ${key}: [File] ${value.name} (${value.size} bytes)`); + console.log( + ` ${key}: [File] ${value.name} (${value.size} bytes)` + ); } else { console.log(` ${key}: ${value}`); } @@ -43,7 +44,6 @@ export const useMovementFormHandlers = (initialValuesId?: number) => { finalPayload = formData as unknown as CreateMovementPayload; } else { - // Tidak ada dokumen: kirim sebagai JSON biasa console.log('3. Sending as JSON (no documents)'); console.log('4. Payload:', JSON.stringify(payload, null, 2)); finalPayload = payload; @@ -64,7 +64,11 @@ export const useMovementFormHandlers = (initialValuesId?: number) => { ); const updateMovementHandler = useCallback( - async (movementId: number, payload: UpdateMovementPayload, documents: File[] = []) => { + async ( + movementId: number, + payload: UpdateMovementPayload, + documents: File[] = [] + ) => { console.log('=== UPDATE HANDLER DEBUG ==='); console.log('1. Received payload:', payload); console.log('2. Movement ID:', movementId); @@ -73,7 +77,6 @@ export const useMovementFormHandlers = (initialValuesId?: number) => { let finalPayload: UpdateMovementPayload | FormData; if (documents.length > 0) { - // Ada dokumen: kirim sebagai FormData dengan "data" field console.log('4. Creating FormData (has documents)'); const formData = new FormData(); formData.append('data', JSON.stringify(payload)); @@ -84,7 +87,9 @@ export const useMovementFormHandlers = (initialValuesId?: number) => { console.log('5. FormData entries:'); for (const [key, value] of formData.entries()) { if (value instanceof File) { - console.log(` ${key}: [File] ${value.name} (${value.size} bytes)`); + console.log( + ` ${key}: [File] ${value.name} (${value.size} bytes)` + ); } else { console.log(` ${key}: ${value}`); } @@ -92,7 +97,6 @@ export const useMovementFormHandlers = (initialValuesId?: number) => { finalPayload = formData as unknown as UpdateMovementPayload; } else { - // Tidak ada dokumen: kirim sebagai JSON biasa console.log('4. Sending as JSON (no documents)'); console.log('5. Payload:', JSON.stringify(payload, null, 2)); finalPayload = payload; From 157dfc75ed4317df8b5b1118c7cdbe701342c1a4 Mon Sep 17 00:00:00 2001 From: rstubryan Date: Thu, 16 Oct 2025 15:47:29 +0700 Subject: [PATCH 36/55] refactor(FE-63): remove debug logging from form submission and movement handlers --- .../inventory/movement/form/MovementForm.tsx | 27 ------------ .../movement/form/useMovementFormHandlers.ts | 42 ------------------- src/services/http/client.ts | 5 ++- 3 files changed, 4 insertions(+), 70 deletions(-) diff --git a/src/components/pages/inventory/movement/form/MovementForm.tsx b/src/components/pages/inventory/movement/form/MovementForm.tsx index 898a1d56..a6a75394 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.tsx +++ b/src/components/pages/inventory/movement/form/MovementForm.tsx @@ -68,27 +68,15 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { validateOnMount: false, enableReinitialize: true, onSubmit: async (values) => { - console.log('=== FORM SUBMIT DEBUG ==='); - console.log('1. Form values received:', values); - setMovementFormErrorMessage(''); const documents: File[] = []; const deliveriesPayload = values.deliveries.map((d, idx) => { let documentIndex = 0; - console.log(`2. Processing delivery ${idx}:`, { - driver_name: d.driver_name, - document: d.document, - documentType: d.document instanceof File ? 'File' : typeof d.document, - documentSize: d.document instanceof File ? d.document.size : 'N/A', - }); - if (d.document && d.document instanceof File) { documents.push(d.document); documentIndex = documents.length - 1; - console.log(` → Document added at index ${documentIndex}`); } else { - console.log(` → No document for delivery ${idx}, using index 0`); } return { @@ -117,21 +105,6 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { deliveries: deliveriesPayload, }; - console.log('3. Final payload structure:', { - ...payload, - }); - - console.log( - '4. Document indices in deliveries:', - deliveriesPayload.map((d, i) => ({ - delivery: i, - document_index: d.document_index, - })) - ); - - console.log('5. Total documents:', documents.length); - console.log('=== END SUBMIT DEBUG ==='); - switch (type) { case 'add': await createMovementHandler(payload, documents); diff --git a/src/components/pages/inventory/movement/form/useMovementFormHandlers.ts b/src/components/pages/inventory/movement/form/useMovementFormHandlers.ts index 3f46b71a..5c3d80d1 100644 --- a/src/components/pages/inventory/movement/form/useMovementFormHandlers.ts +++ b/src/components/pages/inventory/movement/form/useMovementFormHandlers.ts @@ -17,40 +17,20 @@ export const useMovementFormHandlers = (initialValuesId?: number) => { const createMovementHandler = useCallback( async (payload: CreateMovementPayload, documents: File[] = []) => { - console.log('=== CREATE HANDLER DEBUG ==='); - console.log('1. Received payload:', payload); - console.log('2. Documents count:', documents.length); - let finalPayload: CreateMovementPayload | FormData; if (documents.length > 0) { - console.log('3. Creating FormData (has documents)'); const formData = new FormData(); formData.append('data', JSON.stringify(payload)); documents.forEach((file, index) => { formData.append(`documents[${index}]`, file); }); - console.log('4. FormData entries:'); - for (const [key, value] of formData.entries()) { - if (value instanceof File) { - console.log( - ` ${key}: [File] ${value.name} (${value.size} bytes)` - ); - } else { - console.log(` ${key}: ${value}`); - } - } - finalPayload = formData as unknown as CreateMovementPayload; } else { - console.log('3. Sending as JSON (no documents)'); - console.log('4. Payload:', JSON.stringify(payload, null, 2)); finalPayload = payload; } - console.log('=== END CREATE HANDLER DEBUG ==='); - const res = await MovementApi.create(finalPayload); if (isResponseError(res)) { console.error('API Error:', res); @@ -69,44 +49,22 @@ export const useMovementFormHandlers = (initialValuesId?: number) => { payload: UpdateMovementPayload, documents: File[] = [] ) => { - console.log('=== UPDATE HANDLER DEBUG ==='); - console.log('1. Received payload:', payload); - console.log('2. Movement ID:', movementId); - console.log('3. Documents count:', documents.length); - let finalPayload: UpdateMovementPayload | FormData; if (documents.length > 0) { - console.log('4. Creating FormData (has documents)'); const formData = new FormData(); formData.append('data', JSON.stringify(payload)); documents.forEach((file, index) => { formData.append(`documents[${index}]`, file); }); - console.log('5. FormData entries:'); - for (const [key, value] of formData.entries()) { - if (value instanceof File) { - console.log( - ` ${key}: [File] ${value.name} (${value.size} bytes)` - ); - } else { - console.log(` ${key}: ${value}`); - } - } - finalPayload = formData as unknown as UpdateMovementPayload; } else { - console.log('4. Sending as JSON (no documents)'); - console.log('5. Payload:', JSON.stringify(payload, null, 2)); finalPayload = payload; } - console.log('=== END UPDATE HANDLER DEBUG ==='); - const res = await MovementApi.update(movementId, finalPayload); if (res?.status === 'error') { - console.error('API Error:', res); setMovementFormErrorMessage(res.message); return; } diff --git a/src/services/http/client.ts b/src/services/http/client.ts index adba75e9..9dd382ca 100644 --- a/src/services/http/client.ts +++ b/src/services/http/client.ts @@ -14,6 +14,9 @@ export async function httpClient( (!opts.auth && opts.auth !== 'none' && opts.auth !== 'bearer'); const isBearerAuth = opts.auth === 'bearer' && !!opts.token; + const isFormData = + typeof FormData !== 'undefined' && opts.body instanceof FormData; + const config: AxiosRequestConfig = { url: path, method: opts.method ?? 'GET', @@ -22,7 +25,7 @@ export async function httpClient( timeout: opts.timeoutMs ?? 10_000, withCredentials: isCookieAuth && !isBearerAuth, headers: { - 'Content-Type': 'application/json', + ...(isFormData ? {} : { 'Content-Type': 'application/json' }), ...(opts.headers ?? {}), ...(isBearerAuth && !isCookieAuth ? { Authorization: `Bearer ${opts.token}` } From a2a57f758c0c7744dd9431aa85dc3e018bf8d5d2 Mon Sep 17 00:00:00 2001 From: rstubryan Date: Thu, 16 Oct 2025 16:24:42 +0700 Subject: [PATCH 37/55] refactor(FE-63,63,65): enhance MovementForm to fetch and display product details from ProductWarehouse --- .../movement/form/MovementForm.schema.ts | 96 +++++---- .../inventory/movement/form/MovementForm.tsx | 182 +++++++++++++++++- 2 files changed, 237 insertions(+), 41 deletions(-) diff --git a/src/components/pages/inventory/movement/form/MovementForm.schema.ts b/src/components/pages/inventory/movement/form/MovementForm.schema.ts index 148a7dce..d5d660ae 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.schema.ts +++ b/src/components/pages/inventory/movement/form/MovementForm.schema.ts @@ -119,43 +119,61 @@ export type MovementFormValues = Yup.InferType; export const getMovementFormInitialValues = ( initialValues?: Movement -): MovementFormValues => ({ - transfer_reason: initialValues?.transfer_reason ?? '', - transfer_date: initialValues?.transfer_date ?? '', - source_warehouse: initialValues?.source_warehouse - ? { - value: initialValues.source_warehouse.id, - label: initialValues.source_warehouse.name, - } - : null, - source_warehouse_id: initialValues?.source_warehouse?.id ?? 0, - destination_warehouse: initialValues?.destination_warehouse - ? { - value: initialValues.destination_warehouse.id, - label: initialValues.destination_warehouse.name, - } - : null, - destination_warehouse_id: initialValues?.destination_warehouse?.id ?? 0, - products: - initialValues?.details?.map((p) => ({ - product: { value: p.product_id, label: '' }, - product_id: p.product_id, - product_qty: p.quantity, - })) ?? [], - deliveries: - initialValues?.deliveries?.map((d) => ({ - delivery_cost: d.shipping_cost_total, - delivery_cost_per_item: d.shipping_cost_item, - document_index: 0, - document: d.document_path || null, - driver_name: d.driver_name, - vehicle_plate: d.vehicle_plate, - supplier: { value: d.supplier.id, label: d.supplier.name }, - supplier_id: d.supplier_id, - products: d.items.map((p) => ({ - product: { value: 0, label: '' }, - product_id: 0, +): MovementFormValues => { + const detailIdToProductId = new Map(); + initialValues?.details?.forEach((detail) => { + detailIdToProductId.set(detail.id, detail.product_id); + }); + + return { + transfer_reason: initialValues?.transfer_reason ?? '', + transfer_date: initialValues?.transfer_date ?? '', + source_warehouse: initialValues?.source_warehouse + ? { + value: initialValues.source_warehouse.id, + label: initialValues.source_warehouse.name, + } + : null, + source_warehouse_id: initialValues?.source_warehouse?.id ?? 0, + destination_warehouse: initialValues?.destination_warehouse + ? { + value: initialValues.destination_warehouse.id, + label: initialValues.destination_warehouse.name, + } + : null, + destination_warehouse_id: initialValues?.destination_warehouse?.id ?? 0, + products: + initialValues?.details?.map((p) => ({ + product: { value: p.product_id, label: `Product ID: ${p.product_id}` }, + product_id: p.product_id, product_qty: p.quantity, - })), - })) ?? [], -}); + })) ?? [], + deliveries: + initialValues?.deliveries?.map((d) => { + return { + delivery_cost: d.shipping_cost_total, + delivery_cost_per_item: d.shipping_cost_item, + document_index: 0, + document: d.document_path || null, + driver_name: d.driver_name, + vehicle_plate: d.vehicle_plate, + supplier: d.supplier + ? { value: d.supplier.id, label: d.supplier.name } + : null, + supplier_id: d.supplier_id, + products: d.items.map((item) => { + const productId = + detailIdToProductId.get(item.stock_transfer_detail_id) ?? 0; + return { + product: + productId > 0 + ? { value: productId, label: `Product ID: ${productId}` } + : null, + product_id: productId, + product_qty: item.quantity, + }; + }), + }; + }) ?? [], + }; +}; diff --git a/src/components/pages/inventory/movement/form/MovementForm.tsx b/src/components/pages/inventory/movement/form/MovementForm.tsx index a6a75394..adf6658d 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.tsx +++ b/src/components/pages/inventory/movement/form/MovementForm.tsx @@ -25,7 +25,11 @@ import { DeliverySchema, } from '@/components/pages/inventory/movement/form/MovementForm.schema'; import { useMovementFormHandlers } from './useMovementFormHandlers'; -import { SupplierApi, WarehouseApi } from '@/services/api/master-data'; +import { + SupplierApi, + WarehouseApi, + ProductApi, +} from '@/services/api/master-data'; import { ProductWarehouseApi } from '@/services/api/inventory'; import { toast } from 'react-hot-toast'; import FileInput from '@/components/input/FileInput'; @@ -43,6 +47,9 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { ] = useState(''); const [selectedProducts, setSelectedProducts] = useState([]); const [selectedDeliveries, setSelectedDeliveries] = useState([]); + const [fetchedProductIds, setFetchedProductIds] = useState>( + new Set() + ); const { deleteModal, @@ -338,12 +345,183 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { }, [formik.values.deliveries]); useEffect(() => { - if (formik.values.source_warehouse_id && type !== 'edit') { + if ( + formik.values.source_warehouse_id && + type !== 'edit' && + type !== 'detail' + ) { formik.setFieldValue('products', []); formik.setFieldValue('deliveries', []); } }, [formik.values.source_warehouse_id]); + // Effect to populate product labels from ProductWarehouse data + useEffect(() => { + if (!productWarehouses || !isResponseSuccess(productWarehouses)) return; + if (type !== 'edit' && type !== 'detail') return; + + let hasUpdates = false; + const updatedProducts = formik.values.products?.map((product) => { + if (product.product && product.product.label.startsWith('Product ID:')) { + const productWarehouse = productWarehouses.data.find( + (pw) => pw.product.id === product.product_id + ); + if (productWarehouse) { + hasUpdates = true; + return { + ...product, + product: { + value: productWarehouse.product.id, + label: productWarehouse.product.name, + }, + }; + } + } + return product; + }); + + if (hasUpdates && updatedProducts) { + formik.setFieldValue('products', updatedProducts); + + const updatedDeliveries = formik.values.deliveries?.map((delivery) => { + const updatedDeliveryProducts = delivery.products.map( + (deliveryProduct) => { + if ( + deliveryProduct.product && + deliveryProduct.product.label.startsWith('Product ID:') + ) { + const productWarehouse = productWarehouses.data.find( + (pw) => pw.product.id === deliveryProduct.product_id + ); + if (productWarehouse) { + return { + ...deliveryProduct, + product: { + value: productWarehouse.product.id, + label: productWarehouse.product.name, + }, + }; + } + } + return deliveryProduct; + } + ); + return { + ...delivery, + products: updatedDeliveryProducts, + }; + }); + formik.setFieldValue('deliveries', updatedDeliveries); + } + }, [productWarehouses, type]); + + useEffect(() => { + if (type !== 'edit' && type !== 'detail') return; + + const productIdsToFetch: number[] = []; + + formik.values.products?.forEach((product) => { + if ( + product.product && + product.product.label.startsWith('Product ID:') && + product.product_id > 0 && + !fetchedProductIds.has(product.product_id) + ) { + productIdsToFetch.push(product.product_id); + } + }); + + formik.values.deliveries?.forEach((delivery) => { + delivery.products.forEach((deliveryProduct) => { + if ( + deliveryProduct.product && + deliveryProduct.product.label.startsWith('Product ID:') && + deliveryProduct.product_id > 0 && + !fetchedProductIds.has(deliveryProduct.product_id) + ) { + if (!productIdsToFetch.includes(deliveryProduct.product_id)) { + productIdsToFetch.push(deliveryProduct.product_id); + } + } + }); + }); + + if (productIdsToFetch.length === 0) return; + + const fetchProducts = async () => { + const productMap = new Map(); + const newFetchedIds = new Set(fetchedProductIds); + + for (const productId of productIdsToFetch) { + try { + const response = await ProductApi.getSingle(productId); + if (isResponseSuccess(response)) { + const product = response.data; + productMap.set(product.id, { id: product.id, name: product.name }); + newFetchedIds.add(productId); + } + } catch (error) { + console.error(`Failed to fetch product ${productId}:`, error); + newFetchedIds.add(productId); // Mark as fetched to avoid retry loops + } + } + + if (productMap.size > 0) { + const updatedProducts = formik.values.products?.map((p) => { + const productData = productMap.get(p.product_id); + if (productData) { + return { + ...p, + product: { + value: productData.id, + label: productData.name, + }, + }; + } + return p; + }); + + const updatedDeliveries = formik.values.deliveries?.map((delivery) => { + const updatedDeliveryProducts = delivery.products.map( + (deliveryProduct) => { + const productData = productMap.get(deliveryProduct.product_id); + if (productData) { + return { + ...deliveryProduct, + product: { + value: productData.id, + label: productData.name, + }, + }; + } + return deliveryProduct; + } + ); + return { + ...delivery, + products: updatedDeliveryProducts, + }; + }); + + if (updatedProducts) { + formik.setFieldValue('products', updatedProducts); + } + if (updatedDeliveries) { + formik.setFieldValue('deliveries', updatedDeliveries); + } + } + + setFetchedProductIds(newFetchedIds); + }; + + fetchProducts(); + }, [ + formik.values.products, + formik.values.deliveries, + type, + fetchedProductIds, + ]); + const getFilteredProductWarehouseOptions = useCallback(() => { return ( formik.values.products From 501a68267e0bbaa820b64fa819d1c601e568a37a Mon Sep 17 00:00:00 2001 From: rstubryan Date: Thu, 16 Oct 2025 16:24:42 +0700 Subject: [PATCH 38/55] refactor(FE-63,63,65): enhance MovementForm to fetch and display product details from ProductWarehouse --- .../movement/form/MovementForm.schema.ts | 96 +++++---- .../inventory/movement/form/MovementForm.tsx | 184 +++++++++++++++++- 2 files changed, 238 insertions(+), 42 deletions(-) diff --git a/src/components/pages/inventory/movement/form/MovementForm.schema.ts b/src/components/pages/inventory/movement/form/MovementForm.schema.ts index 148a7dce..d5d660ae 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.schema.ts +++ b/src/components/pages/inventory/movement/form/MovementForm.schema.ts @@ -119,43 +119,61 @@ export type MovementFormValues = Yup.InferType; export const getMovementFormInitialValues = ( initialValues?: Movement -): MovementFormValues => ({ - transfer_reason: initialValues?.transfer_reason ?? '', - transfer_date: initialValues?.transfer_date ?? '', - source_warehouse: initialValues?.source_warehouse - ? { - value: initialValues.source_warehouse.id, - label: initialValues.source_warehouse.name, - } - : null, - source_warehouse_id: initialValues?.source_warehouse?.id ?? 0, - destination_warehouse: initialValues?.destination_warehouse - ? { - value: initialValues.destination_warehouse.id, - label: initialValues.destination_warehouse.name, - } - : null, - destination_warehouse_id: initialValues?.destination_warehouse?.id ?? 0, - products: - initialValues?.details?.map((p) => ({ - product: { value: p.product_id, label: '' }, - product_id: p.product_id, - product_qty: p.quantity, - })) ?? [], - deliveries: - initialValues?.deliveries?.map((d) => ({ - delivery_cost: d.shipping_cost_total, - delivery_cost_per_item: d.shipping_cost_item, - document_index: 0, - document: d.document_path || null, - driver_name: d.driver_name, - vehicle_plate: d.vehicle_plate, - supplier: { value: d.supplier.id, label: d.supplier.name }, - supplier_id: d.supplier_id, - products: d.items.map((p) => ({ - product: { value: 0, label: '' }, - product_id: 0, +): MovementFormValues => { + const detailIdToProductId = new Map(); + initialValues?.details?.forEach((detail) => { + detailIdToProductId.set(detail.id, detail.product_id); + }); + + return { + transfer_reason: initialValues?.transfer_reason ?? '', + transfer_date: initialValues?.transfer_date ?? '', + source_warehouse: initialValues?.source_warehouse + ? { + value: initialValues.source_warehouse.id, + label: initialValues.source_warehouse.name, + } + : null, + source_warehouse_id: initialValues?.source_warehouse?.id ?? 0, + destination_warehouse: initialValues?.destination_warehouse + ? { + value: initialValues.destination_warehouse.id, + label: initialValues.destination_warehouse.name, + } + : null, + destination_warehouse_id: initialValues?.destination_warehouse?.id ?? 0, + products: + initialValues?.details?.map((p) => ({ + product: { value: p.product_id, label: `Product ID: ${p.product_id}` }, + product_id: p.product_id, product_qty: p.quantity, - })), - })) ?? [], -}); + })) ?? [], + deliveries: + initialValues?.deliveries?.map((d) => { + return { + delivery_cost: d.shipping_cost_total, + delivery_cost_per_item: d.shipping_cost_item, + document_index: 0, + document: d.document_path || null, + driver_name: d.driver_name, + vehicle_plate: d.vehicle_plate, + supplier: d.supplier + ? { value: d.supplier.id, label: d.supplier.name } + : null, + supplier_id: d.supplier_id, + products: d.items.map((item) => { + const productId = + detailIdToProductId.get(item.stock_transfer_detail_id) ?? 0; + return { + product: + productId > 0 + ? { value: productId, label: `Product ID: ${productId}` } + : null, + product_id: productId, + product_qty: item.quantity, + }; + }), + }; + }) ?? [], + }; +}; diff --git a/src/components/pages/inventory/movement/form/MovementForm.tsx b/src/components/pages/inventory/movement/form/MovementForm.tsx index a6a75394..5c5138bf 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.tsx +++ b/src/components/pages/inventory/movement/form/MovementForm.tsx @@ -25,7 +25,11 @@ import { DeliverySchema, } from '@/components/pages/inventory/movement/form/MovementForm.schema'; import { useMovementFormHandlers } from './useMovementFormHandlers'; -import { SupplierApi, WarehouseApi } from '@/services/api/master-data'; +import { + SupplierApi, + WarehouseApi, + ProductApi, +} from '@/services/api/master-data'; import { ProductWarehouseApi } from '@/services/api/inventory'; import { toast } from 'react-hot-toast'; import FileInput from '@/components/input/FileInput'; @@ -43,6 +47,9 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { ] = useState(''); const [selectedProducts, setSelectedProducts] = useState([]); const [selectedDeliveries, setSelectedDeliveries] = useState([]); + const [fetchedProductIds, setFetchedProductIds] = useState>( + new Set() + ); const { deleteModal, @@ -338,12 +345,183 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { }, [formik.values.deliveries]); useEffect(() => { - if (formik.values.source_warehouse_id && type !== 'edit') { + if ( + formik.values.source_warehouse_id && + type !== 'edit' && + type !== 'detail' + ) { formik.setFieldValue('products', []); formik.setFieldValue('deliveries', []); } }, [formik.values.source_warehouse_id]); + // Effect to populate product labels from ProductWarehouse data + useEffect(() => { + if (!productWarehouses || !isResponseSuccess(productWarehouses)) return; + if (type !== 'edit' && type !== 'detail') return; + + let hasUpdates = false; + const updatedProducts = formik.values.products?.map((product) => { + if (product.product && product.product.label.startsWith('Product ID:')) { + const productWarehouse = productWarehouses.data.find( + (pw) => pw.product.id === product.product_id + ); + if (productWarehouse) { + hasUpdates = true; + return { + ...product, + product: { + value: productWarehouse.product.id, + label: productWarehouse.product.name, + }, + }; + } + } + return product; + }); + + if (hasUpdates && updatedProducts) { + formik.setFieldValue('products', updatedProducts); + + const updatedDeliveries = formik.values.deliveries?.map((delivery) => { + const updatedDeliveryProducts = delivery.products.map( + (deliveryProduct) => { + if ( + deliveryProduct.product && + deliveryProduct.product.label.startsWith('Product ID:') + ) { + const productWarehouse = productWarehouses.data.find( + (pw) => pw.product.id === deliveryProduct.product_id + ); + if (productWarehouse) { + return { + ...deliveryProduct, + product: { + value: productWarehouse.product.id, + label: productWarehouse.product.name, + }, + }; + } + } + return deliveryProduct; + } + ); + return { + ...delivery, + products: updatedDeliveryProducts, + }; + }); + formik.setFieldValue('deliveries', updatedDeliveries); + } + }, [productWarehouses, type]); + + useEffect(() => { + if (type !== 'edit' && type !== 'detail') return; + + const productIdsToFetch: number[] = []; + + formik.values.products?.forEach((product) => { + if ( + product.product && + product.product.label.startsWith('Product ID:') && + product.product_id > 0 && + !fetchedProductIds.has(product.product_id) + ) { + productIdsToFetch.push(product.product_id); + } + }); + + formik.values.deliveries?.forEach((delivery) => { + delivery.products.forEach((deliveryProduct) => { + if ( + deliveryProduct.product && + deliveryProduct.product.label.startsWith('Product ID:') && + deliveryProduct.product_id > 0 && + !fetchedProductIds.has(deliveryProduct.product_id) + ) { + if (!productIdsToFetch.includes(deliveryProduct.product_id)) { + productIdsToFetch.push(deliveryProduct.product_id); + } + } + }); + }); + + if (productIdsToFetch.length === 0) return; + + const fetchProducts = async () => { + const productMap = new Map(); + const newFetchedIds = new Set(fetchedProductIds); + + for (const productId of productIdsToFetch) { + try { + const response = await ProductApi.getSingle(productId); + if (isResponseSuccess(response)) { + const product = response.data; + productMap.set(product.id, { id: product.id, name: product.name }); + newFetchedIds.add(productId); + } + } catch (error) { + console.error(`Failed to fetch product ${productId}:`, error); + newFetchedIds.add(productId); // Mark as fetched to avoid retry loops + } + } + + if (productMap.size > 0) { + const updatedProducts = formik.values.products?.map((p) => { + const productData = productMap.get(p.product_id); + if (productData) { + return { + ...p, + product: { + value: productData.id, + label: productData.name, + }, + }; + } + return p; + }); + + const updatedDeliveries = formik.values.deliveries?.map((delivery) => { + const updatedDeliveryProducts = delivery.products.map( + (deliveryProduct) => { + const productData = productMap.get(deliveryProduct.product_id); + if (productData) { + return { + ...deliveryProduct, + product: { + value: productData.id, + label: productData.name, + }, + }; + } + return deliveryProduct; + } + ); + return { + ...delivery, + products: updatedDeliveryProducts, + }; + }); + + if (updatedProducts) { + formik.setFieldValue('products', updatedProducts); + } + if (updatedDeliveries) { + formik.setFieldValue('deliveries', updatedDeliveries); + } + } + + setFetchedProductIds(newFetchedIds); + }; + + fetchProducts(); + }, [ + formik.values.products, + formik.values.deliveries, + type, + fetchedProductIds, + ]); + const getFilteredProductWarehouseOptions = useCallback(() => { return ( formik.values.products @@ -484,7 +662,7 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { return ( <> -
    +
    Date: Thu, 16 Oct 2025 16:46:33 +0700 Subject: [PATCH 39/55] refactor(FE-62,63): update MovementForm to handle 'detail' type with appropriate validations and stock checks --- .../inventory/movement/form/MovementForm.tsx | 44 ++++++++++++------- 1 file changed, 28 insertions(+), 16 deletions(-) diff --git a/src/components/pages/inventory/movement/form/MovementForm.tsx b/src/components/pages/inventory/movement/form/MovementForm.tsx index 5c5138bf..0483b4c1 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.tsx +++ b/src/components/pages/inventory/movement/form/MovementForm.tsx @@ -462,7 +462,7 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { } } catch (error) { console.error(`Failed to fetch product ${productId}:`, error); - newFetchedIds.add(productId); // Mark as fetched to avoid retry loops + newFetchedIds.add(productId); } } @@ -535,16 +535,18 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { const getAvailableStock = useCallback( (productId: number) => { + if (type === 'detail') return 0; const productWarehouse = productWarehouseOptions.find( (pw) => pw.product_id === productId ); return productWarehouse?.quantity ?? 0; }, - [productWarehouseOptions] + [productWarehouseOptions, type] ); const getProductQtyError = useCallback( (productIdx: number) => { + if (type === 'detail') return null; const product = formik.values.products?.[productIdx]; if (!product || !product.product_id) return null; @@ -557,11 +559,12 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { return null; }, - [formik.values.products, getAvailableStock] + [formik.values.products, getAvailableStock, type] ); const validateDeliveryQty = useCallback( (deliveryIdx: number, deliveryProductIdx: number, qty: number) => { + if (type === 'detail') return true; const delivery = formik.values.deliveries?.[deliveryIdx]; if (!delivery) return true; @@ -592,11 +595,12 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { return totalQtyUsed + qty <= Number(relatedProduct.product_qty); }, - [formik.values.deliveries, formik.values.products] + [formik.values.deliveries, formik.values.products, type] ); const getDeliveryQtyError = useCallback( (deliveryIdx: number, deliveryProductIdx: number) => { + if (type === 'detail') return null; const delivery = formik.values.deliveries?.[deliveryIdx]; if (!delivery) return null; @@ -633,32 +637,40 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { return null; }, - [formik.values.deliveries, formik.values.products] + [formik.values.deliveries, formik.values.products, type] ); const invalidQtyRows = useMemo( () => - formik.values.deliveries?.flatMap((delivery, deliveryIdx) => - delivery.products.map((product, productIdx) => { - const qty = Number(product.product_qty) || 0; - return !validateDeliveryQty(deliveryIdx, productIdx, qty); - }) - ) ?? [], - [formik.values.deliveries, formik.values.products, validateDeliveryQty] + type === 'detail' + ? [] + : (formik.values.deliveries?.flatMap((delivery, deliveryIdx) => + delivery.products.map((product, productIdx) => { + const qty = Number(product.product_qty) || 0; + return !validateDeliveryQty(deliveryIdx, productIdx, qty); + }) + ) ?? []), + [ + formik.values.deliveries, + formik.values.products, + validateDeliveryQty, + type, + ] ); const hasInvalidQty = useMemo( - () => invalidQtyRows.some(Boolean), - [invalidQtyRows] + () => (type === 'detail' ? false : invalidQtyRows.some(Boolean)), + [invalidQtyRows, type] ); const hasExceededStock = useMemo(() => { + if (type === 'detail') return false; return ( formik.values.products?.some((product, idx) => { return getProductQtyError(idx) !== null; }) ?? false ); - }, [formik.values.products, getProductQtyError]); + }, [formik.values.products, getProductQtyError, type]); return ( <> @@ -960,7 +972,7 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { wrapper: 'w-full min-w-24', }} /> - {product.product_id && ( + {type !== 'detail' && product.product_id && (
    Stok tersedia: From 8c662a51528b404f08cc82744600b2edc2e599f8 Mon Sep 17 00:00:00 2001 From: rstubryan Date: Fri, 17 Oct 2025 09:22:49 +0700 Subject: [PATCH 40/55] refactor(FE-65): update DeliveryObjectSchema to enforce minimum delivery costs of 1 --- .../pages/inventory/movement/form/MovementForm.schema.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/components/pages/inventory/movement/form/MovementForm.schema.ts b/src/components/pages/inventory/movement/form/MovementForm.schema.ts index d5d660ae..f33888e2 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.schema.ts +++ b/src/components/pages/inventory/movement/form/MovementForm.schema.ts @@ -58,11 +58,11 @@ const DeliveryProductObjectSchema = Yup.object({ const DeliveryObjectSchema: Yup.ObjectSchema = Yup.object({ delivery_cost: Yup.number() .required('Biaya pengiriman wajib diisi!') - .min(0, 'Biaya minimal 0!') + .min(1, 'Biaya minimal 1!') .typeError('Biaya harus berupa angka!'), delivery_cost_per_item: Yup.number() .transform((value) => (isNaN(value) ? undefined : value)) - .min(0, 'Biaya per item minimal 0!') + .min(1, 'Biaya per item minimal 1!') .typeError('Biaya per item harus berupa angka!'), document_index: Yup.number().optional(), document: Yup.mixed() From 8bf7603f6652cff01dcbb1ca292fb0e7b9eb7903 Mon Sep 17 00:00:00 2001 From: rstubryan Date: Fri, 17 Oct 2025 09:46:07 +0700 Subject: [PATCH 41/55] refactor(FE-64): update MovementTable and TableRowOptions to conditionally show edit and delete options --- .../inventory/movement/MovementTable.tsx | 6 +++-- src/components/table/TableRowOptions.tsx | 26 ++++++++++++------- 2 files changed, 20 insertions(+), 12 deletions(-) diff --git a/src/components/pages/inventory/movement/MovementTable.tsx b/src/components/pages/inventory/movement/MovementTable.tsx index e0bc9541..61be40f8 100644 --- a/src/components/pages/inventory/movement/MovementTable.tsx +++ b/src/components/pages/inventory/movement/MovementTable.tsx @@ -157,8 +157,9 @@ const MovementTable = () => { type='dropdown' recordId={props.row.original.id} basePath='/inventory/movement' - onDelete={deleteClickHandler} queryParam='movementId' + showEdit={false} + showDelete={false} /> )} @@ -169,8 +170,9 @@ const MovementTable = () => { type='collapse' recordId={props.row.original.id} basePath='/inventory/movement' - onDelete={deleteClickHandler} queryParam='movementId' + showEdit={false} + showDelete={false} /> )} diff --git a/src/components/table/TableRowOptions.tsx b/src/components/table/TableRowOptions.tsx index e34f2ad4..4e2e2c93 100644 --- a/src/components/table/TableRowOptions.tsx +++ b/src/components/table/TableRowOptions.tsx @@ -8,6 +8,8 @@ interface TableRowOptionsProps { basePath: string; onDelete?: () => void; queryParam?: string; + showEdit?: boolean; + showDelete?: boolean; } export const TableRowOptions = ({ @@ -16,6 +18,8 @@ export const TableRowOptions = ({ basePath, onDelete, queryParam = 'id', + showEdit = true, + showDelete = true, }: TableRowOptionsProps) => (
    Detail - - {onDelete && ( + {showEdit && ( + + )} + {showDelete && onDelete && (
    + handleDeliveryCostPerItemChange(idx, e.target.value) } - readOnly - className={{ - input: 'bg-base-200', - }} + onBlur={formik.handleBlur} + {...isRepeaterInputError( + 'deliveries', + 'delivery_cost_per_item', + idx + )} + readOnly={type === 'detail'} /> From 0676411dd5d65344e5340ef747889d04e5f11327 Mon Sep 17 00:00:00 2001 From: rstubryan Date: Fri, 17 Oct 2025 19:23:19 +0700 Subject: [PATCH 47/55] refactor(FE-62,65): enhance product quantity display and stock information in MovementForm --- .../inventory/movement/form/MovementForm.tsx | 109 +++++++++++------- 1 file changed, 66 insertions(+), 43 deletions(-) diff --git a/src/components/pages/inventory/movement/form/MovementForm.tsx b/src/components/pages/inventory/movement/form/MovementForm.tsx index 2ec06e66..d77f2de4 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.tsx +++ b/src/components/pages/inventory/movement/form/MovementForm.tsx @@ -266,6 +266,13 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { location?: string; } + interface ProductWarehouseOptionType extends OptionType { + product_id: number; + warehouse_id: number; + warehouse_name: string; + quantity: number; + } + // Warehouse selection const [warehouseSelectInputValue, setWarehouseSelectInputValue] = useState(''); @@ -304,8 +311,8 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { ); const productWarehouseOptions = isResponseSuccess(productWarehouses) ? productWarehouses?.data.map((pw) => ({ - value: pw.id, - label: pw.product.name, + value: pw.product.id, + label: `${pw.product.name} - ${pw.warehouse.name} (Stock: ${pw.quantity.toLocaleString('id-ID')})`, product_id: pw.product.id, warehouse_id: pw.warehouse.id, warehouse_name: pw.warehouse.name, @@ -438,6 +445,33 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { [productWarehouseOptions, type] ); + const getProductQtyAdornment = useCallback( + (productIdx: number) => { + if (type === 'detail') return null; + const product = formik.values.products?.[productIdx]; + if (!product || !product.product_id) return null; + + const availableStock = getAvailableStock(product.product_id); + const requestedQty = Number(product.product_qty) || 0; + const remainingStock = availableStock - requestedQty; + + if (requestedQty > 0) { + return ( + + (sisa: {remainingStock.toLocaleString('id-ID')}) + + ); + } + + return ( + + (tersedia: {availableStock.toLocaleString('id-ID')}) + + ); + }, + [formik.values.products, getAvailableStock, type] + ); + const getProductQtyError = useCallback( (productIdx: number) => { if (type === 'detail') return null; @@ -813,7 +847,7 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { ); formik.setFieldValue( `products.${idx}.product_id`, - (val as OptionType)?.value + (val as ProductWarehouseOptionType)?.value ); }} options={productWarehouseOptions} @@ -837,46 +871,35 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { /> -
    - - {type !== 'detail' && product.product_id && ( -
    - - Stok tersedia: - {' '} - {getAvailableStock( - product.product_id - ).toLocaleString('id-ID')} -
    - )} -
    +
    From c4de085e111f4f3abcd41916ff2bc3a48a1b0d8e Mon Sep 17 00:00:00 2001 From: rstubryan Date: Sat, 18 Oct 2025 08:40:06 +0700 Subject: [PATCH 48/55] feat(FE-62,63): enhance warehouse stock information display in MovementForm --- .../inventory/movement/form/MovementForm.tsx | 82 +++++++++++++++---- 1 file changed, 65 insertions(+), 17 deletions(-) diff --git a/src/components/pages/inventory/movement/form/MovementForm.tsx b/src/components/pages/inventory/movement/form/MovementForm.tsx index d77f2de4..a35937f7 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.tsx +++ b/src/components/pages/inventory/movement/form/MovementForm.tsx @@ -25,10 +25,7 @@ import { DeliverySchema, } from '@/components/pages/inventory/movement/form/MovementForm.schema'; import { useMovementFormHandlers } from './useMovementFormHandlers'; -import { - SupplierApi, - WarehouseApi, -} from '@/services/api/master-data'; +import { SupplierApi, WarehouseApi } from '@/services/api/master-data'; import { ProductWarehouseApi } from '@/services/api/inventory'; import { toast } from 'react-hot-toast'; import FileInput from '@/components/input/FileInput'; @@ -273,6 +270,36 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { quantity: number; } + const allProductWarehousesUrl = `${ProductWarehouseApi.basePath}`; + const { data: allProductWarehouses } = useSWR( + allProductWarehousesUrl, + ProductWarehouseApi.getAllFetcher + ); + + const warehouseStockMap = useMemo(() => { + if (!isResponseSuccess(allProductWarehouses)) return new Map(); + + const stockMap = new Map< + number, + { totalQty: number; productCount: number } + >(); + + allProductWarehouses.data.forEach((pw) => { + const warehouseId = pw.warehouse.id; + const existing = stockMap.get(warehouseId) || { + totalQty: 0, + productCount: 0, + }; + + stockMap.set(warehouseId, { + totalQty: existing.totalQty + pw.quantity, + productCount: existing.productCount + 1, + }); + }); + + return stockMap; + }, [allProductWarehouses]); + // Warehouse selection const [warehouseSelectInputValue, setWarehouseSelectInputValue] = useState(''); @@ -282,15 +309,22 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { WarehouseApi.getAllFetcher ); const warehouseOptions = isResponseSuccess(warehouses) - ? warehouses?.data.map((w) => ({ - value: w.id, - label: w.name, - area: w.area?.name, - location: - 'type' in w && (w.type === 'LOKASI' || w.type === 'KANDANG') - ? w.location?.name - : undefined, - })) + ? warehouses?.data.map((w) => { + const stockInfo = warehouseStockMap.get(w.id); + const stockLabel = stockInfo + ? ` (Stock: ${stockInfo.totalQty.toLocaleString('id-ID')} items, ${stockInfo.productCount} produk)` + : ' (Kosong)'; + + return { + value: w.id, + label: `${w.name}${stockLabel}`, + area: w.area?.name, + location: + 'type' in w && (w.type === 'LOKASI' || w.type === 'KANDANG') + ? w.location?.name + : undefined, + }; + }) : []; // Product Warehouse selection - Filter by source warehouse @@ -361,7 +395,10 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { const handleDeliveryCostPerItemChange = useCallback( (idx: number, value: string) => { const numValue = parseFloat(value) || 0; - formik.setFieldValue(`deliveries.${idx}.delivery_cost_per_item`, numValue); + formik.setFieldValue( + `deliveries.${idx}.delivery_cost_per_item`, + numValue + ); const delivery = formik.values.deliveries?.[idx]; if (delivery) { @@ -389,7 +426,11 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { ); // If delivery_cost is set, recalculate delivery_cost_per_item - if (delivery.delivery_cost && delivery.delivery_cost > 0 && productQty > 0) { + if ( + delivery.delivery_cost && + delivery.delivery_cost > 0 && + productQty > 0 + ) { const perItem = delivery.delivery_cost / productQty; if (Math.abs((delivery.delivery_cost_per_item || 0) - perItem) > 0.01) { formik.setFieldValue( @@ -410,7 +451,11 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { } } }); - }, [formik.values.deliveries?.map(d => d.products.reduce((sum, p) => sum + p.product_qty, 0)).join(',')]); + }, [ + formik.values.deliveries + ?.map((d) => d.products.reduce((sum, p) => sum + p.product_qty, 0)) + .join(','), + ]); useEffect(() => { if ( @@ -1147,7 +1192,10 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { name={`deliveries.${idx}.delivery_cost_per_item`} value={delivery.delivery_cost_per_item || ''} onChange={(e) => - handleDeliveryCostPerItemChange(idx, e.target.value) + handleDeliveryCostPerItemChange( + idx, + e.target.value + ) } onBlur={formik.handleBlur} {...isRepeaterInputError( From 6fe85fac137acf25d39b1d35f6f9867e2f647298 Mon Sep 17 00:00:00 2001 From: ValdiANS Date: Sat, 18 Oct 2025 13:40:08 +0700 Subject: [PATCH 49/55] feat(FE-113): add Client, Permission, Role, and RoleWithPermission types --- src/types/api/api-general.d.ts | 30 ++++++++++++++++++++++++++++++ 1 file changed, 30 insertions(+) diff --git a/src/types/api/api-general.d.ts b/src/types/api/api-general.d.ts index cf3e57f7..c118b5a4 100644 --- a/src/types/api/api-general.d.ts +++ b/src/types/api/api-general.d.ts @@ -24,6 +24,36 @@ export type LogoutResponse = BaseApiResponse; export type GetMeResponse = BaseApiResponse; +export type Client = { + id: number; + name: stirng; + alias: string; + created_at: string; + updated_at: string; +}; + +export type Permission = { + id: number; + name: string; + action: string; + client: Omit; + created_at: string; + updated_at: string; +}; + +export type Role = { + id: number; + key: string; + name: string; + client: Omit; + created_at: string; + updated_at: string; +}; + +export type RoleWithPermissions = Omit & { + permissions: Omit[]; +}; + export type User = { id: number; email: string; From 2b3aa9c3eed4a11593b1d75035acc81be3faf4ee Mon Sep 17 00:00:00 2001 From: ValdiANS Date: Sat, 18 Oct 2025 13:40:32 +0700 Subject: [PATCH 50/55] feat(FE-113): create permissionCheck helper function --- src/services/hooks/useAuth.tsx | 25 +++++++++++++++++++++++-- 1 file changed, 23 insertions(+), 2 deletions(-) diff --git a/src/services/hooks/useAuth.tsx b/src/services/hooks/useAuth.tsx index 86bf43ed..79fa8981 100644 --- a/src/services/hooks/useAuth.tsx +++ b/src/services/hooks/useAuth.tsx @@ -6,22 +6,43 @@ type AuthStore = { isLoadingUser?: boolean; setUser: (newUserData?: UserWithRoles) => void; setIsLoadingUser: (isLoading?: boolean) => void; + permissionCheck: (permissionName: string) => boolean; }; -const useAuthStore = create()((set) => ({ +const useAuthStore = create()((set, get) => ({ user: undefined, isLoadingUser: false, setUser: (newUserData) => set({ user: newUserData }), setIsLoadingUser: (isLoading) => set({ isLoadingUser: Boolean(isLoading) }), + + permissionCheck: (name) => { + const { user, isLoadingUser } = get(); + + if (!isLoadingUser && user) { + const isAllowed = user.roles.some((role) => { + const isPermissionNameAllowed = role.permissions.some( + (permission) => permission.name === name + ); + + return isPermissionNameAllowed; + }); + + return isAllowed; + } + + return false; + }, })); export const useAuth = () => { - const { user, setUser, isLoadingUser, setIsLoadingUser } = useAuthStore(); + const { user, setUser, isLoadingUser, setIsLoadingUser, permissionCheck } = + useAuthStore(); return { user, setUser, isLoadingUser, setIsLoadingUser, + permissionCheck, }; }; From 376fa29f7edf4da9df926566f270f6053895f6de Mon Sep 17 00:00:00 2001 From: ValdiANS Date: Mon, 20 Oct 2025 09:55:08 +0700 Subject: [PATCH 51/55] fix(FE-40): wrap master data detail with SuspenseHelper --- src/app/master-data/customer/detail/layout.tsx | 11 +++++++++++ src/app/master-data/supplier/detail/layout.tsx | 11 +++++++++++ 2 files changed, 22 insertions(+) create mode 100644 src/app/master-data/customer/detail/layout.tsx create mode 100644 src/app/master-data/supplier/detail/layout.tsx diff --git a/src/app/master-data/customer/detail/layout.tsx b/src/app/master-data/customer/detail/layout.tsx new file mode 100644 index 00000000..7220dfa1 --- /dev/null +++ b/src/app/master-data/customer/detail/layout.tsx @@ -0,0 +1,11 @@ +import SuspenseHelper from '@/components/helper/SuspenseHelper'; + +const Layout = ({ + children, +}: Readonly<{ + children: React.ReactNode; +}>) => { + return {children}; +}; + +export default Layout; diff --git a/src/app/master-data/supplier/detail/layout.tsx b/src/app/master-data/supplier/detail/layout.tsx new file mode 100644 index 00000000..7220dfa1 --- /dev/null +++ b/src/app/master-data/supplier/detail/layout.tsx @@ -0,0 +1,11 @@ +import SuspenseHelper from '@/components/helper/SuspenseHelper'; + +const Layout = ({ + children, +}: Readonly<{ + children: React.ReactNode; +}>) => { + return {children}; +}; + +export default Layout; From d76f897840b37331c1eddcd54d90a6ed0c0575e0 Mon Sep 17 00:00:00 2001 From: rstubryan Date: Mon, 20 Oct 2025 11:32:35 +0700 Subject: [PATCH 52/55] refactor(FE-62): update wrapper class names for improved layout in MovementForm --- .../inventory/movement/form/MovementForm.tsx | 28 ++++++++++++++++++- 1 file changed, 27 insertions(+), 1 deletion(-) diff --git a/src/components/pages/inventory/movement/form/MovementForm.tsx b/src/components/pages/inventory/movement/form/MovementForm.tsx index a35937f7..034e7b91 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.tsx +++ b/src/components/pages/inventory/movement/form/MovementForm.tsx @@ -1081,6 +1081,10 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { options={getFilteredProductWarehouseOptions()} isDisabled={type === 'detail'} isClearable + className={{ + wrapper: + 'w-full min-w-52 md:min-w-72 lg:min-w-80', + }} /> @@ -1103,7 +1107,7 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { } readOnly={type === 'detail'} className={{ - wrapper: 'w-full min-w-24', + wrapper: 'w-full min-w-48', }} /> @@ -1141,6 +1149,10 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { idx )} readOnly={type === 'detail'} + className={{ + wrapper: + 'w-full min-w-52 md:min-w-72 lg:min-w-80', + }} /> @@ -1165,6 +1177,10 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { idx )} readOnly={type === 'detail'} + className={{ + wrapper: + 'w-full min-w-72 md:w-min-80 lg:w-min-96', + }} /> @@ -1183,6 +1199,9 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { idx )} readOnly={type === 'detail'} + className={{ + wrapper: 'w-full min-w-48', + }} /> @@ -1204,6 +1223,9 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { idx )} readOnly={type === 'detail'} + className={{ + wrapper: 'w-full min-w-48', + }} /> @@ -1219,6 +1241,10 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { idx )} readOnly={type === 'detail'} + className={{ + wrapper: + 'w-full min-w-52 md:min-w-72 lg:min-w-80', + }} /> - { - const file = e.target.files?.[0]; - if (file) { - if (file.size > 2 * 1024 * 1024) { - toast.error('Ukuran dokumen maksimal 2 MB!'); - return; + {type === 'detail' ? ( + + ) : ( + { + const file = e.target.files?.[0]; + if (file) { + if (file.size > 2 * 1024 * 1024) { + toast.error( + 'Ukuran dokumen maksimal 2 MB!' + ); + return; + } + formik.setFieldValue( + `deliveries.${idx}.document`, + file + ); } - formik.setFieldValue( - `deliveries.${idx}.document`, - file - ); - } - }} - {...isRepeaterInputError( - 'deliveries', - 'document', - idx - )} - readOnly={type === 'detail'} - className={{ - wrapper: - 'w-full min-w-72 md:w-min-80 lg:w-min-96', - }} - /> + }} + {...isRepeaterInputError( + 'deliveries', + 'document', + idx + )} + className={{ + wrapper: + 'w-full min-w-72 md:w-min-80 lg:w-min-96', + }} + /> + )} Date: Mon, 20 Oct 2025 12:03:58 +0700 Subject: [PATCH 54/55] refactor(FE-62): update layout and remove unused delete confirmation in MovementForm --- .../inventory/movement/form/MovementForm.tsx | 30 ++++--------------- 1 file changed, 6 insertions(+), 24 deletions(-) diff --git a/src/components/pages/inventory/movement/form/MovementForm.tsx b/src/components/pages/inventory/movement/form/MovementForm.tsx index 671615d9..d5d28f5f 100644 --- a/src/components/pages/inventory/movement/form/MovementForm.tsx +++ b/src/components/pages/inventory/movement/form/MovementForm.tsx @@ -914,6 +914,10 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { 'product', idx )} + className={{ + wrapper: + 'w-full min-w-52 md:min-w-72 lg:min-w-80', + }} /> @@ -943,7 +947,8 @@ const MovementForm = ({ type = 'add', initialValues }: MovementFormProps) => { } readOnly={type === 'detail'} className={{ - wrapper: 'w-full min-w-48', + wrapper: + 'w-full min-w-52 md:min-w-72 lg:min-w-80', }} /> {type === 'detail' ? ( - + ) : (