search

falko-org / Falko

:chart_with_upwards_trend: Falko (Front-End): Platform for agile projects management :bar_chart:
MIT License
15 stars 11 forks source link

Validações FrontEnd #227

Closed MateusO97 closed 6 years ago

MateusO97 commented 6 years ago

Mudanças Propostas

Apresentar feedbacks visuais ao usuário ao realizar adições ou edições de algum componente. Informando a ele que os campos precisam ser preenchidos e quando necessário que as datas precisam ser adicionadas corretamente.

Tipo de Mudança

Que tipo de mudanças este Pull Request introduz ao Falko? Marque um x ao que se aplicar

Checklist

Marque um x ao que se aplicar. Se você não tiver certeza sobre algum dos tópicos, não hesite em perguntar. Estamos aqui para ajudar!

Screenshots

validations_frontend

MateusO97 commented 6 years ago

@MatheusRich Atualmente o botão de registrar fica desabilitado enquanto o usuário tiver algum campo inválido, logo ele não vai conseguir clicar no botão como o teste de "Authorization tests should not register invalid user" indica. Devemos tirar esse teste já que não é mais possivel clicar no botão caso tenha algum campo invalido ?

Matheus456 commented 6 years ago

@MateusO97. Não sei se é necessário validar, mas atualmente quando o usuário digita a data ao invés de clicar o sistema permite que o usuário escolha uma data inválida.

alaxalves commented 6 years ago

@MateusO97 Porque a validação mostra o atributo que é necessário preencher, não tem como ajeitar isso? Estou falando disso aqui: image

alaxalves commented 6 years ago

Existem ainda alguns conflitos para serem arrumados :D

alaxalves commented 6 years ago

Não consegui contornar esse teste que quebra, acho que por enquanto pode comentá-lo, a não ser q o @MatheusRich ja tenha pensado em algo

alaxalves commented 6 years ago

image

MateusO97 commented 6 years ago

validationsnovas @alaxalves Consertei os nomes para deixar de maneira mais agradável visualmente, realmente daquela forma ficava estranho. Arrumei também os conflitos e comentei o teste, porém o travis vai rodar os testes que estão na devel e por isso ainda não esta passando, né? Não consegui pensar em nada para substitui-lo no momento, @MatheusRich se tiver alguma ideia para isso tento arrumar o mais breve possivel :D

MatheusRich commented 6 years ago

@MateusO97 O Travis vai rodar os testes da sua branch. Se vc modificou algo no funcionamento, modifique o teste para a nova condição.

alaxalves commented 6 years ago

EAEE @MateusO97 Arruma aí, os conflitos tão só aumentando

MateusO97 commented 6 years ago

@matheussbernardo vou precisar de uma ajuda para resolver esses testes que não passaram, os conflitos posso resolver mas os testes fiquei com bastante dificuldade :/