###### tags: `Softplan` `Entrevista Front` `Avaliação Frontend`

# Feedback's Avaliações

## Ricardofreitasdev 19/11/21
Em geral, conseguiu entregar boa parte dos requisitos solicitados, segue algumas observações:

### Prós

* Uso de uma lib para formulários (formik)
* Uso de uma frameworkUi (mui)
* Responsividade
* Foto de usuário
* Extras
* Back-end próprio
* Documentação ok

### Contras

* Aplicação
* Usuário do tipo `user` consegue criar novos usuários acessando diretamente pela rota.
* Erro ao tentar editar usuários inexistentes (não tratado)
* Campos de input pré-preenchidos (login/cadastro/criação de usuário)
* Falta de redirecionamento em login caso usuário ja logado
* Warnings/Errors no console do navegador durante uso do app
* Falta de padronização de código (eslint/prettier poderia ajudar a padronizar a escrita)
* Dependencias
* Excesso/Redundância (Ex: @material-ui / @mui)
* Não utilização (ex: @emotion, @material-ui/icons, @mui/styles, @testing-library)
* Arquitetura
* Sentimos falta de um padrão de projeto
* Inconsistência de código
* Falta de testes
* Falta de boas praticas de clean code
* Erros de instalação da aplicação (npm install - 13 erros/ 1 erro crítico)

---

## lucassardelari 19/11/21
Em geral, conseguiu entregar boa parte dos requisitos solicitados, segue algumas observações:

### Prós
* Cypress
* Documentação
* Typescript
* pré-processador (Sass)
* Uso de uma frameworkUi (mui)

### Contras
* Aplicação
* Layout inconsistente
* Falta de responsividade do layout
* Ao atualizar meu usuário, a senha vem criptografada, entrando em um looping de criptografia
* Warnings/Erros no `console` do navegador durante uso do app
* Falta de validação no cadastro de usuário (email e senha)
* A tela fica "quebrada" ao trocar a paginação da listagem de usuários
* A tela fica "quebrada" quando o usuário não tem foto cadastrada
* Falta de padronização de código (eslint/prettier poderia ajudar a padronizar a escrita)
* Dependencias
* Excesso/Redundância (Ex: @material-ui / @mui)
* Não utilização (ex: react-router-dom, @material-ui/data-grid, @mui/styles, @testing-library)
* Uso de 'alert' na tela de Login, mesmo tendo disponível o `react-notifications-component`
* Arquitetura
* useEffect sem Array de dependências
* Falta de um gerenciador de rotas (mesmo tendo o 'react-router-dom' nas dependencias)
* Não instalou @types das libs como dependências de desenvolvimento
* Falta de testes unitários