caroliny-gomes-lima / BANCO-COMUNITARIO-TAPERA-CRED

0 stars 0 forks source link

implementação com banco de dados, cadastro do cliente e procar client… #4

Open caroliny-gomes-lima opened 3 weeks ago

caroliny-gomes-lima commented 3 weeks ago

Observação: Para visualizar todas as minhas implementações completas, sugiro acessar o repositório do projeto. As mudanças nesta pull request acabaram apenas pegando comentários relacionados às minhas implementações (comentários dos scripts).

caroliny-gomes-lima commented 3 weeks ago

Gostaria de informar que esta pull request não tem tantas implementações quanto o necessario para o desafio da semana 8. Devido à falta de tempo, não consegui concluir todas as tarefas, pois tive uma carga de trabalho extra no meu emprego como freelancer. Gostaria de ter feito mais, mas infelizmente não foi possível. Agradeço pela compreensão e me comprometo a entregar um projeto mais completo no final, pois esse é o meu objetivo. 🙏 🙏

vanessaribeiro03 commented 3 weeks ago

Oi Caroliny, espero que esteja bem! Desde já, foi muito enriquecedor fazer o review do seu codigo! Vamos a minha breve avaliação...

Pontos positivos:

Gostei muito da forma como você estruturou o código e das definições que forneceu para determinados assuntos. Isso me ajudou a entender melhor não só o código em si, mas também o seu raciocínio ao implementá-lo.

O arquivo de validação está muito bom! Separar as validações em outro arquivo deixou o código muito mais limpo e organizado. A gente sabe bem que validações podem, muitas vezes, ocupar bastante espaço, e a maneira como você gerenciou isso foi perfeita, me inspirou a trazer isso pro meu codigo!

Conclusão

De maneira geral, considerando o tempo limitado que você teve, incluindo questões pessoais para resolver, achei sua proposta muito bem executada. Você se preocupou em aplicar vários conceitos que aprendemos, e o resultado final ficou ótimo. Parabéns por isso, você vai longe!