turicas / brasil.io

Backend do Brasil.IO (para código dos scripts de coleta de dados, veja o link na página de cada dataset)
https://brasil.io/
GNU General Public License v3.0
921 stars 145 forks source link

Fix/auth backend #521

Closed rhenanbartels closed 3 years ago

rhenanbartels commented 3 years ago

Este PR está relacionado a issu #511 e implementa:

  1. a validação do username no momento do Cadastro;
    • A partir de agora apenas letras, número e _ são permitidos. Caso contrário um erro de validação de Form é levantado.
  2. descrição dos erros do Form de login no template HTML.
  3. atualização do script de migração de usernames com '@'.
    • adiciona testes para as funções;
    • atualiza os registro no banco;
    • escreve um arquivo CSV no path '/data/fixed-usernames.csv' com os registros atualizados.

Para rodar o comando de atualização dos usernames:

python manage.py runscript migrate_wrong_usernames
rhenanbartels commented 3 years ago

Fala @marcmatias,

pode dar uma olhada nesse commit?

1f26787e3803a8b930481e2bccfd3da40c68ec

A ideia foi colocar um @ no início do input de username, tanto no login, quanto no signup.

image

marcmatias commented 3 years ago

Fala @marcmatias,

pode dar uma olhada nesse commit?

1f26787

A ideia foi colocar um @ no início do input de username, tanto no login, quanto no signup.

image

Opa, tranquilo. Um modo que exigiria menos código seria pondo o @ como bg no inputfield. subi aqui: Codepen exemplo resultado semelhante ao apresentado.