Skip to content

Corrige execução manual de tasks e defaults de resources#122

Merged
pitangainnovare merged 4 commits intoscieloorg:mainfrom
pitangainnovare:issue-121-task-settings-cleanup
May 5, 2026
Merged

Corrige execução manual de tasks e defaults de resources#122
pitangainnovare merged 4 commits intoscieloorg:mainfrom
pitangainnovare:issue-121-task-settings-cleanup

Conversation

@pitangainnovare
Copy link
Copy Markdown
Contributor

O que esse PR faz?

Corrige a execução manual de tasks no admin para não adicionar user_id em tasks que não aceitam esse parâmetro, mantém nomes de tasks Celery como literais estáveis e move defaults de resources para settings Django. Também atualiza o carregamento de GeoIP para tentar o MMDB do mês atual e, se indisponível, o mês anterior.

Por onde a revisão poderia começar?

django_celery_beat/views.py e depois resources/tasks.py.

Como este poderia ser testado manualmente?

  1. Iniciar o ambiente local com make up.
  2. Acessar o Wagtail admin em http://localhost:8009/admin.
  3. Executar manualmente tasks que aceitam user_id, como rotinas de carga de dados.
  4. Executar manualmente tasks que não aceitam user_id, como [Resources] Load Geolocation Data.
  5. Confirmar que a execução não falha por argumento inesperado e que o GeoIP usa candidatos baseados em MMDB_URL_TEMPLATE.

Algum cenário de contexto que queira dar?

O problema aparece quando uma task registrada no Celery é acionada pelo admin e a view adiciona user_id mesmo quando a assinatura da task não possui esse parâmetro. Em resources, os defaults estavam em um módulo local de constants; agora ficam em settings para permitir configuração por ambiente.

Screenshots

N/A

Quais são tickets relevantes?

#121

Referências

N/A

@pitangainnovare pitangainnovare self-assigned this May 5, 2026
@pitangainnovare pitangainnovare added the bug Something isn't working label May 5, 2026
@pitangainnovare pitangainnovare marked this pull request as ready for review May 5, 2026 13:33
@pitangainnovare pitangainnovare merged commit 7e6e30e into scieloorg:main May 5, 2026
2 checks passed
@pitangainnovare pitangainnovare deleted the issue-121-task-settings-cleanup branch May 5, 2026 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Erro 500 ao clicar em run nas tasks periódicas para carregar robôs e mapas

1 participant