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

projeto #9

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

projeto #9

wants to merge 1 commit into from

Conversation

hustenhabas
Copy link

Frontend feito com React e backend com node.js. Database mongodb instanciado no docker. Utilizar npm start para o client e npm run dev para a api. Apenas a funçao de hits não foi implementada.

@higorfreire217
Copy link

Frontend feito com React e backend com node.js. Database mongodb instanciado no docker. Utilizar npm start para o client e npm run dev para a api. Apenas a funçao de hits não foi implementada.

Pré-requisitos:

(*) Front-end e Back-end desacoplados?
Sim

(*) Front-end feito em React?
Sim

(*) Back-end feito em Node?
Sim

Layout:

(*) Seguiu referencia?
Meio certo, seguiu mas faltou detalhes. Como o arredondamento da borda no contador de Hits.

(*) Usou cores do Guideline?
Não em tudo, faltou a borda do contador, e a numeração da contagem esta em cor diferente da requisitada

(*) Usou as fontes corretas?
Quase, faltou o texto abaixo do contador

(*) HTML está semântico?
Pegou bem a componentização, contudo, ainda era possível melhorar a semântica, retirando tags em excesso e repetidas.

(*) Javascript em seu escopo global esta limpo ou poluído?
Esta limpo, sem ressalvas

(*) Há muitas dependencias?
Não, as dependencias estão em equilibrio e todos os recursos são utilizados no projeto

Diferenciais:

(*) Boa documentação?
A Documentação é um diferencial, nele poderia estar descrito como instanciar o servidor mongoDB pelo docker e quais preparativos devem ser feitos para que possamos rodar o app do zero como o importantíssimo "npm install" antes do projeto. O conteúdo da documentação se encontra no link: https://facebook.github.io/create-react-app/docs/available-scripts quando deveria ser específico do projeto explicando a arvore de arquivos e diretórios e outras especificações como as dependencias do projeto.

(*) Usou Sass, Less ou outro pré-processador CSS
Não

(*) A Pagina está responsiva
Meio-responsiva, há bastante melhorias a serem consideradas. Como o tamanho do logotipo ou a quebra de linha do botão com o campo de texto do encurtador.

(*) Usou script de Automatização
Não

(*) Interação ao clicar no link
Não

(*) Consumiu JSON especificado para o back-end para a seção TOP 5
Não

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