generated from future4code/pokedex-template
-
Notifications
You must be signed in to change notification settings - Fork 2
PR de Correção #16
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
Open
indiomedeiros
wants to merge
43
commits into
correcao-projeto
Choose a base branch
from
master
base: correcao-projeto
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
PR de Correção #16
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Inicio Projeto
router atualizado
Inserindo useRequest
inserindo logica do PokeCard
atualização url.js e homepage.js
atualização CardPoke
Estilização dos cards
atualização header
detalhes de pokemon com a rota feitos
alguns ajustes
…m vez de ele fazer a requisição da foto, assim existe apenas 1 fonte de verdade, e o trabalho finalizou =)
Correção da responsividade do cardDetails
listas funcionais, precisa arrumar o card que puxa as fotos
readme e pagina de erro feita
cursor pointer
labenu-bot
approved these changes
Jun 21, 2021
Contributor
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 Pokemon | ✅ |
| Cada Pokemon da lista é representado como um card com opções de adicionar à Pokedex e ver detalhes | ✅ |
| O usuário consegue adicionar um Pokemon à sua Pokedex | ✅ |
| O usuário consegue remover um Pokemon da Pokedex | ✅ |
| Não é possível adicionar duas vezes o mesmo Pokemon na Pokedex | ✅ |
| Cabeçalho com botões para transitar entre Home e Pokedex | ✅ |
| O usuário consegue usar o botão no cabeçalho da página de Detalhes para adicionar ou remover Pokemon da Pokedex | ✅ |
| O usuário consegue acessar a página da Pokedex, onde encontra uma lista dos Pokemons adicionados | ✅ |
| O usuário consegue abrir uma página de detalhes específicos de um Pokemon, esteja ele na Home ou na Pokedex | ✅ |
| Listas de Pokemons paginadas [Desafio] | - |
| O usuário consegue comparar dois Pokemons [Desafio] | - |
| Todas as telas do site são responsivas [Desafio] | ✅ |
| Quão fácil foi achar as informações no site? | Fácil |
| Quão fácil foi realizar as operações pedidas? | Fácil |
| O que você achou do design do site? | Bom |
| Funcionalidade extra | - |
| O usuário consegue realizar uma batalha Pokemon | - |
| Qualidade de código | Alto |
| Feedback do código | Feito |
|---|---|
Isolou corretamente as constantes do axiosConfig |
✅ |
| Requisições isoladas dos componentes | ✅ |
| Utilizou estado global para armazenar a lista de Pokemons | ✅ |
| Não 'forçou' a atualização da página ao remover Pokemon | ✅ |
| Utilizou corretamente os custom hooks | ✅ |
| Boa indentação | ✅ |
| Utilizou corretamente o router | ✅ |
Comentários da pessoa avaliadora
Oi Gente, parabéns pelo trabalho incrível! O projeto atende as principais funcionalidades com um código bem estruturado e organizado. È hora de comemorar com o time, trocar elogios e agradecer pela colaboração.
Análise realizada por: Índio
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Não precisa mergear