Skip to content

PR de correção#30

Open
bellacbs wants to merge 121 commits intocorrecao-projetofrom
master
Open

PR de correção#30
bellacbs wants to merge 121 commits intocorrecao-projetofrom
master

Conversation

@bellacbs
Copy link

Olá Galeraa, segue o PR de correção, não precisam fazer merge

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

lá Galeraa, que site lindoo, cores bem harmônicas e funcionalidades funcionando, adorei colocarem a cor predominante do pokemon como background, tenho apenas uma observação:

  • na página de detalhes não consegui adicionar um pokemon à pokedex, uma sujestão seria colocar a função de adicionarPokemon no estado global, porque ai vocês conseguiriam utilizar tanto no CardPokemons, quanto na página de detalhes
  • ao fazer requisição dos dados, pode ser que em algum momento a Api apresente erro, seria interessante, no catch do useRequetData, colocar um alert para o usuário, poderia ser do tipo “Erro inesperado, tente novamente”

No mais, vocês estão de parabéns, amei o site, serão excelentes desenvolvedores

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.

4 participants