-
Notifications
You must be signed in to change notification settings - Fork 0
Feedback #4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: correcao-projeto
Are you sure you want to change the base?
Feedback #4
Conversation
Pokedex_V1
pokedex_card
projeto-finalizado
labenu-bot
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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
Parabéns pela entrega. O projeto está excelente.
Visualmente está muito bom! Curti a forma como os dados foram tratados no estado global.
Dei check na parte responsiva, de modo geral está funcionando, porém deu uma quebradinha bem de leve na parte de detalhes onde fica sobrando mais espaço no lado direito, mas nada que comprometa.
Algumas sugestões de melhorias:
Tratar erro de requisição da API, só no console.log deixará usuário perdido.
E tratar caso o usuário "caia" em uma página que não exista, setando na rota uma pagina de erro.
A pasta Pages não foi utilizada da forma esperada, as paginas estavam vindo de Components, não é um defeito em si, mas é confuso para quem for ler o código.
Faltou um readme mais completo, documentando o projeto.
Mais uma vez parabéns pela entrega e pelo empenho durante a realização do projeto. Ficou excelente! Continue assim
Análise realizada por: Andrea Soares
POR FAVOR, EDITE ESSA MENSAGEM INSERINDO AS INFORMAÇÕES DO SEU PROJETO. COMECE APAGANDO ESSA LINHA.
INSIRA O NOME NOME DO PROJETO AQUI
O que funciona
O que não funciona
Link Surge
A PARTIR DA SEMANA 5, INSIRA AQUI O LINK DO SURGE. ANTES DISSO, APAGUE ESSAS DUAS LINHAS.
Imagens
TIRE PRINTS DAS TELAS DO SEU SITE E COLE AQUI