scieloorg / Web

SciELO Web
www.scielo.br
6 stars 21 forks source link

Cria parte do utilitário para incluir DOI nos registros h e f de uma data bases #717

Closed robertatakenaka closed 4 years ago

robertatakenaka commented 4 years ago

O que esse PR faz?

Cria o módulo cisis e testes para gerar comandos para atualizar uma dada base com dados de DOI (veja a proposta completa (não aprovada) em #713)

Onde a revisão poderia começar?

por commits

Como este poderia ser testado manualmente?

Dentro proc/scielo_doi, execute:

python setup.py test

Algum cenário de contexto que queira dar?

n/a

Screenshots

n/a

Quais são tickets relevantes?

713

Referências

n/a

jamilatta commented 4 years ago

@robertatakenaka valiadndo esse PR.

jamilatta commented 4 years ago

@robertatakenaka

Olhando a descrição do tíquete e a implementação não encontrei no código o trecho que realizar o "desligamento" da criação de DOIs por idioma, se existir poderia me indicar?

Veja no descrição da atividade, https://github.com/scieloorg/Web/issues/713:

O utilitário deverá produzir um DOI para cada idioma, no caso de artigos com traduções. Este comportamento deverá poder ser desligado pelo usuário.

jamilatta commented 4 years ago

@robertatakenaka

Estou tentando realizar os testes porém estou recebendo o seguinte erro:

Captura de Tela 2020-05-05 às 13 46 31

Verifiquei que no arquivo de setup.py indica que o python deve ser em python >= 3.6.

Para resolver esse problema de rodar os testes verifiquei que é necessário criar o init.py dentro da pasta de tests, veja:

Captura de Tela 2020-05-05 às 13 50 54

Aí os testes rodaram, veja:

Captura de Tela 2020-05-05 às 13 51 32

robertatakenaka commented 4 years ago

"desligamento" da criação de DOIs por idioma, se existir poderia me indicar?

Por enquanto, isso é só parte do utilitário. Neste ponto só são algumas funções soltas. Não há nenhuma lógica, portanto não há neste momento o que ligar ou desligar.

robertatakenaka commented 4 years ago

@scieloorg/scielo-brazil-developers podem revisar novamente?