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

Fix/auth backend #521

Merged
merged 18 commits into from
Apr 9, 2021
Merged

Fix/auth backend #521

merged 18 commits into from
Apr 9, 2021

Conversation

rhenanbartels
Copy link
Collaborator

@rhenanbartels rhenanbartels commented Mar 30, 2021

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
Copy link
Collaborator Author

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

Copy link
Owner

@turicas turicas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Muito bom! Coloquei alguns comentários de alteração, mas são relativamente simples, apenas com intuito de melhorar a legibilidade e tempo de execução.

brasilio_auth/forms.py Outdated Show resolved Hide resolved
brasilio_auth/forms.py Outdated Show resolved Hide resolved
brasilio_auth/templates/brasilio_auth/login.html Outdated Show resolved Hide resolved
brasilio_auth/scripts/migrate_wrong_usernames.py Outdated Show resolved Hide resolved
brasilio_auth/tests/test_scripts.py Show resolved Hide resolved
@marcmatias
Copy link
Collaborator

marcmatias commented Mar 31, 2021

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.

@rhenanbartels rhenanbartels force-pushed the fix/auth-backend branch 2 times, most recently from 70541ce to 00e5a02 Compare April 2, 2021 21:55
@turicas turicas merged commit 9a7ef9e into turicas:develop Apr 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants