Skip to content

Inicio Case Chama#60

Open
Thaysaadj wants to merge 5 commits into
masterfrom
rodada-cases-semana-6
Open

Inicio Case Chama#60
Thaysaadj wants to merge 5 commits into
masterfrom
rodada-cases-semana-6

Conversation

@Thaysaadj

@Thaysaadj Thaysaadj commented Mar 22, 2022

Copy link
Copy Markdown
Collaborator

Rodada de Cases - Projeto Chama

O que funciona
Busca de usuáro pelo user name
Renderização da foto, nome, número de repositórios e link no github

O que não funciona
Tela que exibe os históricos de busca

Link Surge
projeto-chama.surge.sh

Imagens
image

image

@labenu-bot labenu-bot left a comment

Copy link
Copy Markdown

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
Fez formulário de pesquisa
Ao enviar o nome de usuário o dados do usuário são apresentados da tela.
As informações de perfil apresentadas são no mínimo 'imagem de avatar', 'nome', 'e-mail', 'biografia'.
As informações do repositório são, no mínimo, 'nome', 'descrição', 'URL' -
Fez campo para exibição do histórico de buscas
Ordenou as buscas, com as mais recentes no topo. -
Ao clicar no item do histórico o usuário pode buscar novamente usando o termo pesquisado -
Usou React & Hooks
Usou CSS(styled-components, CSS modules, ou plain CSS/SASS/LESS)
Criou link do surge
Uso Typescript. -
Funcionalidade extra -
Feedback do código Feito
Criou um README.md -
Isolou corretamente as constantes do `axiosConfig -
Todos os erros são tratados e exibidos adequadamente para o usuário -
Não causou um loop infinito nas requisições chamadas no useEffect
Criou componentes
Bons nomes para variáveis e funções -
Responsivo para dispositivos móveis -

Comentários da pessoa avaliadora

Olá, Thayna! Gostaria de primeiramente parabenizá-la por concluir o terceiro case de final de curso da Labenu! \o/

Pude perceber que você conseguiu desenvolver bem o projeto proposto, conseguindo gerenciar os recursos necessários, ainda que partes não estivessem funcionando! Tá bem bacana :)

De toda forma, gostaria de pontuar algumas coisas que talvez te ajudem nas atividades:

  1. Uma boa prática no desenvolvimento de projetos consiste em tornar a sua utilização eficaz e entendível. No caso do seu código, senti falta de uma explicação mais clara de como o usuário consumiria seus recursos. Como sugestão, você poderia incluir estas instruções no README.md, descrevendo o seu projeto, fornecendo o deploy do projeto, suas limitações e também dando essas instruções para que qualquer pessoa que eventualmente vá acessar suas estruturas consiga fazê-lo de forma prática ;)

  2. Senti um pouco de falta de separação de responsabilidades no seu código, mas num geral tá tudo funcionando direitinho! ;)

  3. Ao testar o seu deploy, percebi que ele não é responsivo, o que é uma pena, pois o projeto está bem bacana ;)

  4. Infelizmente, não consegui testar a lógica de históricos de busca :/ Como sugestão, aqui você poderia pensar em armazenar o que foi digitado pelo usuário em uma variável de estado de tipo array e, toda vez que a página fosse acessada novamente (ou integrando o useEffect como um ComponentDidUpdate), você retornaria a lista de buscas. ;)

No mais, muito sucesso para ti nesta jornada! :)

Análise realizada por: Bruno

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