Skip to content

Aplica correções sugeridas pela revisão realizada no Pull Request #1 - #12

Merged
pitangainnovare merged 24 commits into
scieloorg:mainfrom
pitangainnovare:fix/pr-1
May 10, 2024
Merged

Aplica correções sugeridas pela revisão realizada no Pull Request #1#12
pitangainnovare merged 24 commits into
scieloorg:mainfrom
pitangainnovare:fix/pr-1

Conversation

@pitangainnovare

Copy link
Copy Markdown
Contributor

O que esse PR faz?

Aplica todas as correções solicitadas no Pull Request #1.

Por onde a revisão poderia começar?

Pelo commit 2643f46.

Como este poderia ser testado manualmente?

Exatamente como feito no Pull Request #1.

Algum cenário de contexto que queira dar?

  1. A biblioteca wagtailautocomplete impede a atualização do wagtail para 6. Vide git da referida biblioteca.
  2. A biblioteca django-haystack só possui suporte para Django 5 na versão experimental. Vide Django Haystack (PyPI).

Screenshots

N/A

Quais são tickets relevantes?

N/A

Referências

N/A

Comment threadlog_manager/tasks.py

if not (temporal_reference or from_date) or file_ctime > obj_from_date:
task_create_log_file(col, file_path, user_id, username)
task_create_log_file.apply_async(args=(collection_acron2, file_path, user_id, username))

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Excluir a linha 43

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

Feito.

@pitangainnovare
pitangainnovare merged commit 31c9607 into scieloorg:mainMay 10, 2024
@pitangainnovare
pitangainnovare deleted the fix/pr-1 branch May 10, 2024 20:43
Sign up for freeto 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.

2 participants

@pitangainnovare@samuelveigarangel