From 14cfb2c76218b0ba2a11ff959ffb0334d0c58144 Mon Sep 17 00:00:00 2001 From: Wojciech Maj Date: Wed, 26 Jul 2023 00:05:35 +0200 Subject: [PATCH] Remove propTypes from internal components --- test/LocaleOptions.tsx | 6 ------ test/MaxDetailOptions.tsx | 7 ------- test/MinDetailOptions.tsx | 7 ------- test/ValidityOptions.tsx | 10 ---------- test/ValueOptions.tsx | 10 ---------- test/ViewOptions.tsx | 14 -------------- test/package.json | 1 - test/yarn.lock | 1 - 8 files changed, 56 deletions(-) diff --git a/test/LocaleOptions.tsx b/test/LocaleOptions.tsx index d91fb60b..564c5d8d 100644 --- a/test/LocaleOptions.tsx +++ b/test/LocaleOptions.tsx @@ -1,5 +1,4 @@ import React, { useRef } from 'react'; -import PropTypes from 'prop-types'; type LocaleOptionsProps = { locale: string | undefined; @@ -103,8 +102,3 @@ export default function LocaleOptions({ locale, setLocale }: LocaleOptionsProps) ); } - -LocaleOptions.propTypes = { - locale: PropTypes.string, - setLocale: PropTypes.func.isRequired, -}; diff --git a/test/MaxDetailOptions.tsx b/test/MaxDetailOptions.tsx index d5d94ced..3bd21d56 100644 --- a/test/MaxDetailOptions.tsx +++ b/test/MaxDetailOptions.tsx @@ -1,5 +1,4 @@ import React from 'react'; -import PropTypes from 'prop-types'; import type { Detail } from './shared/types'; @@ -49,9 +48,3 @@ export default function MaxDetailOptions({ ); } - -MaxDetailOptions.propTypes = { - maxDetail: PropTypes.oneOf(allViews).isRequired, - minDetail: PropTypes.oneOf(allViews).isRequired, - setMaxDetail: PropTypes.func.isRequired, -}; diff --git a/test/MinDetailOptions.tsx b/test/MinDetailOptions.tsx index ee38f78e..007a640a 100644 --- a/test/MinDetailOptions.tsx +++ b/test/MinDetailOptions.tsx @@ -1,5 +1,4 @@ import React from 'react'; -import PropTypes from 'prop-types'; import type { Detail } from './shared/types'; @@ -49,9 +48,3 @@ export default function MinDetailOptions({ ); } - -MinDetailOptions.propTypes = { - maxDetail: PropTypes.oneOf(allViews).isRequired, - minDetail: PropTypes.oneOf(allViews).isRequired, - setMinDetail: PropTypes.func.isRequired, -}; diff --git a/test/ValidityOptions.tsx b/test/ValidityOptions.tsx index 24d36d3c..20c7a845 100644 --- a/test/ValidityOptions.tsx +++ b/test/ValidityOptions.tsx @@ -1,5 +1,4 @@ import React from 'react'; -import PropTypes from 'prop-types'; import { getISOLocalDate } from '@wojtekmaj/date-utils'; type ValidityOptionsProps = { @@ -75,12 +74,3 @@ export default function ValidityOptions({ ); } - -ValidityOptions.propTypes = { - maxDate: PropTypes.instanceOf(Date), - minDate: PropTypes.instanceOf(Date), - required: PropTypes.bool, - setMaxDate: PropTypes.func.isRequired, - setMinDate: PropTypes.func.isRequired, - setRequired: PropTypes.func.isRequired, -}; diff --git a/test/ValueOptions.tsx b/test/ValueOptions.tsx index aaaabcef..9b9c2abd 100644 --- a/test/ValueOptions.tsx +++ b/test/ValueOptions.tsx @@ -1,5 +1,4 @@ import React from 'react'; -import PropTypes from 'prop-types'; import { getDayStart, getDayEnd, getISOLocalDate } from '@wojtekmaj/date-utils'; import type { LooseValue } from './shared/types'; @@ -92,12 +91,3 @@ export default function ValueOptions({ setValue, value }: ValueOptionsProps) { ); } - -const isValue = PropTypes.oneOfType([PropTypes.string, PropTypes.instanceOf(Date)]); - -const isValueOrValueArray = PropTypes.oneOfType([isValue, PropTypes.arrayOf(isValue)]); - -ValueOptions.propTypes = { - setValue: PropTypes.func.isRequired, - value: isValueOrValueArray, -}; diff --git a/test/ViewOptions.tsx b/test/ViewOptions.tsx index 22142cf3..67a78018 100644 --- a/test/ViewOptions.tsx +++ b/test/ViewOptions.tsx @@ -1,5 +1,4 @@ import React from 'react'; -import PropTypes from 'prop-types'; type ViewOptionsProps = { disabled: boolean; @@ -107,16 +106,3 @@ export default function ViewOptions({ ); } - -ViewOptions.propTypes = { - disabled: PropTypes.bool.isRequired, - renderInPortal: PropTypes.bool.isRequired, - setDisabled: PropTypes.func.isRequired, - setRenderInPortal: PropTypes.func.isRequired, - setShowLeadingZeros: PropTypes.func.isRequired, - setShowNeighboringMonth: PropTypes.func.isRequired, - setShowWeekNumbers: PropTypes.func.isRequired, - showLeadingZeros: PropTypes.bool.isRequired, - showNeighboringMonth: PropTypes.bool.isRequired, - showWeekNumbers: PropTypes.bool.isRequired, -}; diff --git a/test/package.json b/test/package.json index 431a0829..a3cf4645 100644 --- a/test/package.json +++ b/test/package.json @@ -17,7 +17,6 @@ "dependencies": { "@wojtekmaj/date-utils": "^1.0.0", "@wojtekmaj/react-daterange-picker": "portal:../", - "prop-types": "^15.6.0", "react": "^18.2.0", "react-dom": "^18.2.0" }, diff --git a/test/yarn.lock b/test/yarn.lock index 1610bfaf..7f2a0ccd 100644 --- a/test/yarn.lock +++ b/test/yarn.lock @@ -1644,7 +1644,6 @@ __metadata: "@vitejs/plugin-react": ^3.0.0 "@wojtekmaj/date-utils": ^1.0.0 "@wojtekmaj/react-daterange-picker": "portal:../" - prop-types: ^15.6.0 react: ^18.2.0 react-dom: ^18.2.0 typescript: ^5.0.0