fix(projectcard): make numberAriaLabel optional but warn

src/components/ProjectCard/ProjectCard.test.tsx

@@ -1,6 +1,17 @@
 import { generateSnapshots } from '@chanzuckerberg/story-utils';
+import { render } from '@testing-library/react';
+import React from 'react';
+import { ProjectCard } from './ProjectCard';
 import * as stories from './ProjectCard.stories';
 
 describe('<ProjectCard />', () => {
   generateSnapshots(stories);
+
+  it('should warn if number is passed without numberAriaLabel', () => {
+    expect(() => {
+      render(<ProjectCard number={1} />);
+    }).toThrow(
+      /You must provide a "numberAriaLabel" for the number icon if a "number" has been passed/,
+    );
+  });
 });

src/components/ProjectCard/ProjectCard.tsx

@@ -56,9 +56,9 @@ export interface Props {
    */
   number?: number;
   /**
-   * Number aria label
+   * Number aria label. Required if number prop is passed.
    */
-  numberAriaLabel: string;
+  numberAriaLabel?: string;
   /**
    * Property passed in to style draggable project card
    */
@@ -81,6 +81,11 @@ export const ProjectCard = ({
   isDragging,
   ...other
 }: Props) => {
+  if (number && !numberAriaLabel && process.env.NODE_ENV !== 'production') {
+    throw new Error(
+      'You must provide a "numberAriaLabel" for the number icon if a "number" has been passed',
+    );
+  }
   const componentClassName = clsx(
     styles['project-card'],
     behavior === 'draggable' && styles['project-card--draggable'],