Fix types to ensure calendarType is explicitly passed

wojtekmaj · Oct 17, 2023 · 10fcd92 · 10fcd92
1 parent 746b966
commit 10fcd92
Show file tree

Hide file tree

Showing 7 changed files with 9 additions and 7 deletions.
diff --git a/packages/react-calendar/src/MonthView.spec.tsx b/packages/react-calendar/src/MonthView.spec.tsx
@@ -14,6 +14,7 @@ const { format } = new Intl.DateTimeFormat('en-US', {
 describe('MonthView', () => {
   const defaultProps = {
     activeStartDate: new Date(2017, 0, 1),
+    calendarType: 'iso8601',
     valueType: 'day',
   } satisfies React.ComponentProps<typeof MonthView>;
 

diff --git a/packages/react-calendar/src/MonthView/Day.spec.tsx b/packages/react-calendar/src/MonthView/Day.spec.tsx
@@ -6,10 +6,11 @@ import Day from './Day.js';
 
 const tileProps = {
   activeStartDate: new Date(2018, 0, 1),
+  calendarType: 'iso8601',
   classes: ['react-calendar__tile'],
   currentMonthIndex: 0,
   date: new Date(2018, 0, 1),
-};
+} satisfies React.ComponentProps<typeof Day>;
 
 describe('Day', () => {
   it('applies given classNames properly', () => {

diff --git a/packages/react-calendar/src/MonthView/Day.tsx b/packages/react-calendar/src/MonthView/Day.tsx
@@ -15,7 +15,7 @@ import type { CalendarType, DeprecatedCalendarType } from '../shared/types.js';
 const className = 'react-calendar__month-view__days__day';
 
 type DayProps = {
-  calendarType?: CalendarType | DeprecatedCalendarType;
+  calendarType: CalendarType | DeprecatedCalendarType | undefined;
   classes?: string[];
   currentMonthIndex: number;
   formatDay?: typeof defaultFormatDay;

diff --git a/packages/react-calendar/src/MonthView/Days.tsx b/packages/react-calendar/src/MonthView/Days.tsx
@@ -11,7 +11,7 @@ import type { CalendarType, DeprecatedCalendarType } from '../shared/types.js';
 
 type DaysProps = {
   activeStartDate: Date;
-  calendarType?: CalendarType | DeprecatedCalendarType;
+  calendarType: CalendarType | DeprecatedCalendarType | undefined;
   showFixedNumberOfWeeks?: boolean;
   showNeighboringMonth?: boolean;
 } & Omit<
@@ -89,8 +89,8 @@ export default function Days(props: DaysProps) {
           key={date.getTime()}
           {...otherProps}
           {...otherTileProps}
-          calendarType={calendarType}
           activeStartDate={activeStartDate}
+          calendarType={calendarTypeOrDeprecatedCalendarType}
           currentMonthIndex={monthIndex}
           date={date}
         />

diff --git a/packages/react-calendar/src/MonthView/WeekNumbers.tsx b/packages/react-calendar/src/MonthView/WeekNumbers.tsx
@@ -15,7 +15,7 @@ import type {
 
 type WeekNumbersProps = {
   activeStartDate: Date;
-  calendarType?: CalendarType | DeprecatedCalendarType;
+  calendarType: CalendarType | DeprecatedCalendarType | undefined;
   onClickWeekNumber?: OnClickWeekNumberFunc;
   onMouseLeave?: () => void;
   showFixedNumberOfWeeks?: boolean;

diff --git a/packages/react-calendar/src/MonthView/Weekdays.tsx b/packages/react-calendar/src/MonthView/Weekdays.tsx
@@ -17,7 +17,7 @@ const className = 'react-calendar__month-view__weekdays';
 const weekdayClassName = `${className}__weekday`;
 
 type WeekdaysProps = {
-  calendarType?: CalendarType | DeprecatedCalendarType;
+  calendarType: CalendarType | DeprecatedCalendarType | undefined;
   formatShortWeekday?: typeof defaultFormatShortWeekday;
   formatWeekday?: typeof defaultFormatWeekday;
   locale?: string;

diff --git a/packages/react-calendar/src/shared/utils.ts b/packages/react-calendar/src/shared/utils.ts
@@ -164,7 +164,7 @@ const calendarTypeMap: Record<DeprecatedCalendarType, CalendarType> = {
 };
 
 function isDeprecatedCalendarType(
-  calendarType?: CalendarType | DeprecatedCalendarType,
+  calendarType: CalendarType | DeprecatedCalendarType | undefined,
 ): calendarType is DeprecatedCalendarType {
   return calendarType !== undefined && calendarType in DEPRECATED_CALENDAR_TYPES;
 }