Feat/solucionador
Implementar tela inicial:
- Adicionar tabela de solucionador
- Reutilizar tabela de clientes
- Adicionar paineis em ambos
- Arrumar erros de lint em todo o projeto
Obs : Há um erro no console quando expande os painéis. Ele é do material-ui e ainda está sendo arrumado, será arrumado na versão 5.x (nesse link dá pra ver que teve movimentação sobre essa issue há 3 dias)
Template Code Review
- Dar contexto (breve resumo) da task desenvolvida, e explicar solução implementada
- Código na mesma linguagem: português
- Convenções de naming e de estrutura de diretórios da linguagem: https://www.w3schools.com/js/js_conventions.asp
- Nomes precisos e consistentes
- Remover dead code e imports não utilizados
- Remover coisas “development only” hardcoded no código
- Utilizar encapsulamento (modificadores de acesso)
- Evitar variáveis globais, favorecer imutabilidade (https://martinfowler.com/bliki/ValueObject.html)
- Não deixar console.log de debugging
-
Máximo 10 arquivos
- Quebrar em PRs pequenos, incrementais
- Mais seguro, menos chance de passar algo errado
-
Mínimo 10 testes unitários
- Testar edge cases, passando null, passando valores muito grandes, etc (pensar e validar todas possibilidades)
- Testes que façam sentido, testar lógica
-
Design está complexo?
- Muitas classes / métodos?
- Classes / métodos rasos, com pouco valor, ou que não adicionam nenhuma abstração?
- Código duplicado, uma mesma informação espalhada em muitos lugares?
- Nomes vagos?
- Código obscuro, trechos de código não óbvios? (adicionar comentários)