From ea4eeee49ea09f90d51e3dc3ff26524543745545 Mon Sep 17 00:00:00 2001
From: Akshat Garg <60404253+akshatgarg12@users.noreply.github.com>
Date: Fri, 25 Jun 2021 21:13:04 +0530
Subject: [PATCH] feat(client): ts-migrate header components (#42287)

---
 client/src/components/Header/Header.test.js   |  6 +--
 .../Header/components/{Login.js => Login.tsx} | 37 +++++++------
 .../Header/components/MenuButton.js           | 38 --------------
 .../{AuthOrProfile.js => auth-or-profile.tsx} | 23 ++++----
 .../Header/components/menu-button.tsx         | 44 ++++++++++++++++
 .../components/{NavLinks.js => nav-links.tsx} | 52 +++++++++++--------
 .../components/{NavLogo.js => nav-logo.tsx}   |  4 +-
 .../{universalNav.css => universal-nav.css}   |  0
 .../{UniversalNav.js => universal-nav.tsx}    | 42 ++++++++-------
 .../{UserState.js => user-state.tsx}          | 28 +++++-----
 .../components/Header/{index.js => index.tsx} | 43 +++++++++------
 11 files changed, 175 insertions(+), 142 deletions(-)
 rename client/src/components/Header/components/{Login.js => Login.tsx} (64%)
 delete mode 100644 client/src/components/Header/components/MenuButton.js
 rename client/src/components/Header/components/{AuthOrProfile.js => auth-or-profile.tsx} (67%)
 create mode 100644 client/src/components/Header/components/menu-button.tsx
 rename client/src/components/Header/components/{NavLinks.js => nav-links.tsx} (81%)
 rename client/src/components/Header/components/{NavLogo.js => nav-logo.tsx} (82%)
 rename client/src/components/Header/components/{universalNav.css => universal-nav.css} (100%)
 rename client/src/components/Header/components/{UniversalNav.js => universal-nav.tsx} (66%)
 rename client/src/components/Header/components/{UserState.js => user-state.tsx} (69%)
 rename client/src/components/Header/{index.js => index.tsx} (58%)

diff --git a/client/src/components/Header/Header.test.js b/client/src/components/Header/Header.test.js
index e93717a9f1..7311a3f992 100644
--- a/client/src/components/Header/Header.test.js
+++ b/client/src/components/Header/Header.test.js
@@ -1,9 +1,9 @@
 import React from 'react';
 import ShallowRenderer from 'react-test-renderer/shallow';
 
-import { UniversalNav } from './components/UniversalNav';
-import { NavLinks } from './components/NavLinks';
-import AuthOrProfile from './components/AuthOrProfile';
+import { UniversalNav } from './components/universal-nav';
+import { NavLinks } from './components/nav-links';
+import AuthOrProfile from './components/auth-or-profile';
 
 import envData from '../../../../config/env.json';
 
diff --git a/client/src/components/Header/components/Login.js b/client/src/components/Header/components/Login.tsx
similarity index 64%
rename from client/src/components/Header/components/Login.js
rename to client/src/components/Header/components/Login.tsx
index 4e92238532..6045ce46cb 100644
--- a/client/src/components/Header/components/Login.js
+++ b/client/src/components/Header/components/Login.tsx
@@ -1,5 +1,9 @@
+/* eslint-disable react/prop-types */
+/* eslint-disable @typescript-eslint/no-unsafe-call */
+/* eslint-disable @typescript-eslint/restrict-template-expressions */
+/* eslint-disable import/no-unresolved */
+/* eslint-disable @typescript-eslint/no-unsafe-assignment */
 import React from 'react';
-import PropTypes from 'prop-types';
 import { connect } from 'react-redux';
 import { createSelector } from 'reselect';
 import { Button } from '@freecodecamp/react-bootstrap';
@@ -16,14 +20,21 @@ const mapStateToProps = createSelector(isSignedInSelector, isSignedIn => ({
   isSignedIn
 }));
 
-function Login(props) {
+export interface LoginProps {
+  block?: boolean;
+  children?: unknown;
+  'data-test-label'?: string;
+  isSignedIn?: boolean;
+}
+
+const Login = ({
+  block,
+  children,
+  'data-test-label': dataTestLabel,
+  isSignedIn
+}: LoginProps): JSX.Element => {
   const { t } = useTranslation();
-  const {
-    block,
-    'data-test-label': dataTestLabel,
-    children,
-    isSignedIn
-  } = props;
+
   const href = isSignedIn ? `${homeLocation}/learn` : `${apiLocation}/signin`;
   return (
     <Button
@@ -35,14 +46,8 @@ function Login(props) {
       {children || t('buttons.sign-in')}
     </Button>
   );
-}
-
-Login.displayName = 'Login';
-Login.propTypes = {
-  block: PropTypes.bool,
-  children: PropTypes.any,
-  'data-test-label': PropTypes.string,
-  isSignedIn: PropTypes.bool
 };
 
+Login.displayName = 'Login';
+
 export default connect(mapStateToProps)(Login);
diff --git a/client/src/components/Header/components/MenuButton.js b/client/src/components/Header/components/MenuButton.js
deleted file mode 100644
index 2aba749603..0000000000
--- a/client/src/components/Header/components/MenuButton.js
+++ /dev/null
@@ -1,38 +0,0 @@
-import React from 'react';
-import PropTypes from 'prop-types';
-import { useTranslation } from 'react-i18next';
-import AuthOrProfile from './AuthOrProfile';
-
-const MenuButton = props => {
-  const { t } = useTranslation();
-
-  return (
-    <>
-      <button
-        aria-expanded={props.displayMenu}
-        className={
-          'toggle-button-nav' +
-          (props.displayMenu ? ' reverse-toggle-color' : '')
-        }
-        onClick={props.onClick}
-        ref={props.innerRef}
-      >
-        {t('buttons.menu')}
-      </button>
-      <span className='navatar'>
-        <AuthOrProfile user={props.user} />
-      </span>
-    </>
-  );
-};
-
-MenuButton.displayName = 'MenuButton';
-MenuButton.propTypes = {
-  className: PropTypes.string,
-  displayMenu: PropTypes.bool.isRequired,
-  innerRef: PropTypes.object,
-  onClick: PropTypes.func.isRequired,
-  user: PropTypes.object
-};
-
-export default MenuButton;
diff --git a/client/src/components/Header/components/AuthOrProfile.js b/client/src/components/Header/components/auth-or-profile.tsx
similarity index 67%
rename from client/src/components/Header/components/AuthOrProfile.js
rename to client/src/components/Header/components/auth-or-profile.tsx
index 71815125a3..29efaf69ab 100644
--- a/client/src/components/Header/components/AuthOrProfile.js
+++ b/client/src/components/Header/components/auth-or-profile.tsx
@@ -1,16 +1,18 @@
-/* eslint-disable react/sort-prop-types */
+/* eslint-disable @typescript-eslint/ban-types */
+/* eslint-disable @typescript-eslint/restrict-template-expressions */
+/* eslint-disable @typescript-eslint/ban-ts-comment */
+/* eslint-disable @typescript-eslint/no-unsafe-member-access */
+/* eslint-disable @typescript-eslint/no-unsafe-assignment */
+// @ts-nocheck
 import React from 'react';
 import { Link, borderColorPicker, AvatarRenderer } from '../../helpers';
-import PropTypes from 'prop-types';
 import { useTranslation } from 'react-i18next';
+import Login from './Login';
 
-import Login from '../components/Login';
-
-const propTypes = {
-  user: PropTypes.object
-};
-
-export function AuthOrProfile({ user }) {
+export interface AuthOrProfileProps {
+  user?: Object;
+}
+const AuthOrProfile = ({ user }: AuthOrProfileProps): JSX.Element => {
   const { t } = useTranslation();
   const isUserDonating = user && user.isDonating;
   const isUserSignedIn = user && user.username;
@@ -39,8 +41,7 @@ export function AuthOrProfile({ user }) {
       </>
     );
   }
-}
+};
 
-AuthOrProfile.propTypes = propTypes;
 AuthOrProfile.displayName = 'AuthOrProfile';
 export default AuthOrProfile;
diff --git a/client/src/components/Header/components/menu-button.tsx b/client/src/components/Header/components/menu-button.tsx
new file mode 100644
index 0000000000..8dbbef13fa
--- /dev/null
+++ b/client/src/components/Header/components/menu-button.tsx
@@ -0,0 +1,44 @@
+/* eslint-disable @typescript-eslint/ban-types */
+/* eslint-disable react/prop-types */
+import React, { RefObject } from 'react';
+import { useTranslation } from 'react-i18next';
+import AuthOrProfile from './auth-or-profile';
+
+export interface MenuButtonProps {
+  className?: string;
+  displayMenu?: boolean;
+  innerRef?: RefObject<HTMLButtonElement>;
+  onClick?: React.MouseEventHandler<HTMLButtonElement> | undefined;
+  user?: Object;
+}
+
+const MenuButton = ({
+  displayMenu,
+  innerRef,
+  onClick,
+  user
+}: MenuButtonProps): JSX.Element => {
+  const { t } = useTranslation();
+
+  return (
+    <>
+      <button
+        aria-expanded={displayMenu}
+        className={
+          'toggle-button-nav' + (displayMenu ? ' reverse-toggle-color' : '')
+        }
+        onClick={onClick}
+        ref={innerRef}
+      >
+        {t('buttons.menu')}
+      </button>
+      <span className='navatar'>
+        <AuthOrProfile user={user} />
+      </span>
+    </>
+  );
+};
+
+MenuButton.displayName = 'MenuButton';
+
+export default MenuButton;
diff --git a/client/src/components/Header/components/NavLinks.js b/client/src/components/Header/components/nav-links.tsx
similarity index 81%
rename from client/src/components/Header/components/NavLinks.js
rename to client/src/components/Header/components/nav-links.tsx
index eae270cc9c..550f0f1d36 100644
--- a/client/src/components/Header/components/NavLinks.js
+++ b/client/src/components/Header/components/nav-links.tsx
@@ -1,6 +1,15 @@
+/* eslint-disable @typescript-eslint/ban-ts-comment */
+/* eslint-disable @typescript-eslint/no-unsafe-call */
+/* eslint-disable @typescript-eslint/explicit-module-boundary-types */
+/* eslint-disable @typescript-eslint/no-unsafe-return */
+/* eslint-disable @typescript-eslint/no-explicit-any */
+/* eslint-disable @typescript-eslint/ban-types */
+/* eslint-disable react/prop-types */
+/* eslint-disable @typescript-eslint/no-unsafe-member-access */
+/* eslint-disable @typescript-eslint/restrict-template-expressions */
+// @ts-nocheck
 import React, { Component, Fragment } from 'react';
 import { connect } from 'react-redux';
-import PropTypes from 'prop-types';
 import { withTranslation } from 'react-i18next';
 import { FontAwesomeIcon } from '@fortawesome/react-fontawesome';
 import {
@@ -15,33 +24,33 @@ import { updateUserFlag } from '../../../redux/settings';
 import envData from '../../../../../config/env.json';
 import createLanguageRedirect from '../../createLanguageRedirect';
 import createExternalRedirect from '../../createExternalRedirects';
-
-const { clientLocale, radioLocation, apiLocation } = envData;
-
-const {
+import {
   availableLangs,
   i18nextCodes,
   langDisplayNames
-} = require('../../../../../config/i18n/all-langs');
+} from '../../../../../config/i18n/all-langs';
+
+const { clientLocale, radioLocation, apiLocation } = envData;
 
 const locales = availableLangs.client;
 
-const propTypes = {
-  displayMenu: PropTypes.bool,
-  fetchState: PropTypes.shape({ pending: PropTypes.bool }),
-  i18n: PropTypes.object,
-  t: PropTypes.func,
-  toggleDisplayMenu: PropTypes.func,
-  toggleNightMode: PropTypes.func.isRequired,
-  user: PropTypes.object
-};
+export interface NavLinksProps {
+  displayMenu?: boolean;
+  fetchState?: { pending: boolean };
+  i18n: Object;
+  t: (x: any) => any;
+  toggleDisplayMenu?: React.MouseEventHandler<HTMLButtonElement>;
+  toggleNightMode: (x: any) => any;
+  user?: Record<string, unknown>;
+}
 
 const mapDispatchToProps = {
-  toggleNightMode: theme => updateUserFlag({ theme })
+  toggleNightMode: (theme: unknown) => updateUserFlag({ theme })
 };
 
-export class NavLinks extends Component {
-  toggleTheme(currentTheme = 'default', toggleNightMode) {
+export class NavLinks extends Component<NavLinksProps, {}> {
+  static displayName: string;
+  toggleTheme(currentTheme = 'default', toggleNightMode: any) {
     toggleNightMode(currentTheme === 'night' ? 'default' : 'night');
   }
 
@@ -54,7 +63,7 @@ export class NavLinks extends Component {
       toggleDisplayMenu,
       toggleNightMode,
       user: { isDonating = false, username, theme }
-    } = this.props;
+    }: NavLinksProps = this.props;
 
     const { pending } = fetchState;
     return pending ? (
@@ -141,7 +150,7 @@ export class NavLinks extends Component {
           }
           disabled={!username}
           key='theme'
-          onClick={() => this.toggleTheme(theme, toggleNightMode)}
+          onClick={() => this.toggleTheme(String(theme), toggleNightMode)}
         >
           {username ? (
             <>
@@ -165,7 +174,7 @@ export class NavLinks extends Component {
             <button
               className='nav-link nav-link-lang nav-link-flex'
               key={'lang-' + lang}
-              onClick={() => toggleDisplayMenu()}
+              onClick={toggleDisplayMenu}
             >
               <span>{langDisplayNames[lang]}</span>
               <FontAwesomeIcon icon={faCheck} />
@@ -202,7 +211,6 @@ export class NavLinks extends Component {
   }
 }
 
-NavLinks.propTypes = propTypes;
 NavLinks.displayName = 'NavLinks';
 
 export default connect(null, mapDispatchToProps)(withTranslation()(NavLinks));
diff --git a/client/src/components/Header/components/NavLogo.js b/client/src/components/Header/components/nav-logo.tsx
similarity index 82%
rename from client/src/components/Header/components/NavLogo.js
rename to client/src/components/Header/components/nav-logo.tsx
index 49b3917a1b..ace653c28c 100644
--- a/client/src/components/Header/components/NavLogo.js
+++ b/client/src/components/Header/components/nav-logo.tsx
@@ -1,9 +1,9 @@
 import React from 'react';
 import FreeCodeCampLogo from '../../../assets/icons/FreeCodeCamp-logo';
 
-function NavLogo() {
+const NavLogo = (): JSX.Element => {
   return <FreeCodeCampLogo />;
-}
+};
 
 NavLogo.displayName = 'NavLogo';
 export default NavLogo;
diff --git a/client/src/components/Header/components/universalNav.css b/client/src/components/Header/components/universal-nav.css
similarity index 100%
rename from client/src/components/Header/components/universalNav.css
rename to client/src/components/Header/components/universal-nav.css
diff --git a/client/src/components/Header/components/UniversalNav.js b/client/src/components/Header/components/universal-nav.tsx
similarity index 66%
rename from client/src/components/Header/components/UniversalNav.js
rename to client/src/components/Header/components/universal-nav.tsx
index 4cfb9e58c1..ee8a2de0ec 100644
--- a/client/src/components/Header/components/UniversalNav.js
+++ b/client/src/components/Header/components/universal-nav.tsx
@@ -1,20 +1,31 @@
-import React from 'react';
-import PropTypes from 'prop-types';
-
+/* eslint-disable @typescript-eslint/no-unsafe-call */
+/* eslint-disable @typescript-eslint/no-unsafe-assignment */
+/* eslint-disable @typescript-eslint/ban-ts-comment */
+/* eslint-disable react/prop-types */
+// @ts-nocheck
+import React, { Ref } from 'react';
 import { Link, SkeletonSprite } from '../../helpers';
-import NavLogo from './NavLogo';
-import MenuButton from './MenuButton';
-import NavLinks from './NavLinks';
-import './universalNav.css';
+import NavLogo from './nav-logo';
+import MenuButton from './menu-button';
+import NavLinks from './nav-links';
 import { isLanding } from '../../../utils/path-parsers';
-
 import Loadable from '@loadable/component';
 
+import './universal-nav.css';
+
 const SearchBar = Loadable(() => import('../../search/searchBar/SearchBar'));
-const SearchBarOptimized = Loadable(() =>
-  import('../../search/searchBar/search-bar-optimized')
+const SearchBarOptimized = Loadable(
+  () => import('../../search/searchBar/search-bar-optimized')
 );
 
+export interface UniversalNavProps {
+  displayMenu?: boolean;
+  fetchState?: { pending: boolean };
+  menuButtonRef?: Ref<HTMLButtonElement> | undefined;
+  searchBarRef?: unknown;
+  toggleDisplayMenu?: React.MouseEventHandler<HTMLButtonElement> | undefined;
+  user?: Record<string, unknown>;
+}
 export const UniversalNav = ({
   displayMenu,
   toggleDisplayMenu,
@@ -22,7 +33,7 @@ export const UniversalNav = ({
   searchBarRef,
   user,
   fetchState
-}) => {
+}: UniversalNavProps): JSX.Element => {
   const { pending } = fetchState;
 
   const search =
@@ -77,12 +88,3 @@ export const UniversalNav = ({
 
 UniversalNav.displayName = 'UniversalNav';
 export default UniversalNav;
-
-UniversalNav.propTypes = {
-  displayMenu: PropTypes.bool,
-  fetchState: PropTypes.shape({ pending: PropTypes.bool }),
-  menuButtonRef: PropTypes.object,
-  searchBarRef: PropTypes.object,
-  toggleDisplayMenu: PropTypes.func,
-  user: PropTypes.object
-};
diff --git a/client/src/components/Header/components/UserState.js b/client/src/components/Header/components/user-state.tsx
similarity index 69%
rename from client/src/components/Header/components/UserState.js
rename to client/src/components/Header/components/user-state.tsx
index 3d598891eb..72981789d0 100644
--- a/client/src/components/Header/components/UserState.js
+++ b/client/src/components/Header/components/user-state.tsx
@@ -1,5 +1,8 @@
+/* eslint-disable react/prop-types */
+/* eslint-disable @typescript-eslint/no-unsafe-member-access */
+/* eslint-disable @typescript-eslint/no-explicit-any */
+/* eslint-disable @typescript-eslint/no-unsafe-assignment */
 import React from 'react';
-import PropTypes from 'prop-types';
 import { connect } from 'react-redux';
 import { createSelector } from 'reselect';
 import { Link } from 'gatsby';
@@ -12,21 +15,20 @@ import Login from './Login';
 const mapStateToProps = createSelector(
   userFetchStateSelector,
   isSignedInSelector,
-  (fetchState, isSignedIn) => ({
+  (fetchState: any, isSignedIn: any) => ({
     isSignedIn,
     showLoading: fetchState.pending
   })
 );
 
-const propTypes = {
-  disableSettings: PropTypes.bool,
-  email: PropTypes.string,
-  isSignedIn: PropTypes.bool,
-  name: PropTypes.string,
-  showLoading: PropTypes.bool
-};
-
-function UserState(props) {
+export interface UserStateProps {
+  disableSettings?: boolean;
+  email?: string;
+  isSignedIn?: boolean;
+  name?: string;
+  showLoading?: boolean;
+}
+const UserState = (props: UserStateProps): JSX.Element => {
   const { isSignedIn, showLoading, disableSettings } = props;
   const { t } = useTranslation();
 
@@ -39,7 +41,6 @@ function UserState(props) {
         className='user-state-spinner'
         color='white'
         fadeIn='none'
-        height='38px'
         name='line-scale'
       />
     );
@@ -51,9 +52,8 @@ function UserState(props) {
   ) : (
     <Login />
   );
-}
+};
 
 UserState.displayName = 'UserState';
-UserState.propTypes = propTypes;
 
 export default connect(mapStateToProps)(UserState);
diff --git a/client/src/components/Header/index.js b/client/src/components/Header/index.tsx
similarity index 58%
rename from client/src/components/Header/index.js
rename to client/src/components/Header/index.tsx
index 768f9b22b5..faaef88a1e 100644
--- a/client/src/components/Header/index.js
+++ b/client/src/components/Header/index.tsx
@@ -1,18 +1,28 @@
+/* eslint-disable @typescript-eslint/no-unsafe-member-access */
+/* eslint-disable @typescript-eslint/explicit-module-boundary-types */
+/* eslint-disable @typescript-eslint/no-unsafe-call */
+/* eslint-disable @typescript-eslint/no-explicit-any */
+/* eslint-disable @typescript-eslint/unbound-method */
+/* eslint-disable @typescript-eslint/ban-types */
 import React from 'react';
 import Helmet from 'react-helmet';
-import PropTypes from 'prop-types';
 
-import UniversalNav from './components/UniversalNav';
+import UniversalNav from './components/universal-nav';
 
 import './header.css';
 
-const propTypes = {
-  fetchState: PropTypes.shape({ pending: PropTypes.bool }),
-  user: PropTypes.object
-};
-
-export class Header extends React.Component {
-  constructor(props) {
+export interface HeaderProps {
+  fetchState: { pending: boolean };
+  user: Record<string, any>;
+}
+export class Header extends React.Component<
+  HeaderProps,
+  { displayMenu: boolean }
+> {
+  menuButtonRef: React.RefObject<any>;
+  searchBarRef: React.RefObject<any>;
+  static displayName: string;
+  constructor(props: HeaderProps) {
     super(props);
     this.state = {
       displayMenu: false
@@ -23,15 +33,15 @@ export class Header extends React.Component {
     this.toggleDisplayMenu = this.toggleDisplayMenu.bind(this);
   }
 
-  componentDidMount() {
+  componentDidMount(): void {
     document.addEventListener('click', this.handleClickOutside);
   }
 
-  componentWillUnmount() {
+  componentWillUnmount(): void {
     document.removeEventListener('click', this.handleClickOutside);
   }
 
-  handleClickOutside(event) {
+  handleClickOutside(event: any): void {
     if (
       this.state.displayMenu &&
       this.menuButtonRef.current &&
@@ -43,10 +53,12 @@ export class Header extends React.Component {
     }
   }
 
-  toggleDisplayMenu() {
-    this.setState(({ displayMenu }) => ({ displayMenu: !displayMenu }));
+  toggleDisplayMenu(): void {
+    this.setState(({ displayMenu }: { displayMenu: boolean }) => ({
+      displayMenu: !displayMenu
+    }));
   }
-  render() {
+  render(): JSX.Element {
     const { displayMenu } = this.state;
     const { fetchState, user } = this.props;
     return (
@@ -69,7 +81,6 @@ export class Header extends React.Component {
   }
 }
 
-Header.propTypes = propTypes;
 Header.displayName = 'Header';
 
 export default Header;