search

db1group / contra-covid19

Notifica Saúde - Uma ferramenta para centralizar notificações de casos de COVID-19
Other
14 stars 8 forks source link

Campo CPF considerando caracteres especiais. #188

Closed nasouza2 closed 4 years ago

nasouza2 commented 4 years ago

Ambiente https://covid19-db1-frontend.herokuapp.com/

Reprodução Dado que eu esteja na tela de Cadastro de notificação Quando eu informar o CPF com os caracteres especiais Então é apresentada a mensagem: "O campo deve possuir 11 caracteres.".

Problema Campo Número do documento considerando os carateres especiais informados, quando o tipo de documento é CPF. Se tirar os caracteres, o campo é validado corretamente.

Resultado esperado Desconsiderar os caracteres especiais informados.

img1 img2

wildeee commented 4 years ago

PR disponível pra merge. Foi uma alteração relativamente grande, então peço que teste bem a parte de tipo de documento, número do documento, classificação da pessoa (indígena, outros, etc.), obrigatoriedade dos campos, e modo de visualização.

CristianeBorges05 commented 4 years ago

@wildeee @nasouza2 Nati se vc quiser dar uma validada também. Pois acredito que a solução da máscara ficou ok, mas achei estranho o comportamento ao alterar o tipo de doc, onde não limpa o preenchimento do campo, para opções diferentes de CPF. Acho interessante mais um parecer rs CPF

wildeee commented 4 years ago

Como preferirem, mudar esse comportamento é fácil.

nasouza2 commented 4 years ago

@CristianeBorges05 e @wildeee Oi pessoal, concordo com a Cris. Quando alterar o tipo de documento, então limpa o campo "Número do documento", caso esse já foi informado. Isso vai evitar informar número de documento com a máscara incorreta.

@wildeee dá para arrumar nesse card ou prefere que abra outro?

wildeee commented 4 years ago

@nasouza2 Posso ajustar nesse mesmo, sem problema.

nasouza2 commented 4 years ago

Ok, validado.