refactor: add PropTypes

Author: mrholek

.eslintrc.js

@@ -20,9 +20,9 @@ module.exports = {
     'plugin:prettier/recommended', // Enables eslint-plugin-prettier and eslint-config-prettier. This will display prettier errors as ESLint errors. Make sure this is always the last configuration in the extends array.
   ],
   plugins: ['jsdoc', '@typescript-eslint', 'react', 'react-hooks'],
-  rules: {
-    // Place to specify ESLint rules. Can be used to overwrite rules specified from the extended configs
-    // e.g. "@typescript-eslint/explicit-function-return-type": "off",
-    'react/prop-types': 'off',
-  },
+  // rules: {
+  //   // Place to specify ESLint rules. Can be used to overwrite rules specified from the extended configs
+  //   // e.g. "@typescript-eslint/explicit-function-return-type": "off",
+  //   'react/prop-types': 'off',
+  // },
 }

src/components/accordion/CAccordion.tsx

@@ -1,4 +1,5 @@
 import React, { forwardRef, HTMLAttributes } from 'react'
+import PropTypes from 'prop-types'
 import classNames from 'classnames'
 
 export interface CAccordionProps extends HTMLAttributes<HTMLDivElement> {
@@ -23,4 +24,10 @@ export const CAccordion = forwardRef<HTMLDivElement, CAccordionProps>(
   },
 )
 
+CAccordion.propTypes = {
+  children: PropTypes.node,
+  className: PropTypes.string,
+  flush: PropTypes.bool,
+}
+
 CAccordion.displayName = 'CAccordion'

src/components/accordion/CAccordionBody.tsx

@@ -1,4 +1,5 @@
 import React, { forwardRef, HTMLAttributes } from 'react'
+import PropTypes from 'prop-types'
 import classNames from 'classnames'
 
 export interface CAccordionBodyProps extends HTMLAttributes<HTMLDivElement> {
@@ -20,4 +21,9 @@ export const CAccordionBody = forwardRef<HTMLDivElement, CAccordionBodyProps>(
   },
 )
 
+CAccordionBody.propTypes = {
+  children: PropTypes.node,
+  className: PropTypes.string,
+}
+
 CAccordionBody.displayName = 'CAccordionBody'

src/components/accordion/CAccordionButton.tsx

@@ -1,11 +1,15 @@
 import React, { forwardRef, HTMLAttributes } from 'react'
+import PropTypes from 'prop-types'
 import classNames from 'classnames'
 
 export interface CAccordionButtonProps extends HTMLAttributes<HTMLButtonElement> {
   /**
    * A string of all className you want applied to the base component. [docs]
    */
   className?: string
+  /**
+   * TODO:. [docs]
+   */
   collapsed?: boolean
 }
 
@@ -21,4 +25,10 @@ export const CAccordionButton = forwardRef<HTMLButtonElement, CAccordionButtonPr
   },
 )
 
+CAccordionButton.propTypes = {
+  children: PropTypes.node,
+  className: PropTypes.string,
+  collapsed: PropTypes.bool,
+}
+
 CAccordionButton.displayName = 'CAccordionButton'

src/components/accordion/CAccordionCollapse.tsx

@@ -1,4 +1,5 @@
 import React, { forwardRef } from 'react'
+import PropTypes from 'prop-types'
 import { CCollapse, CCollapseProps } from '../collapse/CCollapse'
 
 export const CAccordionCollapse = forwardRef<HTMLDivElement, CCollapseProps>(
@@ -11,4 +12,8 @@ export const CAccordionCollapse = forwardRef<HTMLDivElement, CCollapseProps>(
   },
 )
 
+CAccordionCollapse.propTypes = {
+  children: PropTypes.node,
+}
+
 CAccordionCollapse.displayName = 'CAccordionCollapse'

src/components/accordion/CAccordionHeader.tsx

@@ -1,4 +1,5 @@
 import React, { forwardRef, HTMLAttributes } from 'react'
+import PropTypes from 'prop-types'
 import classNames from 'classnames'
 
 export interface CAccordionHeaderProps extends HTMLAttributes<HTMLDivElement> {
@@ -20,4 +21,9 @@ export const CAccordionHeader = forwardRef<HTMLDivElement, CAccordionHeaderProps
   },
 )
 
