Merge pull request #186 from Andres2D/remove-errors-and-add-title

Remove errors and add title
Andres2D · Jan 22, 2023 · cee6fa5 · cee6fa5 · vercel · Jan 22, 2023
2 parents 41f631b + e2e020e
commit cee6fa5
Show file tree

Hide file tree

Showing 13 changed files with 169 additions and 139 deletions.
diff --git a/accessibility/header-setting.tsx b/accessibility/header-setting.tsx
@@ -0,0 +1,21 @@
+import type { NextPage } from 'next';
+import Head from 'next/head';
+
+interface Props {
+  title?: string;
+  description?: string;
+}
+
+const HeaderSettings: NextPage<Props> = ({title = 'CotejoApp', description = 'CotejoApp'}) => {
+  return (
+    <Head>
+      <title>{title}</title>
+      <meta 
+        name='description'
+        content={description}
+      />
+    </Head>
+  )
+}
+
+export default HeaderSettings;
diff --git a/components/layout/navbar.tsx b/components/layout/navbar.tsx
@@ -44,7 +44,7 @@ const Navbar: NextPage = () => {
         />
         <Avatar
           className={styles.avatar}
-          src={image || 'https://bit.ly/broken-link'}
+          src={image || ''}
           onClick={onOpen}
           ref={btnRef}
         />
@@ -61,7 +61,7 @@ const Navbar: NextPage = () => {
           <DrawerHeader className={styles.headerMenu}>
             <Avatar
               className={styles.avatarMenu}
-              src={image || 'https://bit.ly/broken-link'}
+              src={image || ''}
               onClick={() => {
                 onClose();
                 router.push('/profile');
@@ -99,7 +99,7 @@ const Navbar: NextPage = () => {
             >
               Log out
             </Button>
-            <p>Version 2.3.0</p>
+            <p>Version 2.3.1</p>
           </DrawerFooter>
         </DrawerContent>
       </Drawer>

diff --git a/components/matches/detail/avatar/change-player.modal.tsx b/components/matches/detail/avatar/change-player.modal.tsx
@@ -158,7 +158,7 @@ const ChangePlayerModal: NextPage = () => {
           <div className={styles.playerAction}>
             <Avatar
               size='lg'
-              src={inPlayer?.image || 'https://bit.ly/broken-link'}
+              src={inPlayer?.image || ''}
             />
             <div className={styles.inputChange}>
               <Input

diff --git a/components/profile/player-card/avatar-selector.tsx b/components/profile/player-card/avatar-selector.tsx
@@ -47,7 +47,7 @@ const AvatarSelector: NextPage = () => {
       <div className={styles.container} key={key}>
         <Avatar
           className={styles.avatarList}
-          src={src || 'https://bit.ly/broken-link'}
+          src={src || ''}
           size="xl"
           onClick={() => changeAvatarHandler(src)}
         />
@@ -71,7 +71,7 @@ const AvatarSelector: NextPage = () => {
       <div className={styles.avatarContainer} onClick={onOpen}>
         <Avatar
           className={styles.avatar}
-          src={image || 'https://bit.ly/broken-link'}
+          src={image || ''}
           size="2xl"
         />
         <IconButton

diff --git a/constants/players-bots-data.ts b/constants/players-bots-data.ts
@@ -5,176 +5,154 @@ export const botsData: IPlayerList[] = [
     nationality: 'World',
     position: 'CM',
     name: 'Bot one',
-    image:
-      'https://bit.ly/broken-link',
+    image: '',
     _id: '637917fe1ad0a31f963f8ccd',
   },
   {
     nationality: 'World',
     position: 'CM',
     name: 'Bot Two',
-    image:
-      'https://bit.ly/broken-link',
+    image: '',
     _id: '6379187a1ad0a31f963f8cdc',
   },
   {
     nationality: 'World',
     position: 'CM',
     name: 'Bot Three',
-    image:
-      'https://bit.ly/broken-link',
+    image: '',
     _id: '637918931ad0a31f963f8ce5',
   },
   {
     nationality: 'World',
     position: 'CM',
     name: 'Bot Four',
-    image:
-      'https://bit.ly/broken-link',
+    image: '',
     _id: '637c1082f70e87605d1e699d',
   },
   {
     nationality: 'World',
     position: 'CM',
     name: 'Bot Five',
-    image:
-      'https://bit.ly/broken-link',
+    image: '',
     _id: '637918a91ad0a31f963f8cee',
   },
   {
     nationality: 'World',
     position: 'CM',
     name: 'Bot Six',
-    image:
-      'https://bit.ly/broken-link',
+    image: '',
     _id: '637918c51ad0a31f963f8cf7',
   },
   {
     nationality: 'World',
     position: 'CM',
     name: 'Bot Seven',
-    image:
-      'https://bit.ly/broken-link',
+    image: '',
     _id: '637918e61ad0a31f963f8d00',
   },
   {
     nationality: 'World',
     position: 'CM',
     name: 'Bot Eight',
-    image:
-      'https://bit.ly/broken-link',
+    image: '',
     _id: '637919111ad0a31f963f8d09',
   },
   {
     nationality: 'World',
     position: 'CM',
     name: 'Bot Nine',
-    image:
-      'https://bit.ly/broken-link',
+    image: '',
     _id: '637919321ad0a31f963f8d12',
   },
   {
     nationality: 'World',
     position: 'CM',
     name: 'Bot Ten',
-    image:
-      'https://bit.ly/broken-link',
+    image: '',
     _id: '637919451ad0a31f963f8d18',
   },
   {
     nationality: 'World',
     position: 'CM',
     name: 'Bot Eleven',
-    image:
-      'https://bit.ly/broken-link',
+    image: '',
     _id: '6379195b1ad0a31f963f8d1e',
   },
   {
     nationality: 'World',
     position: 'CM',
     name: 'Bot Twelve',
-    image:
-      'https://bit.ly/broken-link',
+    image: '',
     _id: '637919b41ad0a31f963f8d24',
   },
   {
     nationality: 'World',
     position: 'CM',
     name: 'Bot Thirteen',
-    image:
-      'https://bit.ly/broken-link',
+    image: '',
     _id: '637919de1ad0a31f963f8d2a',
   },
   {
     nationality: 'World',
     position: 'CM',
     name: 'Bot Fourteen',
-    image:
-      'https://bit.ly/broken-link',
+    image: '',
     _id: '63791a011ad0a31f963f8d30',
   },
   {
     nationality: 'World',
     position: 'CM',
     name: 'Bot Fiveteen',
-    image:
-      'https://bit.ly/broken-link',
+    image: '',
     _id: '63791a1b1ad0a31f963f8d36',
   },
   {
     nationality: 'World',
     position: 'CM',
     name: 'Bot Sixteen',
-    image:
-      'https://bit.ly/broken-link',
+    image: '',
     _id: '63791a321ad0a31f963f8d3c',
   },
   {
     nationality: 'World',
     position: 'CM',
     name: 'Bot Seventeen',
-    image:
-      'https://bit.ly/broken-link',
+    image: '',
     _id: '63791a511ad0a31f963f8d42',
   },
   {
     nationality: 'World',
     position: 'CM',
     name: 'Bot Eighteen',
-    image:
-      'https://bit.ly/broken-link',
+    image: '',
     _id: '63791b081ad0a31f963f8d48',
   },
   {
     nationality: 'World',
     position: 'CM',
     name: 'Bot Nineteen',
-    image:
-      'https://bit.ly/broken-link',
+    image: '',
     _id: '63791b291ad0a31f963f8d4e',
   },
   {
     nationality: 'World',
     position: 'CM',
     name: 'Bot Twenty',
-    image:
-      'https://bit.ly/broken-link',
+    image: '',
     _id: '63791b491ad0a31f963f8d54',
   },
   {
     nationality: 'World',
     position: 'CM',
     name: 'Bot Twenty One',
-    image:
-      'https://bit.ly/broken-link',
+    image: '',
     _id: '63791b711ad0a31f963f8d5a',
   },
   {
     nationality: 'World',
     position: 'CM',
     name: 'Bot Twenty Two',
-    image:
-      'https://bit.ly/broken-link',
+    image: '',
     _id: '63791b881ad0a31f963f8d60',
   },
 ];
diff --git a/pages/api/player/index.ts b/pages/api/player/index.ts
@@ -49,7 +49,7 @@ const registerPlayer = async(req: any, res: any) => {
       nickname: name,
       number: 0,
       status: '',
-      image: 'https://bit.ly/broken-link',
+      image: '',
       google: false,
       nationality: 'Colombia',
       position: 'CM'

diff --git a/pages/auth/index.tsx b/pages/auth/index.tsx
@@ -4,24 +4,28 @@ import { Link, Image } from '@chakra-ui/react';
 import { getSession } from 'next-auth/react';
 import styles from './index.module.scss';
 import LoginForm from '../../components/auth/login-form';
+import HeaderSettings from '../../accessibility/header-setting';
 
 const Login: NextPage = () => {
   return (
-    <div className={styles.login}>
-      <Image
-        src={'/images/app-logo-regular.png'}
-        alt='Cotejo app logo'
-        width={325}
-        height={219}
-        className={styles.logo}
-      />
-      <LoginForm />
-      <NextLink href='/sign-up'>
-        <Link className={styles.signUpLink}>
-          {`Don't have an account?, Sign up`}
-        </Link>
-      </NextLink>
-    </div>
+    <>
+      <HeaderSettings title='Login' description='Login to CotejoApp' />
+      <div className={styles.login}>
+        <Image
+          src={'/images/app-logo-regular.png'}
+          alt='Cotejo app logo'
+          width={325}
+          height={219}
+          className={styles.logo}
+        />
+        <LoginForm />
+        <NextLink href='/sign-up'>
+          <Link className={styles.signUpLink}>
+            {`Don't have an account?, Sign up`}
+          </Link>
+        </NextLink>
+      </div>
+    </>
   );
 }
 

diff --git a/pages/matches/[...matchId]/index.tsx b/pages/matches/[...matchId]/index.tsx
@@ -3,6 +3,7 @@ import type { NextPage } from 'next';
 import { getSession } from 'next-auth/react';
 import { useEffect } from 'react';
 import { useDispatch } from 'react-redux';
+import HeaderSettings from '../../../accessibility/header-setting';
 import MatchDetailsLayout from '../../../components/matches/detail/match-detail';
 import { IMatchDetailsResponse } from '../../../interfaces/Match';
 import { getMatch } from '../../../server/matches';
@@ -44,7 +45,13 @@ const MatchDetails: NextPage<Props> = ({match, profile}) => {
   }, [dispatch, matchDetail])
 
   return (
-    <MatchDetailsLayout />
+    <>
+      <HeaderSettings 
+        title={`${matchDetail.match.home_team.name} vs ${matchDetail.match.away_team.name}`} 
+        description='Set the line uo' 
+      />
+      <MatchDetailsLayout />
+    </>
   );
 }
 

diff --git a/pages/matches/index.tsx b/pages/matches/index.tsx
@@ -11,6 +11,7 @@ import styles from './matches.module.scss';
 import { FullMatch, IMatch } from '../../interfaces/Match';
 import { profileActions } from '../../store/profile.slice';
 import { getProfile } from '../../server/player';
+import HeaderSettings from '../../accessibility/header-setting';
 
 interface Props {
   matches: string;
@@ -41,18 +42,21 @@ const Matches: NextPage<Props> = ({matches, profile}) => {
   }, []);
 
   return (
-    <section className={styles.matches}>
-      <Button 
-        size='lg'
-        colorScheme='brand'
-        onClick={() => router.push('/matches/new')}
-      >
-        New Match
-      </Button>
-      <MatchList
-        matches={matchesList}
-      />
-    </section>
+    <>
+      <HeaderSettings title='Matches' description='Select your match' />
+      <section className={styles.matches}>
+        <Button 
+          size='lg'
+          colorScheme='brand'
+          onClick={() => router.push('/matches/new')}
+        >
+          New Match
+        </Button>
+        <MatchList
+          matches={matchesList}
+        />
+      </section>
+    </>
   );
 }