fix(prop-types): remove usage of proptypes (#1140)

Author: philibea

packages/use-dataloader/package.json

@@ -25,9 +25,6 @@
     "directory": "packages/use-dataloader"
   },
   "license": "MIT",
-  "dependencies": {
-    "prop-types": "15.8.1"
-  },
   "peerDependencies": {
     "react": "18.x"
   }

packages/use-dataloader/src/DataLoaderProvider.tsx

@@ -1,4 +1,3 @@
-import PropTypes from 'prop-types'
 import type { ReactElement, ReactNode } from 'react'
 import { createContext, useCallback, useContext, useMemo, useRef } from 'react'
 import {
@@ -56,17 +55,19 @@ export type IDataLoaderContext = {
 // @ts-expect-error we force the context to undefined, should be corrected with default values
 export const DataLoaderContext = createContext<IDataLoaderContext>(undefined)
 
+type DataLoaderProviderProps = {
+  children: ReactNode
+  cacheKeyPrefix?: string
+  onError?: OnErrorFn
+  maxConcurrentRequests?: number
+}
+
 const DataLoaderProvider = ({
   children,
-  cacheKeyPrefix,
+  cacheKeyPrefix = '',
   onError,
-  maxConcurrentRequests,
-}: {
-  children: ReactNode
-  cacheKeyPrefix: string
-  onError: OnErrorFn
-  maxConcurrentRequests?: number
-}): ReactElement => {
+  maxConcurrentRequests = DEFAULT_MAX_CONCURRENT_REQUESTS,
+}: DataLoaderProviderProps): ReactElement => {
   const requestsRef = useRef<Requests>({})
 
   const computeKey = useCallback(
@@ -82,7 +83,7 @@ const DataLoaderProvider = ({
   const addRequest = useCallback(
     (key: string, args: UseDataLoaderInitializerArgs) => {
       if (DataLoader.maxConcurrent !== maxConcurrentRequests) {
-        DataLoader.maxConcurrent = maxConcurrentRequests as number
+        DataLoader.maxConcurrent = maxConcurrentRequests
       }
       if (key && typeof key === 'string') {
         const newRequest = new DataLoader({
@@ -212,19 +213,6 @@ const DataLoaderProvider = ({
   )
 }
 
-DataLoaderProvider.propTypes = {
-  cacheKeyPrefix: PropTypes.string,
-  children: PropTypes.node.isRequired,
-  maxConcurrentRequests: PropTypes.number,
-  onError: PropTypes.func,
-}
-
-DataLoaderProvider.defaultProps = {
-  cacheKeyPrefix: undefined,
-  maxConcurrentRequests: DEFAULT_MAX_CONCURRENT_REQUESTS,
-  onError: undefined,
-}
-
 export const useDataLoaderContext = (): IDataLoaderContext =>
   useContext(DataLoaderContext)
 

packages/use-dataloader/src/__tests__/usePaginatedDataLoader.test.tsx

@@ -19,7 +19,6 @@ const initialProps = {
       }),
   ),
 }
-// eslint-disable-next-line react/prop-types
 const wrapper = ({ children }: { children?: ReactNode }) => (
   <DataLoaderProvider>{children}</DataLoaderProvider>
 )

packages/use-i18n/package.json

@@ -33,8 +33,7 @@
     "date-fns": "2.29.3",
     "filesize": "10.0.6",
     "international-types": "0.3.6",
-    "intl-messageformat": "10.2.5",
-    "prop-types": "15.8.1"
+    "intl-messageformat": "10.2.5"
   },
   "peerDependencies": {
     "date-fns": "2.x",

packages/use-i18n/src/usei18n.tsx

@@ -2,7 +2,6 @@ import type { NumberFormatOptions } from '@formatjs/ecma402-abstract'
 import type { Locale as DateFnsLocale } from 'date-fns'
 import { formatDistanceToNow, formatDistanceToNowStrict } from 'date-fns'
 import type { BaseLocale } from 'international-types'
-import PropTypes from 'prop-types'
 import type { ReactElement, ReactNode } from 'react'
 import {
   createContext,
@@ -397,17 +396,4 @@ const I18nContextProvider = ({
   return <I18nContext.Provider value={value}>{children}</I18nContext.Provider>
 }
 
-I18nContextProvider.propTypes = {
-  children: PropTypes.node.isRequired,
-  defaultDateLocale: PropTypes.shape({}),
-  defaultLoad: PropTypes.func.isRequired,
-  defaultLocale: PropTypes.string.isRequired,
-  defaultTranslations: PropTypes.shape({}),
-  enableDebugKey: PropTypes.bool,
-  enableDefaultLocale: PropTypes.bool,
-  loadDateLocale: PropTypes.func,
-  localeItemStorage: PropTypes.string,
-  supportedLocales: PropTypes.arrayOf(PropTypes.string).isRequired,
-}
-
 export default I18nContextProvider