search

ufabc-next / ufabc-next-web

📈 Plataforma web para auxiliar na graduação dos alunos da UFABC com informações sobre a performance, processo de matrículas e os discentes da universidade
https://ufabcnext.com
GNU Affero General Public License v3.0
55 stars 17 forks source link

fix(form): whitespace form validation #313

Closed mateusbrg closed 2 months ago

mateusbrg commented 3 months ago

Descrição

Corrigido validações faltantes nos campos de emails de formulários da aplicação

Tickets relacionados

312

mateusbrg commented 3 months ago

@Joabesv ainda resta uma validação de lowercase nesse ultimo commit (675fefa) que realizei. Na sua visão, onde ficaria melhor aplicá-lo?

Joabesv commented 3 months ago

@mateusbrg no proprio emailField, tem q tratar

Joabesv commented 2 months ago

Aqui, identifiquei dois caminhos:

  • o primeiro é o que foi realizado aqui, aplicar o .toLowerCase() na passagem de parâmetros para a mutation.
  • O segundo é adicionar um watch para sempre forçar o campo a ser lowercase durante a digitação do usuário, algo como (não dou certeza que seja assim):
watch(email, (newValue) => email.value = email.value.toLowerCase())

Qual prefere?

Muito melhor fazer o lowerCase quando passa para a mutation. Utilzar o watch iria prejudicar muito na visao do usuario