Skip to content

Conversation

@0dayIg0r
Copy link
Collaborator

No description provided.

Copy link

@labenu-bot labenu-bot left a 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
A aplicação terá que suportar 6 sorteios: Mega-sena, Quina, Lotofácil, Lotomania, Timemania e Dia de sorte
Seguiu o Layout padrão e seus temas -
Ao mudar esse combo-box, terá que mudar o tema do sorteio, número do sorteio, data do sorteio e números sorteados. -
A aplicação terá que ser responsiva, pelo menos para celulares, conforme layout. -
Criar rotas com React Router DOM (opcional). -
Criou um Readme com descrição do projeto. -
Disponibilizou o link do deploy
Feedback do código Feito
React com TypeScript -
Testes com React Testing Library e/ou Cypress -
O consumo da API pode ser feito via REST ou GraphQL.
Todos esses sorteios estarão em um combo-box/select.

Comentários da pessoa avaliadora

Olá Igor!

Parabéns pela entrega deste case!

No seu case percebe-se que você usou o layout como um "guia", porém algumas coisas ficaram diferentes do solicitado, como a curva da lateral (que aparece apenas no mobile) e as cores usadas em algumas das loterias não são as mesmas do layout disponibilizado no figma.
Sobre o mobile, apesar de a curva estar aparecendo, ele acabou deixando ocultas algumas informações da parte escrita.

Na exibição de número e data do sorteio aparece o número do concurso, mas a data de hoje e não a data de realização do sorteio, como é pedido e disponibilizado na API.

Mais um detalhe é que para exibir este projeto em um portfólio eu recomendo que corrija estes detalhes citados, além de fazer um bom readme. Trazer informações sobre como foi desenvolvido o projeto, quem desenvolveu (se foi sozinho ou em grupo), quais as tecnologias você usou, qual a proposta deste projeto, o que funciona e como usar.

No mais, parabens por mais um case concluído e tenha uma excelente semana!

Análise realizada por: Clara

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.

3 participants