Skip to content

Atualiza os href dos links do menu “Lista de periódicos” do SciELO.org.#595

Open
samuelveigarangel wants to merge 1 commit intoscieloorg:masterfrom
samuelveigarangel:issue-594
Open

Atualiza os href dos links do menu “Lista de periódicos” do SciELO.org.#595
samuelveigarangel wants to merge 1 commit intoscieloorg:masterfrom
samuelveigarangel:issue-594

Conversation

@samuelveigarangel
Copy link
Collaborator

O que esse PR faz?

  • Atualiza os href dos links do menu “Lista de periódicos” do SciELO.org.

Onde a revisão poderia começar?

Indique o caminho do arquivo e o arquivo onde o revisor deve iniciar a leitura do código.

Como este poderia ser testado manualmente?

docker-compose  -f docker-compose-dev.yml up

acessar: localhost

Algum cenário de contexto que queira dar?

N/A

Screenshots

pt:
image
en:
image

es:
image

Quais são tickets relevantes?

CLOSE #594

Referências

N/A

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the menu links for "Lista de periódicos" (Journal List) in SciELO.org, migrating from legacy PHP-based URLs to modern REST-style URLs across three language versions.

  • Updates journal list links from old applications/scielo-org/php/secondLevel.php endpoints to new cleaner URL patterns
  • Upgrades links from HTTP to HTTPS protocol for improved security
  • Maintains consistent language-specific paths: /pt/periodicos/, /es/revistas/, /en/journals/

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
iahx/templates/custom/top-menu-pt.html Updates Portuguese menu links to new REST-style URLs without trailing slashes
iahx/templates/custom/top-menu-es.html Updates Spanish menu links to new REST-style URLs with trailing slashes
iahx/templates/custom/top-menu-en.html Updates English menu links to new REST-style URLs with trailing slashes
iahx-sites/scieloorg/templates/custom/top-menu-pt.html Updates Portuguese menu links (duplicate location) to new REST-style URLs without trailing slashes
iahx-sites/scieloorg/templates/custom/top-menu-es.html Updates Spanish menu links (duplicate location) to new REST-style URLs with trailing slashes
iahx-sites/scieloorg/templates/custom/top-menu-en.html Updates English menu links (duplicate location) to new REST-style URLs with trailing slashes

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +9 to +10
<li><a href="https://www.scielo.org/pt/periodicos/listar-por-ordem-alfabetica">Lista alfabética de periódicos</a></li>
<li><a href="https://www.scielo.org/pt/periodicos/listar-por-assunto">Lista de periódicos por assunto</a></li>
Copy link

Copilot AI Nov 11, 2025

Choose a reason for hiding this comment

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

Inconsistent trailing slash usage: Portuguese URLs don't have trailing slashes while Spanish and English versions do. For consistency and to avoid potential redirect issues, all URLs should follow the same pattern. Consider either adding trailing slashes to Portuguese URLs or removing them from Spanish and English URLs.

Suggested change
<li><a href="https://www.scielo.org/pt/periodicos/listar-por-ordem-alfabetica">Lista alfabética de periódicos</a></li>
<li><a href="https://www.scielo.org/pt/periodicos/listar-por-assunto">Lista de periódicos por assunto</a></li>
<li><a href="https://www.scielo.org/pt/periodicos/listar-por-ordem-alfabetica/">Lista alfabética de periódicos</a></li>
<li><a href="https://www.scielo.org/pt/periodicos/listar-por-assunto/">Lista de periódicos por assunto</a></li>

Copilot uses AI. Check for mistakes.
Comment on lines +9 to +10
<li><a href="https://www.scielo.org/pt/periodicos/listar-por-ordem-alfabetica">Lista alfabética de periódicos</a></li>
<li><a href="https://www.scielo.org/pt/periodicos/listar-por-assunto">Lista de periódicos por assunto</a></li>
Copy link

Copilot AI Nov 11, 2025

Choose a reason for hiding this comment

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

Inconsistent trailing slash usage: Portuguese URLs don't have trailing slashes while Spanish and English versions do. For consistency and to avoid potential redirect issues, all URLs should follow the same pattern. Consider either adding trailing slashes to Portuguese URLs or removing them from Spanish and English URLs.

Suggested change
<li><a href="https://www.scielo.org/pt/periodicos/listar-por-ordem-alfabetica">Lista alfabética de periódicos</a></li>
<li><a href="https://www.scielo.org/pt/periodicos/listar-por-assunto">Lista de periódicos por assunto</a></li>
<li><a href="https://www.scielo.org/pt/periodicos/listar-por-ordem-alfabetica/">Lista alfabética de periódicos</a></li>
<li><a href="https://www.scielo.org/pt/periodicos/listar-por-assunto/">Lista de periódicos por assunto</a></li>

Copilot uses AI. Check for mistakes.
@gitnnolabs gitnnolabs self-assigned this Nov 12, 2025
@gitnnolabs
Copy link
Collaborator

@samuelveigarangel revisando...

@gitnnolabs
Copy link
Collaborator

Teste locais funcionando corretamente.

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.

Links quebrados do menu principal

3 participants