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

Adiciona a caixa de artigos relacionados no topo da página do artigo. #2056

Merged
merged 7 commits into from
Oct 14, 2021

Conversation

gitnnolabs
Copy link
Collaborator

O que esse PR faz?

Quando estiver presente na entidade de artigo o campo related_articles será exibido os DOIs que relacionam o artigo acessado.

Onde a revisão poderia começar?

Por commit.

Como este poderia ser testado manualmente?

Rodando a aplicação e contendo na entidade de artigo o campo related_articles, veja:

Captura de Tela 2021-09-21 às 08 45 48

Algum cenário de contexto que queira dar?

Para que efetivamente vejamos isso em produção é necessário uma alteração que está sendo realizado no opac-airflow para garantir a existência do campo related_articles na entidade de artigo.

Screenshots

Captura de Tela 2021-09-21 às 08 38 26

Captura de Tela 2021-09-21 às 08 38 14

Captura de Tela 2021-09-21 às 08 28 24

Captura de Tela 2021-09-21 às 08 51 09

Quais são tickets relevantes?

#2052

Referências

N/A

@gitnnolabs gitnnolabs changed the title Adiciona a caixa de artigos relacionados no top da página do artigo. Adiciona a caixa de artigos relacionados no topo da página do artigo. Sep 21, 2021

#: opac/webapp/templates/article/includes/related-article.html:11
msgid "Este artigo contém retratação"
msgstr ""
Copy link
Member

Choose a reason for hiding this comment

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

@gitnnolabs solicitar as traduções dos textos se já não foi informado

Copy link
Collaborator Author

@gitnnolabs gitnnolabs Sep 22, 2021

Choose a reason for hiding this comment

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

@robertatakenaka o arquivo .pot é um template de tradução, contém somente o texto encontrado nos templates (.html) pela tag {% trans %}, as traduções vão no arquivos .po e realizei todas as traduções, veja no PR os arquivos .po

Copy link
Member

@robertatakenaka robertatakenaka left a comment

Choose a reason for hiding this comment

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

No documento vinculado, haverá também uma caixa?

@gitnnolabs
Copy link
Collaborator Author

@robertatakenaka vou colocar no contexto da errata, mas isso se estende para os demais documentos ['retratação' e 'adendo']

Sim, o que conversei com a @RPostalli é que deve conter uma caixa na errata para os documentos corrigidos(já existe via packtools) e uma caixa no documento para a errata.

@robertatakenaka
Copy link
Member

@gitnnolabs se esta caixa fica na errata, então o texto está incorreto. O texto que fica na errata deve ser algo como este documento corrige e não este documento tem correções, pois quem tem correções é o documento que está incorreto e não a errata.

@gitnnolabs
Copy link
Collaborator Author

gitnnolabs commented Sep 22, 2021

@robertatakenaka esse texto aparece no documento que esta sendo corrigido e não na errata.

Atualmente quem coloca a caixa da errata é o packtools, como não conversamos, pensei que o related_articles seria somente do documento -> para a errata.

Porém, com o esclarecimento que o related-articles é de ida e volta é necessário avaliar o related_type para alterar essa msg.

@gitnnolabs
Copy link
Collaborator Author

Vou remodelar essa implementação conforme as alterações no PR scieloorg/opac-airflow#302

@gitnnolabs
Copy link
Collaborator Author

gitnnolabs commented Oct 5, 2021

@robertatakenaka poderia validar novamente, segue imgs de como ficou:

Esse é o artigo:

Captura de Tela 2021-10-05 às 12 09 58

Essa seria a errada do artigo, porém, ignora o fato de esse artigo não ser realmente uma errata.
Captura de Tela 2021-10-05 às 12 10 03

Essas imagens são basicamente para valiar a interface!

@robertatakenaka
Copy link
Member

@gitnnolabs não ficou claro para mim estas duas imagens. O que é cada uma delas. Fig1 é .. e a Fig 2 é ?

@gitnnolabs
Copy link
Collaborator Author

@robertatakenaka

Sobre esse comentário #2056 (comment)

Veja se melhorou as alterações que realizei na descrição desse comentário!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants