Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix: Corrigir chamada para arquivos estáticos #592

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ramoncordini
Copy link

O que esse PR faz?

Atualiza a url para arquivos estáticos que foram migrados para o https://ds.scielo.org/static/

Onde a revisão poderia começar?

Após fazer a atualização nos arquivos estático, verificar alguns pontos ao navegar pela página de artigos:

  • o layout está ok?
  • os modais de imagens e tabelas estão abrindo?
  • no menu flutuante azul, ao clicar em algum dos itens, estes estão abrindo o modal?

Todos os itens devem estar funcionando corretamente.

Como este poderia ser testado manualmente?

Siga os itens descritos anteriormente.

Algum cenário de contexto que queira dar?

Ao realizar os testes para os novos estáticos foi percebida a ausência de algumas imagens no Design System.
Arquivos como list.loading.gif serão adicionados em um PR específico do design system. Outros arquivos como o dashline.png serão removidos pois estão obsoletos.

Screenshots

Screen Shot 2024-04-11 at 10 35 11
Screen Shot 2024-04-11 at 10 35 25
Screen Shot 2024-04-11 at 10 35 45

Quais são tickets relevantes?

Indique uma issue ao qual o pull request faz relacionamento.

Referências

@robertatakenaka
Copy link
Member

@ramoncordini acho que a correção não é no packtools, mas sim no opac_5.

Captura de Tela 2024-04-12 às 14 49 54

Acho que teria que corrigir nestes arquivos

> grep -rs article.css webapp/templates
webapp/templates/article/base.html:  <link rel="stylesheet" href="{{ url_for('static', filename='css/scielo-article.css') }}" type="text/css" async/>
webapp/templates/article/base.html:  <link rel="stylesheet" href="{{ url_for('static', filename='css/article.css') }}?v={{ config.VCS_REF }}" type="text/css" async/>

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