From 6a08854603ece7ccd01d005ab3a2a471efd72a44 Mon Sep 17 00:00:00 2001
From: ValdiANS <rvdans002@gmail.com>
Date: Mon, 24 Nov 2025 09:21:32 +0700
Subject: [PATCH] chore(FE-204,207): add validation to check if expense can be
 edited

---
 src/app/expense/detail/edit/page.tsx | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/src/app/expense/detail/edit/page.tsx b/src/app/expense/detail/edit/page.tsx
index b37fdb8f..e254f01d 100644
--- a/src/app/expense/detail/edit/page.tsx
+++ b/src/app/expense/detail/edit/page.tsx
@@ -34,13 +34,15 @@ const ExpenseEditPage = () => {
     return;
   }
 
-  const isExpenseRejectedOrApproved =
+  const isExpenseCanBeEdited =
     !isLoadingExpense &&
     isResponseSuccess(expense) &&
-    (expense.data.approval.action === 'REJECTED' ||
-      expense.data.approval.step_number === 5);
+    expense.data.latest_approval.step_number !== 5 &&
+    (expense.data.latest_approval.step_number === 1 ||
+      expense.data.latest_approval.step_number === 2 ||
+      expense.data.latest_approval.step_number === 3);
 
-  if (isExpenseRejectedOrApproved) {
+  if (!isLoadingExpense && !isExpenseCanBeEdited) {
     router.back();
     return;
   }