Open
Conversation
labenu-bot
approved these changes
Jun 13, 2022
labenu-bot
left a comment
There was a problem hiding this comment.
Parabéns pela entrega!
Requisitos do projeto ✅
| Implementações | Feito |
|---|---|
| É possível criar uma conta no banco se tiver idade igual ou maior do que 18 anos | ✅ |
| Criou um método GET na entidade users para pegar todos os usuários existentes no array de usuários. | - |
| É possível pagar uma conta passando: um valor, uma descrição e uma data de pagamento | - |
| Não é possível agendar um pagamento para um dia anterior | - |
| Não é possível pagar uma conta com o valor maior do que saldo. | - |
| É possível fazer uma transferência bancária somente se for informado o nome, CPF, nome do destinatário, o CPF do destinatário e o valor | - |
| Se a data da transferência bancária não for informada, deve-se considerar que o pagamento é para ser feito no mesmo dia | - |
| Se a data de um pagamento não for informada, a data do pagamento é direcionada para o mesmo dia` | - |
| Não é possível criar usuários diferentes com o mesmo CPF [Desafio] | ✅ |
| Criou um endpoint get que recebe um CPF como parâmetro e retorna o saldo da conta do usuário. Emitiu um erro se o CPF informado não for do usuário [Desafio] | ✅ |
| Criou um endpoint put que recebe um nome, um CPF e um valor para adicionar um valor no saldo do usuário. Emitiu um erro se o CPF informado não for do usuário [Desafio] | - |
| Feedback do código | Feito |
|---|---|
| Criou um tipo para representar uma conta para o usuário | ✅ |
| Criou um array global que armazene usuários na aplicação | ✅ |
| Criou um tipo para representar as transações que serão salvas no extrato | ✅ |
| Criou um endpoint que utiliza o método POST da entidade users para cadastrar um usuário em um array de usuários. | ✅ |
| todos os gastos do usuário é guardado em um extrado, que no código é um array de objetos com as propriedades: valor, data e descrição | ✅ |
| Exibiu uma mensagem de erro se o usuário for menor de 18 anos. | ✅ |
| Foi criado um array para armazenar as transações de um cliente | ✅ |
| Boa indentação | ✅ |
| Qualidade de código | Médio |
Comentários da pessoa avaliadora
Alain,
Parabéns por sua entrega.
Pelo seu código consegui perceber que pegou bem a parte das APIs. Adorei o uso do regex.
Mesmo sem ter implementado todo o projeto, deu para perceber que pegou bem o conteúdo.
Seria interessante terminar o projeto para exercitar, mas sua lógica está muito boa.
Análise realizada por: Juliana
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
POR FAVOR, EDITE ESSA MENSAGEM INSERINDO AS INFORMAÇÕES DO SEU PROJETO. COMECE APAGANDO ESSA LINHA.
Entrega Inicial Projeto Sistema Bancario
O que funciona
O que não funciona
Link Surge
A PARTIR DA SEMANA 5, INSIRA AQUI O LINK DO SURGE. ANTES DISSO, APAGUE ESSAS DUAS LINHAS.
Imagens
TIRE PRINTS DAS TELAS DO SEU SITE E COLE AQUI