Mejora el ejercicio! - Githubissues

oneeyedman commented 5 years ago

Buen trabajo @MartaSobrino !

Algunas pistas para acabarlo:

[x] Como sugerencia general, comprueba continuamente tu aplicación en Chrome y el terminal dónde arrancas la app con npm start. Si algo falla, hay que arreglarlo antes de continuar.
[x] Utiliza el nombre del Pokemon para el Alt de la imagen
[x] Usa el id de cada pokemon para el key de react cuando mapeas y creas un listado de componentes, ten cuidado de no escribir demasiadas {} y acabar pasando un objeto
[x] Añade estilos (un archivo sass o css por componente)
[x] Usa propTypes para comprobar el tipo de dato de las props, siempre que pases props
[x] Revisa los metadatos en index.html (lang, title...)
[x] Publica en Github Pages

Cuando tengas esto puedes:

Mover el array de pokemons a otro archivo e importarlo en App.
Inicializar el estado de App con el array de pokemons y utilizar este en lugar de la constante pokemon.
Añadir un click a Pokemon que: marca o desmarca el pokemon como favorito en el array del estado de App.
Pasar este dato al componente Pokemon por props
Pintar Pokemon con otros estilos cuando por props nos llegue la info de que el pokemon es favorito

Si tienes cualquier duda, avisa!

Al turrón

oneeyedman commented 5 years ago

Hola, @MartaSobrino: está todo genial excepto el sistema de los favoritos. Si quieres mañana nos sentamos y lo vemos.

MartaSobrino commented 5 years ago

Por favor 🙂

El mié., 22 may. 2019 23:57, Carlos Mañas notifications@github.com escribió:

Hola, @MartaSobrino https://github.com/MartaSobrino: está todo genial excepto el sistema de los favoritos. Si quieres mañana nos sentamos y lo vemos.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/Adalab/f-m3-evaluacion-intermedia-MartaSobrino/issues/1?email_source=notifications&email_token=ALPMUSDCM7DSXSIHMFXOSPTPWW6WJA5CNFSM4HNSAJB2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWAPDWQ#issuecomment-494989786, or mute the thread https://github.com/notifications/unsubscribe-auth/ALPMUSDX46LUKQZXJ4QPSTTPWW6WJANCNFSM4HNSAJBQ .

oneeyedman commented 5 years ago

Buén trabajo, @MartaSobrino, lo damos por resuelto!

MartaSobrino / f-m3-evaluacion-intermedia-MartaSobrino

Mejora el ejercicio! #1