From 0468c2f29ca09f89ad68238d845f537d541a1876 Mon Sep 17 00:00:00 2001
From: Daniel James <djdany444@gmail.com>
Date: Sun, 26 Jan 2025 19:42:19 +0530
Subject: [PATCH] Review changes

---
 .../pages/RunValidatorsPage/CompetitionRangeSelector.jsx  | 6 +++---
 .../Panel/pages/RunValidatorsPage/ValidationListView.jsx  | 8 ++++----
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/app/webpacker/components/Panel/pages/RunValidatorsPage/CompetitionRangeSelector.jsx b/app/webpacker/components/Panel/pages/RunValidatorsPage/CompetitionRangeSelector.jsx
index 5c71c08cd2..796d1a0468 100644
--- a/app/webpacker/components/Panel/pages/RunValidatorsPage/CompetitionRangeSelector.jsx
+++ b/app/webpacker/components/Panel/pages/RunValidatorsPage/CompetitionRangeSelector.jsx
@@ -9,14 +9,14 @@ import Errored from '../../../Requests/Errored';
 const RUN_VALIDATORS_QUERY_CLIENT = new QueryClient();
 
 export default function CompetitionRangeSelector({ range, setRange }) {
-  const enableCompetitionCountFetch = Boolean(range?.startDate && range?.endDate);
+  const bothDatesAreSelected = Boolean(range?.startDate && range?.endDate);
 
   const {
     data: competitionCount, isLoading, isError,
   } = useQuery({
     queryKey: ['competitionCountInRange', range?.startDate, range?.endDate],
     queryFn: () => getCompetitionCount(range?.startDate, range?.endDate),
-    enabled: enableCompetitionCountFetch,
+    enabled: bothDatesAreSelected,
   }, RUN_VALIDATORS_QUERY_CLIENT);
 
   return (
@@ -49,7 +49,7 @@ export default function CompetitionRangeSelector({ range, setRange }) {
           });
         }}
       />
-      {enableCompetitionCountFetch && (
+      {bothDatesAreSelected && (
         <CompetitionCountViewer
           isLoading={isLoading}
           isError={isError}
diff --git a/app/webpacker/components/Panel/pages/RunValidatorsPage/ValidationListView.jsx b/app/webpacker/components/Panel/pages/RunValidatorsPage/ValidationListView.jsx
index 287be4757b..d71cbd66b4 100644
--- a/app/webpacker/components/Panel/pages/RunValidatorsPage/ValidationListView.jsx
+++ b/app/webpacker/components/Panel/pages/RunValidatorsPage/ValidationListView.jsx
@@ -1,4 +1,4 @@
-import React, { useMemo } from 'react';
+import React, { Fragment, useMemo } from 'react';
 import _ from 'lodash';
 import { Header, List, ListItem } from 'semantic-ui-react';
 import I18n from '../../../../lib/i18n';
@@ -19,11 +19,11 @@ export default function ValidationListView({ validations, showCompetitionNameOnO
   return (
     <>
       {Object.entries(listByGroup).map(([group, list]) => (
-        <>
+        <Fragment key={group}>
           <Header as="h5">{`${headingPrefixForType(type)} ${group}`}</Header>
           <List bulleted>
             {list.map((validationData) => (
-              <ListItem>
+              <ListItem key={validationData.id}>
                 <ValidationText
                   validationData={validationData}
                   group={group}
@@ -32,7 +32,7 @@ export default function ValidationListView({ validations, showCompetitionNameOnO
               </ListItem>
             ))}
           </List>
-        </>
+        </Fragment>
       ))}
     </>
   );