kinvoapp / kinvo-front-end-test

Teste para candidatos à vaga de Front-End
MIT License
99 stars 91 forks source link

Harlan Cleiton #35

Closed harlancleiton closed 3 years ago

harlancleiton commented 3 years ago

Requisitos realizados

Trainee/Estágio

Junior

Pleno

Sênior/Analista

lucasljsb commented 3 years ago

Oi, @harlancleiton . Vi que marcou a implementação dos gráficos como realizada, porém os gráficos de divisão de carteira não foram implementados. Esqueceu algum commit, teve alguma dificuldade específica ou acabou ficando sem tempo?

harlancleiton commented 3 years ago

Oi @lucasljsb, tudo bem? Primeiramente peço desculpas pelo equivoco, minha intenção era marcar a implementação dos gráficos de forma parcial. O motivo de não ter implementado os gráficos de divisão de carteira foi porque fiquei em dúvida de qual campo seria usado para calcular o gráfico por tipo, acredito que movementTypeId, mas não quis arriscar e acabar fazendo errado. Quanto ao gráfico por título acabei não fazendo porque ao lado ficaria um espaço vazio. Devido a algumas demandas do meu trabalho atual não consegui tempo para tirar essas dúvidas contigo e ainda fazer as implementações. Peço desculpas mais uma vez. De qualquer forma, se estiver certo e o gráfico por tipo for calculado pelo atributo movementTypeId, estou à disposição para implementar esses gráficos e enviar até hoje a noite.

lucasljsb commented 3 years ago

@harlancleiton Sem problemas, Harlan! Só levantei a questão para fins de esclarecimentos mesmo, a avaliação segue normalmente.

Agradeço o rápido retorno.