Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Validador de formulário de inscrições #35

Closed
isaacmsl opened this issue Sep 5, 2020 · 7 comments · Fixed by #43
Closed

Validador de formulário de inscrições #35

isaacmsl opened this issue Sep 5, 2020 · 7 comments · Fixed by #43
Assignees
Labels
enhancement New feature or request front frontend waiting pr Esperando pull request ser aberta para closing

Comments

@isaacmsl
Copy link
Owner

isaacmsl commented Sep 5, 2020

É recomendado que a senha tenha pelo menos 8 dígitos.

@isaacmsl isaacmsl added bug Something isn't working front frontend labels Sep 5, 2020
@isaacmsl isaacmsl self-assigned this Sep 6, 2020
@isaacmsl
Copy link
Owner Author

isaacmsl commented Sep 6, 2020

Existe um attr em html "minlength". O interessante também seria colocar dois campos para a senha (por segurança)

(Nova Issue #40)

@isaacmsl isaacmsl changed the title Recomendar que o participante cadastre com no mínimo 8 dígitos a senha Aumentar segurança do cadastro de senhas Sep 6, 2020
@isaacmsl isaacmsl added enhancement New feature or request and removed bug Something isn't working labels Sep 6, 2020
@isaacmsl
Copy link
Owner Author

isaacmsl commented Sep 6, 2020

Adicionar também a opção de visualizar a senha no formulário com um toggle de type do input (text/password)

(Nova issue #41)

@isaacmsl
Copy link
Owner Author

isaacmsl commented Sep 7, 2020

Essa issue corrobou para a criação de um validador do formulário.

@isaacmsl isaacmsl changed the title Aumentar segurança do cadastro de senhas Validador de formulário de inscrições Sep 7, 2020
@isaacmsl
Copy link
Owner Author

isaacmsl commented Sep 8, 2020

InscricoesValidator criado. Atualmente ele só verifica os valores mais sensíveis e importantes:

  • Nome completo (deve ter mais que 1 caractere e pelo menos dois nomes: "Primeiro Segundo")
  • Email (deve ter mais que 1 caractere e passar no teste de regex para emails: "@.*")
  • Senha (deve ter pelo menos 8 caracteres)
  • Tipo de músico (1, 2 ou 3)
  • Tempo de atuação (1, 2, 3 ou 4)
  • Contato telefônico (deve passar pelo teste do regex para telefones: "(99) 999999999")
  • Oficinas (deve ser uma oficina válida: "Flauta, Clarinete etc")
  • Banda (deve ser uma banda válida: "Não sou integrante de banda etc")

Por quê não validar estado e cidade?

Futuramente pode ser. Mas por enquanto, percebo que não é muito importante para que um participante seja realmente válido. O único problema é a obtenção de informações do Festival (gráficos e análises).

O que acontece se o cliente mudar o value da cidade?

Ela é tratada como válida.

O que acontece se o cliente mudar o estado?

Como o select de cidades depende do valor do estado (api da IBGE), será lançada uma requisição com o valor de estado alterado. Se esse estado não existir, o select de cidades vai ficar vazio. -1 para o cliente.

@isaacmsl
Copy link
Owner Author

isaacmsl commented Sep 8, 2020

Com relação aos trims, foram adicionados para "nome completo" e "email". Mas também pode e deve ser aplicado em "endereço"

Adicionado.

@isaacmsl isaacmsl added the waiting pr Esperando pull request ser aberta para closing label Sep 8, 2020
@isaacmsl
Copy link
Owner Author

isaacmsl commented Sep 8, 2020

@PauloVLB percebeu que faltou a verificação da quantidade de caracteres em "Endereço"

@isaacmsl
Copy link
Owner Author

isaacmsl commented Sep 8, 2020

Adicionado validador para endereço (mais que um caractere e dois nomes: "Algo algo")

@isaacmsl isaacmsl linked a pull request Sep 8, 2020 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request front frontend waiting pr Esperando pull request ser aberta para closing
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant