PR de Correção

[yMarceloMaia]

PR de Correção

[labenu-bot]

Parabéns pela entrega!

Requisitos do projeto ✅

Implementações	Feito
O usuário consegue acessar a página Home, onde encontra uma lista com, no mínimo, 20 Pokémons	✅
Cada Pokémon da lista é representado como um card com opções de adicionar à Pokédex e ver detalhes	✅
O usuário consegue adicionar um Pokémon à sua Pokédex	✅
O usuário consegue remover um Pokémon da Pokédex	✅
Não é possível adicionar duas vezes o mesmo Pokémon na Pokédex	✅
Cabeçalho com botões para transitar entre Home e Pokédex	✅
O usuário consegue usar o botão no cabeçalho da página de Detalhes para adicionar ou remover Pokémon da Pokédex	✅
O usuário consegue acessar a página da Pokédex, onde encontra uma lista dos Pokémons adicionados	✅
O usuário consegue abrir uma página de detalhes específicos de um Pokémon, esteja ele na Home ou na Pokédex	✅
Quão fácil foi achar as informações no site?	Fácil
O que você achou do design do site?	Excelente
[Desafio] Listas de Pokémons paginadas	-
[Desafio] O usuário consegue fazer uma batalha entre dois Pokémons e há uma resposta de quem é o vitorioso	-
[Desafio] Todas as telas do site são responsivas	✅
Funcionalidade extra	-

Feedback do código	Feito
Requisições isoladas dos componentes	-
Utilizou estado global para armazenar a lista de Pokémons	✅
Não 'forçou' a atualização com o método reload() ao remover um Pokémon	✅
Boa indentação	✅
Utilizou corretamente o router	✅
Todos os erros são tratados e exibidos adequadamente para o usuário	-
Criou link do surge	✅
Criou um bom readme com informações relevantes para o projeto	✅

Comentários da pessoa avaliadora

Olá gente!

Parabéns pela entrega do projeto, ficou muito bom!

Alguns pontos sobre o código!

• Vocês conseguiram implementar todas as funcionalidades principais pedidas no projeto, a parte de responsividade funciona bem na tela mobile, mas notei que ao diminuir o tamanho da tela os cards quebram, isso pode ser resolvido usando display flex junto com flex wap nos media screen no lugar de display grid, fora isso, tudo certo, parabéns.

• Quando clico no botão para ir para a pokedex, esse botão ainda fica mesmo estando na pokedex, poderia ser feito a mesma lógica que foi usada na página de detalhes nos botões de capturar e remover da pokedex, para caso esteja na página de pokedex esse botão mudasse para ir para lista de pokemons por exemplo.

• Quando capturo um pokemon e vou na página da pokedex ele está lá e funciona normal, mas ao clicar em remover estando na parte da pokedex ele não some automaticamente precisando atualizar a página, poderia ser feito uma função para atualizar a pokedex e passaria como props para PokemonCard e chamaria essa função no botão de remover.

• No componente de detalhes da pokedex foram criados bastantes estados, poderiam ter feito menos estados e setados esses estados com as listas de informações, depois faria um map para retornar essas informações diminuindo o número de estados.

• Os erros pegos nos catchs estão com console.log como algumas outras informações ainda aparecem no console, o ideal é que o usuário tenha acesso aos erros de requisição por um alert ou algo do tipo, já os outros consoles é interessante limpar eles antes de fazer o deploy, já que esses foram usados como debug ou para ajudar na implementação do código.

No mais, parabéns pela entrega, mandaram bem!

Tenham uma ótima semana!

Análise realizada por: Marcelo Maia