---
# System prepended metadata

title: Feedback's Avaliações
tags: [Softplan, Avaliação Frontend, Entrevista Front]

---

###### 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

