Skip to content

Conversation

@leticia-chijo
Copy link

PR de Correção

Vivian-Oliveira and others added 26 commits February 8, 2021 19:03
criei componentes e fiz rotas das pages
realização da chamada na api e monstam na tela de listar pokemon
criacao da logica para adicionar itens na pokedex
ajustei card das paginas listar e pokedex
Copy link
Author

@leticia-chijo leticia-chijo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Olá! O projeto de vocês foi considerado dentro do esperado, parabéns! 🎉

O layout está bonito e o site funciona. Além disso, o readme está lindinho, continuem fazendo isso, agrega bastante pra quem tá vendo seu projeto (e, portanto, ajuda a ter um bom portfólio no futuro)! Algumas observações:

Um dos requisitos do projeto era que "Ao clicar no botão de adicionar, o card do pokémon clicado deve desaparecer da tela". Vocês implementaram isso, mas quando eu saio da página e volto, os pokémons que foram retirados aparecem e daí vocês fizeram uma verificação se o pokémon existe antes de ele ser adicionado de novo. Isso é uma solução paliativa, mas não é o que foi pedido e não é o melhor para a experiência do usuário.

Uma dica para resolver esse ponto é: a requisição da tela inicial não precisa ser feita todas as vezes, apenas uma. A lista a ser mostrada não é a que vem na API mas sim uma lista controlada por vocês no estado global. Assim, seriam três listas: a de todos os pokémons (vinda da API), a da pokédex (com os pokémons selecionados) e a da página inicial (que basicamente é a de todos os pokémons menos os pokémons selecionados).

Mas é isso gente, o projeto está bem legal mesmo assim. Se algum ponto ainda não ficou claro, assistam ao vídeo com a sugestão de resolução que enviamos na sexta e, se sobrar alguma dúvida, por favor mandem lá no canal! ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants