feat: better violation names on statistics page

ballerine-io · Jan 2, 2025 · b9977f8 · b9977f8
1 parent 24dae62
commit b9977f8
Show file tree

Hide file tree

Showing 5 changed files with 21 additions and 11 deletions.
diff --git a/...ness-reports/hooks/queries/useBusinessReportMetricsQuery/useBusinessReportMetricsQuery.ts b/...ness-reports/hooks/queries/useBusinessReportMetricsQuery/useBusinessReportMetricsQuery.ts
@@ -12,7 +12,13 @@ export const MetricsResponseSchema = z.object({
     high: z.number(),
     critical: z.number(),
   }),
-  violationCounts: z.record(z.string(), z.number()),
+  violationCounts: z.array(
+    z.object({
+      name: z.string(),
+      id: z.string(),
+      count: z.number(),
+    }),
+  ),
 });
 
 export const fetchBusinessReportMetrics = async () => {

diff --git a/...ce-v2/src/pages/Statistics/components/PortfolioRiskStatistics/PortfolioRiskStatistics.tsx b/...ce-v2/src/pages/Statistics/components/PortfolioRiskStatistics/PortfolioRiskStatistics.tsx
@@ -172,11 +172,11 @@ export const PortfolioRiskStatistics: FunctionComponent<z.infer<typeof MetricsRe
                   </TableRow>
                 </TableHeader>
                 <TableBody ref={parent}>
-                  {filteredRiskIndicators.map(({ name, count }, index) => (
+                  {filteredRiskIndicators.map(({ name, count, id }, index) => (
                     <TableRow key={name} className={'border-b-0 hover:bg-[unset]'}>
                       <TableCell className={ctw('pb-0 ps-0', index !== 0 && 'pt-2')}>
                         <Link
-                          to={`/${locale}/merchant-monitoring?findings[0]=${name}`}
+                          to={`/${locale}/merchant-monitoring?findings[0]=${id}`}
                           className={`block h-full cursor-pointer rounded bg-blue-200 p-1 transition-all`}
                           style={{ width: `${widths[index]}%` }}
                         >

diff --git a/...oRiskStatistics/hooks/usePortfolioRiskStatisticsLogic/usePortfolioRiskStatisticsLogic.tsx b/...oRiskStatistics/hooks/usePortfolioRiskStatisticsLogic/usePortfolioRiskStatisticsLogic.tsx
@@ -20,11 +20,10 @@ export const usePortfolioRiskStatisticsLogic = ({
     },
     [],
   );
-  const totalRiskIndicators = Object.values(violationCounts).reduce((acc, curr) => acc + curr, 0);
+  const totalRiskIndicators = violationCounts.reduce((acc, { count }) => acc + count, 0);
   const filteredRiskIndicators = useMemo(
     () =>
-      Object.entries(violationCounts)
-        .map(([name, count]) => ({ name, count }))
+      violationCounts
         .sort((a, b) => (riskIndicatorsSorting === 'asc' ? a.count - b.count : b.count - a.count))
         .slice(0, 5),
     [violationCounts, riskIndicatorsSorting],

diff --git a/services/workflows-service/src/business-report/dtos/business-report-metrics-dto.ts b/services/workflows-service/src/business-report/dtos/business-report-metrics-dto.ts
@@ -47,12 +47,11 @@ export class BusinessReportMetricsDto {
   riskLevelCounts!: RiskLevelCountsDto;
 
   @ApiProperty({
-    description: 'Counts of violations by type',
+    description: 'Detected violations counts',
     example: { PROHIBITED_CONTENT: 2, MISSING_INFORMATION: 1 },
-    type: 'object',
-    additionalProperties: { type: 'number' },
+    type: 'array',
   })
   @IsObject()
   @Type(() => Object)
-  violationCounts!: Record<string, number>;
+  violationCounts!: Array<{ count: number; name: string; id: string }>;
 }
diff --git a/services/workflows-service/src/business-report/merchant-monitoring-client.ts b/services/workflows-service/src/business-report/merchant-monitoring-client.ts
@@ -64,7 +64,13 @@ const MetricsResponseSchema = z.object({
     high: z.number(),
     critical: z.number(),
   }),
-  violationCounts: z.record(z.string(), z.number()),
+  violationCounts: z.array(
+    z.object({
+      name: z.string(),
+      id: z.string(),
+      count: z.number(),
+    }),
+  ),
 });
 
 @Injectable()