fix: Se elimina useEffect del MediaCard para que no rompa el ruteo

taller2fiuba · Jul 31, 2020 · 98a2a37 · 98a2a37
1 parent 652e895
commit 98a2a37
Show file tree

Hide file tree

Showing 5 changed files with 11 additions and 10 deletions.
diff --git a/src/components/AlertaInformativa.jsx b/src/components/AlertaInformativa.jsx
@@ -30,7 +30,7 @@ const AlertInformativa = ({ onClose, token }) => {
           <Alert severity="info">
             <AlertTitle>Importante</AlertTitle>
             El siguiente token de autorización se mostrará una sola vez: <br />
-            <Box width="45%">
+            <Box width="40%">
               <code>{token}</code>
             </Box>
           </Alert>

diff --git a/src/components/MediaCard.jsx b/src/components/MediaCard.jsx
@@ -1,4 +1,4 @@
-import React, { useState, useEffect } from "react";
+import React, { useState } from "react";
 import { withStyles, makeStyles } from "@material-ui/core/styles";
 import Card from "@material-ui/core/Card";
 import CardContent from "@material-ui/core/CardContent";
@@ -56,7 +56,7 @@ const MediaCard = ({ titulo, obtenerTotal, icono, colorFondo, colorIcono }) => {
     }
   };
 
-  useEffect(obtenerSubtitulo, []);
+  obtenerSubtitulo();
 
   return (
     <Card className={classes.root} style={{ backgroundColor: colorFondo }}>

diff --git a/src/components/ResponsiveLineChart.jsx b/src/components/ResponsiveLineChart.jsx
@@ -85,7 +85,8 @@ const ResponsiveLineChart = ({ data, colorSchema, tick }) => (
 );
 
 ResponsiveLineChart.propTypes = {
-  data: PropTypes.node.isRequired,
+  // eslint-disable-next-line react/forbid-prop-types
+  data: PropTypes.array.isRequired,
   colorSchema: PropTypes.string.isRequired,
   tick: PropTypes.string.isRequired,
 };

diff --git a/src/components/SideBarResponsive.jsx b/src/components/SideBarResponsive.jsx
@@ -55,7 +55,7 @@ const SideBarResponsive = (props) => {
           <ListItemIcon>
             <FontAwesomeIcon icon="home" className={classes.MenuItem} />
           </ListItemIcon>
-          <ListItemText primary="Home" />
+          <ListItemText primary="Home" className={classes.MenuItem} />
         </MenuItem>
         <MenuItem
           component={Link}
@@ -66,7 +66,7 @@ const SideBarResponsive = (props) => {
           <ListItemIcon>
             <FontAwesomeIcon icon="chart-bar" className={classes.MenuItem} />
           </ListItemIcon>
-          <ListItemText primary="Estado" />
+          <ListItemText primary="Estado" className={classes.MenuItem} />
         </MenuItem>
         <MenuItem
           component={Link}
@@ -77,7 +77,7 @@ const SideBarResponsive = (props) => {
           <ListItemIcon>
             <FontAwesomeIcon icon="user" className={classes.MenuItem} />
           </ListItemIcon>
-          <ListItemText primary="Usuarios" />
+          <ListItemText primary="Usuarios" className={classes.MenuItem} />
         </MenuItem>
         <MenuItem
           component={Link}
@@ -88,7 +88,7 @@ const SideBarResponsive = (props) => {
           <ListItemIcon>
             <FontAwesomeIcon icon="file-video" className={classes.MenuItem} />
           </ListItemIcon>
-          <ListItemText primary="Videos" />
+          <ListItemText primary="Videos" className={classes.MenuItem} />
         </MenuItem>
         <MenuItem
           component={Link}
@@ -99,7 +99,7 @@ const SideBarResponsive = (props) => {
           <ListItemIcon>
             <FontAwesomeIcon icon="server" className={classes.MenuItem} />
           </ListItemIcon>
-          <ListItemText primary="Servers" />
+          <ListItemText primary="Servers" className={classes.MenuItem} />
         </MenuItem>
       </MenuList>
     </div>

diff --git a/src/estadisticas/PantallaEstadisticas.jsx b/src/estadisticas/PantallaEstadisticas.jsx
@@ -9,8 +9,8 @@ import FormControl from "@material-ui/core/FormControl";
 import Select from "@material-ui/core/Select";
 import BeatLoader from "react-spinners/BeatLoader";
 
-import MediaCard from "../components/MediaCard";
 import ResponsiveLineChart from "../components/ResponsiveLineChart";
+import MediaCard from "../components/MediaCard";
 
 import * as MediaServerService from "../comunications/MediaServerService";
 import * as AuthServerService from "../comunications/AuthServerService";