Skip to content

Realiza ajustes em paramentros na task de coleta do pidprovider#1442

Closed
samuelveigarangel wants to merge 13 commits into
scieloorg:mainfrom
samuelveigarangel:fix-harvest-pidprovider
Closed

Realiza ajustes em paramentros na task de coleta do pidprovider#1442
samuelveigarangel wants to merge 13 commits into
scieloorg:mainfrom
samuelveigarangel:fix-harvest-pidprovider

Conversation

@samuelveigarangel

Copy link
Copy Markdown
Collaborator

O que esse PR faz?

Adiciona ajustes principalmente nos paramentros e na normalização de url para task de coleta do pidprovider.

Onde a revisão poderia começar?

pelos commits

Como este poderia ser testado manualmente?

Criar a task task_dispatch_articles
Args:
{
"username":"your_username",
"collection_acron_list":[
"scl"
],
"from_pub_year":2023
}

Comment thread article/tasks.py
try:
user = _get_user(self.request, username=username, user_id=user_id)

article_source = None

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@samuelveigarangel me parece que as mudanças impactou na lógica original. Por exemplo, o parâmetro pp_xml_id foi ignorado.

@robertatakenaka

Copy link
Copy Markdown
Member

@samuelveigarangel tá muito difícil de revisar porque há muito código que é só mudança de formatação. Faça o seguinte:

  • Crie um backup com todos os arquivos que você modificou neste PR.
  • Crie um novo branch a partir do main.
  • Aplique o black nos arquivos que você sabe que vai modificar
  • Faça o commit
  • Extraia os arquivos que fez o backup
  • Aplique o black nos arquivos que você extraiu do backup
  • Faça os commits das mudanças reais

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.

2 participants