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

Corrige configuração de idioma #19

Merged
merged 6 commits into from
Jun 1, 2024
Merged

Corrige configuração de idioma #19

merged 6 commits into from
Jun 1, 2024

Conversation

pitangainnovare
Copy link
Contributor

O que esse PR faz?

Altera configuração pt-BR para pt-br em settings.py (no campo LANGUAGE_CODE e outros). Remove o prefixo do idioma padrão em URLS, que quando ativado, implica em falha de testes na app User. De modo temporário, desativei esse prefixo quando o idioma em uso for o padrão indicado em settings. Muitas testes verificam por URLS absolutas sem esse prefixo, todos baseados em resolve e reverse. Havia algumas possibilidades de alteração:

  1. Corrigir os testes para que incluam esse prefixo;
  2. Alterar o método get_absolute_url e outros para incluir esse prefixo;
  3. Remover o prefixo quando o idioma for o padrão.

Optou-se pela terceira solução.

Por onde a revisão poderia começar?

Por commits.

Como este poderia ser testado manualmente?

Execute docker compose -f local.yml run django pytest e veja que todos os testes passam.

Algum cenário de contexto que queira dar?

A solução é emergencial para possibilitara construção da imagem, que, até então falhada por conta de testes do app Users, originário do projeto SCMS_TEMPLATE.

Screenshots

N/A

Quais são tickets relevantes?

N/A

Referências

N/A

@pitangainnovare
Copy link
Contributor Author

Alguns defeitos foram identificados e corrigidos. O pytest passa sem erros.

O job linter foi removido, pois há defeitos pequenos demais legados de outros repositórios e arquivos estáticos. Por considerar a urgência do projeto, optei por desativar essa verificação, temporariamente.

Algo menos rigoroso deve ser configurado.

@pitangainnovare pitangainnovare merged commit 4fe34a4 into scieloorg:main Jun 1, 2024
2 checks passed
@pitangainnovare pitangainnovare deleted the fix-language-prefix-error-pytest branch June 1, 2024 19:09
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.

1 participant