discourage using capitalizeWords fn. by being kept as a shared utilit…

…y fn. use tailwindcss's alternative

src/Utils/Notifications.js

@@ -1,8 +1,6 @@
 import { Stack, alert, defaultModules } from "@pnotify/core";
 import * as PNotifyMobile from "@pnotify/mobile";
 
-import { capitalizeWords, cleanStringForNotifications } from "@/Utils/utils";
-
 defaultModules.set(PNotifyMobile, {});
 
 const notifyStack = new Stack({
@@ -36,6 +34,25 @@ const notify = (text, type) => {
   });
 };
 
+/**
+ * Formats input string to a more human readable format
+ * @param {string} key - The key to format
+ * @returns {string} The formatted key
+ * @example
+ * formatKey("patient_name") => "Patient Name"
+ */
+const formatKey = (key) => {
+  return key
+    .replace(/[^a-zA-Z0-9]+/g, " ") // Replace non-alphanumeric characters with a space
+    .trim()
+    .split(" ")
+    .map(
+      (word) =>
+        word.charAt(0).toLocaleUpperCase() + word.slice(1).toLocaleLowerCase(),
+    ) // Capitalize the first letter of each word and lowercase the rest
+    .join(" ");
+};
+
 const notifyError = (error) => {
   let errorMsg = "";
   if (typeof error === "string" || !error) {
@@ -45,7 +62,7 @@ const notifyError = (error) => {
     errorMsg = error.detail;
   } else {
     for (let [key, value] of Object.entries(error)) {
-      let keyName = capitalizeWords(cleanStringForNotifications(key));
+      let keyName = formatKey(key);
       if (Array.isArray(value)) {
         const uniques = [...new Set(value)];
         errorMsg += `${keyName} - ${uniques.splice(0, 5).join(", ")}`;

src/Utils/utils.ts

@@ -545,27 +545,6 @@ export const keysOf = <T extends object>(obj: T) => {
   return Object.keys(obj) as (keyof T)[];
 };
 
-//removes all symbols except _ and - (one part of startCase)
-export const cleanStringForNames = (str: string): string => {
-  return str.replace(/[^a-zA-Z0-9_-]+/g, " ").trim();
-};
-
-export const cleanStringForNotifications = (str: string): string => {
-  return str.replace(/[^a-zA-Z0-9]+/g, " ").trim();
-};
-
-// just capitalizes (as per startCase working) (part of startCase) (only inside Notification.js)
-export const capitalizeWords = (str: string): string => {
-  if (!str) return "";
-  return str
-    .split(" ")
-    .map(
-      (word) =>
-        word.charAt(0).toLocaleUpperCase() + word.slice(1).toLocaleLowerCase(),
-    )
-    .join(" ");
-};
-
 // Utility to check if a value is "empty"
 export const isEmpty = (value: unknown) => {
   return value === "" || value == undefined;
@@ -574,7 +553,7 @@ export const isEmpty = (value: unknown) => {
 // equivalent to lodash omitBy
 export function omitBy<T extends Record<string, unknown>>(
   obj: T,
-  predicate: (value: unknown) => boolean = isEmpty,
+  predicate: (value: unknown) => boolean,
 ): Partial<T> {
   return Object.fromEntries(
     Object.entries(obj).filter(([_, value]) => !predicate(value)),

src/components/Form/Form.tsx

@@ -16,7 +16,7 @@ import {
 
 import { DraftSection, useAutoSaveReducer } from "@/Utils/AutoSave";
 import * as Notification from "@/Utils/Notifications";
-import { classNames, omitBy } from "@/Utils/utils";
+import { classNames, isEmpty, omitBy } from "@/Utils/utils";
 
 type Props<T extends FormDetails> = {
   className?: string;
@@ -57,7 +57,7 @@ const Form = <T extends FormDetails>({
     event.stopPropagation();
 
     if (validate) {
-      const errors = omitBy(validate(state.form)) as FormErrors<T>;
+      const errors = omitBy(validate(state.form), isEmpty) as FormErrors<T>;
 
       if (Object.keys(errors).length) {
         dispatch({ type: "set_errors", errors });