Skip to content

Corrige identidade de artigos e importa histórico parcial com controle - #152

Merged
pitangainnovare merged 7 commits into
scieloorg:mainfrom
pitangainnovare:fix/pid-v2-fluxo-scl
Sep 23, 2026
Merged

pitangainnovare merged 7 commits into
scieloorg:mainfrom
pitangainnovare:fix/pid-v2-fluxo-scl

Conversation

@pitangainnovare

@pitangainnovare pitangainnovare commented Sep 23, 2026 •

Copy link
Copy Markdown
Contributor

O que esse PR faz?

Atualiza scielo_usage_counter para 2.2.4 e confirma que URLs PID v2 e PID v3 do mesmo artigo compartilham a identidade de contagem. Permite importar meses históricos com lacunas usando --allow-partial, sem marcar os dias ausentes como completos. Aceita relatórios parciais na recomposição da cobertura mensal. Atualiza o usage para 2.6.1.

Onde a revisão poderia começar?

metrics/legacy_matomo/validation.py e metrics/management/commands/import_legacy_matomo.py.

Como este poderia ser testado manualmente?

Rodar metrics.tests.integration.test_pipelines e metrics.tests.services.test_legacy_matomo_validation (22 testes passaram). Validar um pacote parcial sem e com --allow-partial; conferir backfill_counter_month_status --report para esse pacote.

Algum cenário de contexto que queira dar?

O histórico Matomo tem dias sem fatos de artigo. Os dias disponíveis podem ser importados, enquanto as lacunas permanecem identificadas e o mês incompleto. Depende de scielo_usage_counter#64 e da publicação da versão 2.2.4 para instalar a dependência no CI. No HML, usage_documents e usage_sources já estão povoados; os fatos antigos sem YOP têm chaves que não encontram esses metadados. Após reconstruir os fatos, conferir a correspondência das chaves, recompor a cobertura mensal e então executar backfill_counter_yop para o histórico Matomo.

Screenshots

Não se aplica.

Quais são os tickets relevantes?

Nenhum.

Referências

Correção do tradutor OPAC.


Segurança da informação (NSI.04)

Este PR manipula dados sensíveis ou pessoais (LGPD)?

  • Sim — descreva os controles de proteção aplicados (criptografia, mascaramento, anonimização, etc.):
  • Não

Este PR altera autenticação, autorização, controle de acesso ou gerenciamento de sessão?

  • Sim — descreva o que mudou e por quê:
  • Não

Este PR introduz, atualiza ou remove dependências de terceiros?

  • Sim — as novas dependências foram verificadas no SBOM/Trivy sem vulnerabilidades críticas/altas em aberto?
    • Verificado e aprovado
    • Pendente / vulnerabilidade aceita com justificativa:
  • Não — atualiza apenas a biblioteca SciELO do PR vinculado.

Este PR foi validado pelo pipeline de segurança (SonarQube / Trivy)?

  • Sim — link do job:
  • Não aplicável a este PR (justifique):

Pendente da execução do pipeline do PR.

Este PR concatena, monta ou executa comandos SQL, HTML ou JavaScript a partir de entrada externa?

  • Sim — confirme que há sanitização/parametrização (prepared statements, escaping, etc.):
  • Não

Este PR expõe novos endpoints, telas ou serviços?

  • Sim — HTTPS obrigatório está garantido e o acesso segue o princípio de menor privilégio?
  • Não

Algum segredo, senha, chave ou token está sendo adicionado ao código-fonte?

  • Não, nenhum segredo foi commitado
  • Sim (bloquear merge e corrigir antes de prosseguir)

@pitangainnovare
pitangainnovare merged commit 690add2 into scieloorg:main Sep 23, 2026
2 checks passed
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