bazza6/star-wars-api

Hi there 👋

I'm Nazareno, a frontend developer based in Barcelona. I'm passionate about building beautiful and functional user interfaces. Currently learning Node.js and improving my skills in React on a daily basis. Looking for new challenges and opportunities to grow as a developer.

star-wars-api's Issues

Has organizado bien el código, se entiende muy bien y es cómodo navegar por el proyecto

Star-Wars-API/src/pages/Main.js

Line 8 in 56e2eec

let [starships, setStarships] = useState({

Está bien pensada la paginación. Se podrían guardar los datos de todas las naves que se van cargando

El estado que guarda la información de las naves que se obtienen por API, en lugar de ser un objeto que guarda únicamente una página cargada, podría ser un objeto que contenga todas las páginas cargadas.
De esta forma, si el usuario quiere volver a ver una página, no habrá que volver a realizar la llamada API. Un ejemplo de la estructura de datos del estado sería:

let starships = {
   1: {...información de la page 1...},
   2: {...información de la page 1...},
   ...
   n: {...información de la page n...},
}

Star-Wars-API/src/pages/Main.js

Line 8 in 56e2eec

let [starships, setStarships] = useState({

Genial, bien resueltas las rutas protegidas

Star-Wars-API/src/components/GuardedRoute.js

Line 5 in 56e2eec

return <Navigate to="/log-in" replace />;