fix: hide groundForRefusal when status is not refused (#400)

* RM#75117
axelor · Feb 9, 2024 · 3776226 · 3776226
1 parent 0e9f110
commit 3776226
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 16 deletions.
diff --git a/packages/apps/hr/src/components/molecules/TimesheetHeader/TimesheetHeader.tsx b/packages/apps/hr/src/components/molecules/TimesheetHeader/TimesheetHeader.tsx
@@ -89,14 +89,15 @@ const TimesheetHeader = ({timesheet, statusSelect}: TimesheetHeaderProps) => {
           />
         )}
       </View>
-      {!checkNullString(timesheet.groundForRefusal) && (
-        <Label
-          message={`${I18n.t('Hr_GroundForRefusal')} : ${
-            timesheet.groundForRefusal
-          }`}
-          type="error"
-        />
-      )}
+      {statusSelect === Timesheet.statusSelect.Refused &&
+        !checkNullString(timesheet.groundForRefusal) && (
+          <Label
+            message={`${I18n.t('Hr_GroundForRefusal')} : ${
+              timesheet.groundForRefusal
+            }`}
+            type="error"
+          />
+        )}
     </View>
   );
 };

diff --git a/packages/apps/hr/src/components/templates/ExpenseHeader/ExpenseHeader.tsx b/packages/apps/hr/src/components/templates/ExpenseHeader/ExpenseHeader.tsx
@@ -53,14 +53,15 @@ const ExpenseHeader = ({}) => {
           ? user?.activeCompany?.currency?.symbol
           : user?.activeCompany?.currency?.code
       }`}</Text>
-      {!checkNullString(expense?.groundForRefusal) && (
-        <Label
-          message={`${I18n.t('Hr_GroundForRefusal')} : ${
-            expense?.groundForRefusal
-          }`}
-          type="error"
-        />
-      )}
+      {expense.statusSelect === Expense.statusSelect.Refused &&
+        !checkNullString(expense?.groundForRefusal) && (
+          <Label
+            message={`${I18n.t('Hr_GroundForRefusal')} : ${
+              expense?.groundForRefusal
+            }`}
+            type="error"
+          />
+        )}
     </View>
   );
 };