Skip to content

Reforco star wars#37

Open
renewrocha wants to merge 8 commits into
masterfrom
reforco-star-wars
Open

Reforco star wars#37
renewrocha wants to merge 8 commits into
masterfrom
reforco-star-wars

Conversation

@renewrocha

Copy link
Copy Markdown
Collaborator

No description provided.

@renewrocha renewrocha requested a review from tayhsn April 30, 2022 02:30
@desatinar

Copy link
Copy Markdown

Parabéns pela entrega!

Requisitos do projeto ✅

Implementações Feito
O usuário deve ser capaz de visualizar uma lista dos personagens
O usuário deve ser capaz de visualizar uma página com os detalhes de cada personagem
Na página de detalhes, o usuário deve ser capaz de visualizar o nome do(a) personagem
Na página de detalhes, o usuário deve ser capaz de visualizar o planeta de origem do(a) personagem
Na página de detalhes, o usuário deve ser capaz de visualizar um botão que volta para a tela de home
Funcionalidade extra -
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? Podemos melhorar
Todas as requisições possuem loadings
Qualidade de código Alto
Feedback do código Feito
Boa indentação
Nomenclatura de variáveis dentro do padrão
Não deixou nenhum log ou registro desnecessário pelo código -
Isolou constantes que se repetem em arquivos separados
Fez uma boa componentização
Usou só componentes funcionais
Usou um único idioma para escrever código (inglês ou português)
Fez componentização em vez de deixar tudo no App
Extraiu a base url para um arquivo separado e a utilizou em todas as requisições
Evitou misturar tipos de estilização (useStyles vs. styled components)
Não utilizou o método reload() para atualizar a página após a finalização de uma requisição
Utilizou os custom hooks fornecidos em aula (useProtectedPage, useForms, useRequestData) -

Comentários da pessoa avaliadora

Olá, Rene!

Primeiramente, parabéns pela entrega de mais um projetinho da Labenu! \o/

Notei que tu conseguiu fazer todos os requisitos propostos, o que demonstra o esforço que tu tá tendo com os estudos 🎉

Vou só deixar umas observações aqui, tá?

  1. Tu tem alguns arquivos vazios no teu código (services/requests.js, pages/CharacterDetailsPage/styled.js). O ideal seria retirar esses arquivos do teu código já que não estão sendo utilizados

  2. Vi que tu botou um botão no planeta e tem também um componente não utilizado na tua aplicação. Será que faltou inserir a lógica do onClick pra redirecionar pra esse componente não utilizado?

Lembra que os meus comentários só são sugestões, tá? Não são críticas.
No mais, parabéns de novo pela entrega e bem-vindo ao backend. 😁

@renewrocha renewrocha requested review from desatinar and tayhsn and removed request for tayhsn May 20, 2022 00:38
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.

2 participants