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

Habilita acesso à página informativa com base na coleção configurada na interface administrativa #195

Merged
merged 2 commits into from
Nov 4, 2024

Conversation

samuelveigarangel
Copy link
Contributor

O que esse PR faz?

Habilita acesso à página informativa com base na coleção configurada na interface administrativa

Onde a revisão poderia começar?

pelos commits

Como este poderia ser testado manualmente?

Acessar página informativa

Algum cenário de contexto que queira dar?

N/A

Screenshots

N/A

Quais são tickets relevantes?

#192

Referências

N/A

@gitnnolabs
Copy link
Collaborator

@samuelveigarangel validando....

@gitnnolabs
Copy link
Collaborator

@samuelveigarangel @robertatakenaka

Validando essa atividade reparei que as páginas sobre o periódicos ficam com diversos seções com valores vazios, seria esse mesmo comportamento esperado?

Vejam:

Screenshot 2024-10-31 at 13 33 42

@gitnnolabs
Copy link
Collaborator

gitnnolabs commented Oct 31, 2024

Outro ponto interessante é que ao não temos conectividade com o https://core.scielo.org para obter a página apresentamos a seguinte página para usuário, veja:

Screenshot 2024-10-31 at 13 37 50

Esse é um ponto de falha que podemos demorar para perceber, sugiro que seja apresentado uma página de erro mais amigável e que de alguma forma sejamos avisados.

Isso pode ser uma melhoria!

@gitnnolabs
Copy link
Collaborator

gitnnolabs commented Oct 31, 2024

Reparei que ao trocar o idioma do site os labels não são alterado para o idioma apresentado:

Screenshot 2024-10-31 at 13 49 40

Repare que o idioma selecionado é o "Inglês" e os termos das seções estão em português. O mesmo acontece para o idioma em "Espanhol".

Copy link
Collaborator

@gitnnolabs gitnnolabs left a comment

Choose a reason for hiding this comment

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

@samuelveigarangel verifica se é possível realizar os ajustes mencionados.

@samuelveigarangel
Copy link
Contributor Author

@gitnnolabs Esse comportamento dos campos vazios é esperado pois não há valor preenchido no formulário no core.

@samuelveigarangel
Copy link
Contributor Author

@gitnnolabs Não está traduzido pois não há os campos traduzidos nos arquivos de traduções no core.

@gitnnolabs gitnnolabs merged commit 08fc1d3 into scieloorg:master Nov 4, 2024
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.

2 participants