fix: switch map to reduce

metrotranscom · Nov 9, 2023 · 3e9a8da · 3e9a8da
1 parent 5bf8306
commit 3e9a8da
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/backend/core/src/email/email.service.ts b/backend/core/src/email/email.service.ts
@@ -46,9 +46,12 @@ const formatUnitDetails = (
   label: string,
   pluralLabel?: string
 ): string => {
-  const mappedField = units
-    .map((unit) => (unit[field] ? Number.parseFloat(unit[field]) : null))
-    .filter((value) => value)
+  const mappedField = units.reduce((values, unit) => {
+    if (unit[field]) {
+      values.push(Number.parseFloat(unit[field]))
+    }
+    return values
+  }, [])
   if (mappedField?.length) {
     const minValue = Math.min(...mappedField)
     const maxValue = Math.max(...mappedField)