fix(web): code smells

Author: alcercu

web/src/components/CasesDisplay/CasesGrid.tsx

@@ -72,16 +72,16 @@ const CasesGrid: React.FC<ICasesGrid> = ({ disputes, casesPerPage, totalPages, c
           <CasesListHeader />
           {isUndefined(disputes)
             ? [...Array(casesPerPage)].map((_, i) => <StyledSkeleton key={i} />)
-            : disputes.map((dispute, i) => {
-                return <DisputeCard key={i} {...dispute} />;
+            : disputes.map((dispute) => {
+                return <DisputeCard key={dispute.id} {...dispute} />;
               })}
         </ListContainer>
       ) : (
         <GridContainer>
           {isUndefined(disputes)
             ? [...Array(casesPerPage)].map((_, i) => <StyledSkeleton key={i} />)
-            : disputes.map((dispute, i) => {
-                return <DisputeCard key={i} {...dispute} overrideIsList />;
+            : disputes.map((dispute) => {
+                return <DisputeCard key={dispute.id} {...dispute} overrideIsList />;
               })}
         </GridContainer>
       )}

web/src/components/Field.tsx

@@ -1,7 +1,6 @@
 import React from "react";
 import { Link } from "react-router-dom";
 import styled from "styled-components";
-import { useIsList } from "context/IsListProvider";
 
 const FieldContainer = styled.div<FieldContainerProps>`
   width: ${({ isList }) => (isList ? "auto" : "100%")};

web/src/hooks/queries/useCourtTree.ts

@@ -34,7 +34,7 @@ const courtTreeQuery = graphql(`
 `);
 
 export const useCourtTree = () => {
-  return useQuery({
+  return useQuery<CourtTreeQuery>({
     queryKey: ["courtTreeQuery"],
     queryFn: async () => await graphqlQueryFnHelper(courtTreeQuery, {}),
   });
@@ -46,8 +46,8 @@ interface IItem {
   children?: IItem[];
 }
 
-export const rootCourtToItems = (court: CourtTreeQuery["court"], value?: "id" | "path"): IItem => ({
-  label: court!.name ? court!.name : "Unnamed Court",
-  value: value === "id" ? court!.id : `/courts/${court!.id}`,
-  children: court!.children.length > 0 ? court!.children.map((child) => rootCourtToItems(child)) : undefined,
+export const rootCourtToItems = (court: NonNullable<CourtTreeQuery["court"]>, value?: "id" | "path"): IItem => ({
+  label: court.name ? court.name : "Unnamed Court",
+  value: value === "id" ? court.id : `/courts/${court.id}`,
+  children: court.children.length > 0 ? court.children.map((child) => rootCourtToItems(child)) : undefined,
 });