+CAccordionHeader.propTypes = {
+  children: PropTypes.node,
+  className: PropTypes.string,
+}
+
 CAccordionHeader.displayName = 'CAccordionHeader'

src/components/accordion/CAccordionItem.tsx

@@ -1,4 +1,5 @@
 import React, { forwardRef, HTMLAttributes } from 'react'
+import PropTypes from 'prop-types'
 import classNames from 'classnames'
 
 export interface CAccordionItemProps extends HTMLAttributes<HTMLDivElement> {
@@ -20,4 +21,9 @@ export const CAccordionItem = forwardRef<HTMLDivElement, CAccordionItemProps>(
   },
 )
 
+CAccordionItem.propTypes = {
+  children: PropTypes.node,
+  className: PropTypes.string,
+}
+
 CAccordionItem.displayName = 'CAccordionItem'

src/components/alert/CAlert.tsx

@@ -1,4 +1,5 @@
 import React, { forwardRef, HTMLAttributes, useState } from 'react'
+import PropTypes from 'prop-types'
 import classNames from 'classnames'
 import { Transition } from 'react-transition-group'
 
@@ -13,7 +14,7 @@ export interface CAlertProps extends HTMLAttributes<HTMLDivElement> {
   /**
    * Sets the color context of the component to one of CoreUI’s themed colors. [docs]
    *
-   * @type 'primary' | 'secondary' | 'success' | 'danger' | 'warning' | 'info' | 'dark' | 'light' | string
+   * @type {'primary' | 'secondary' | 'success' | 'danger' | 'warning' | 'info' | 'dark' | 'light' | string }
    * @default 'primary'
    */
   color: Colors
@@ -107,4 +108,15 @@ export const CAlert = forwardRef<HTMLDivElement, CAlertProps>(
   },
 )
 
+CAlert.propTypes = {
+  children: PropTypes.node,
+  className: PropTypes.string,
+  color: PropTypes.string.isRequired,
+  dismissible: PropTypes.bool,
+  onDismiss: PropTypes.func,
+  onDismissed: PropTypes.func,
+  variant: PropTypes.string,
+  visible: PropTypes.bool,
+}
+
 CAlert.displayName = 'CAlert'

src/components/alert/CAlertHeading.tsx

@@ -1,4 +1,5 @@
 import React, { ElementType, forwardRef, HTMLAttributes } from 'react'
+import PropTypes from 'prop-types'
 import classNames from 'classnames'
 
 export interface CAlertHeadingProps extends HTMLAttributes<HTMLHeadingElement> {
@@ -26,4 +27,10 @@ export const CAlertHeading = forwardRef<HTMLHeadingElement, CAlertHeadingProps>(
   },
 )
 
+CAlertHeading.propTypes = {
+  children: PropTypes.node,
+  className: PropTypes.string,
+  component: PropTypes.elementType,
+}
+
 CAlertHeading.displayName = 'CAlertHeading'

src/components/alert/CAlertLink.tsx

@@ -1,4 +1,5 @@
 import React, { AnchorHTMLAttributes, forwardRef } from 'react'
+import PropTypes from 'prop-types'
 import classNames from 'classnames'
 
 import { CLink } from '../link/CLink'
@@ -8,14 +9,6 @@ export interface CAlertLinkProps extends AnchorHTMLAttributes<HTMLAnchorElement>
    * A string of all className you want applied to the base component. [docs]
    */
   className?: string
-  /**
-   * The href attribute specifies the URL of the page the link goes to. [docs]
-   */
-  href?: string
-  // /**
-  //  * A string representation of the Link location, created by concatenating the location’s pathname, search, and hash properties. [docs]
-  //  */
-  // to?: string
 }
 
 export const CAlertLink = forwardRef<HTMLAnchorElement, CAlertLinkProps>(
@@ -30,4 +23,9 @@ export const CAlertLink = forwardRef<HTMLAnchorElement, CAlertLinkProps>(
   },
 )
 
+CAlertLink.propTypes = {
+  children: PropTypes.node,
+  className: PropTypes.string,
+}
+
 CAlertLink.displayName = 'CAlertLink'