Skip to content

Commit 579de50

Browse files
authored
Merge pull request #430 from richjenks/recipe-selector-roi
Fix recipe selector ROI column sorting
2 parents 1d5d22e + e828403 commit 579de50

1 file changed

Lines changed: 9 additions & 2 deletions

File tree

src/features/planning/components/PlanProductionRecipe.vue

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -105,6 +105,13 @@
105105
);
106106
107107
const cogmWithCX = computed(() => !!props.cxUuid);
108+
109+
function roiSorter(
110+
row1: Record<string, unknown>,
111+
row2: Record<string, unknown>
112+
): number {
113+
return (row1.dailyRevenue as number) - (row2.dailyRevenue as number);
114+
}
108115
</script>
109116

110117
<template>
@@ -255,15 +262,15 @@
255262
</span>
256263
</template>
257264
</XNDataTableColumn>
258-
<XNDataTableColumn key="roi" title="ROI" sorter="default">
265+
<XNDataTableColumn key="roi" title="ROI" :sorter="roiSorter">
259266
<template #render-cell="{ rowData }">
260267
<span
261268
:class="
262269
rowData.roi >= 0
263270
? 'text-positive!'
264271
: 'text-negative!'
265272
">
266-
{{ formatNumber(rowData.roi) }} d
273+
{{ rowData.roi < 0 ? "—" : formatNumber(rowData.roi) + " d" }}
267274
</span>
268275
</template>
269276
</XNDataTableColumn>

0 commit comments

Comments
 (0)