Skip to content
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

Case 4 #59

Merged
merged 24 commits into from
Jun 27, 2022
Merged

Case 4 #59

merged 24 commits into from
Jun 27, 2022

Conversation

mdudapires
Copy link
Collaborator

@mdudapires mdudapires requested a review from bellacbs June 6, 2022 21:29
@mdudapires mdudapires requested a review from FlavioAAMotta June 22, 2022 12:43
Copy link

@FlavioAAMotta FlavioAAMotta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Única página disponível se não estiver logado;✅
Criar um usuário básico para acesso.✅
Uma página de lista de dragões:✅
Os nomes devem estar em ordem alfabética;✅
A partir da lista, deverá ser possível remover e alterar as informações dos dragões.✅
Uma página com os detalhes de um dragão específico:✅
Data de criação;✅
Nome;✅
Tipo.✅
Uma página para cadastro de dragões:✅

Layout responsivo;
Utilizar Stencil, React, Angular 6+ ou Web Components;✅
Usar um sistema de controle de versão para entregar o teste (Github, Bitbucket, ...).✅
Organização do código;✅
Componentização das páginas;✅
Interface organizada e amigável;
Uso adequado do css/sass/less. ✅

Comentários da pessoa avaliadora
Duda, parabéns seu projeto está muito bom.
Alguns pontos podem ser melhorados, como:

  • Design
  • Corrigir useEffect sem vetor na página de listar dragões
  • responsividade
    São questões que se pega com o tempo, no geral acredito que está num caminho muito bom e logo estará em algum lugar que te permitirá crescer muito! Parabéns

@mdudapires mdudapires merged commit 12b5a6f into master Jun 27, 2022
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