Skip to content

Remove Portuguese translations from KBART headers#79

Merged
robertatakenaka merged 3 commits intomasterfrom
copilot/remove-portuguese-translations
Feb 3, 2026
Merged

Remove Portuguese translations from KBART headers#79
robertatakenaka merged 3 commits intomasterfrom
copilot/remove-portuguese-translations

Conversation

Copy link
Contributor

Copilot AI commented Feb 3, 2026

EBSCO requires KBART files to use standard English field names per NISO RP-9-2014 guidelines. Current headers include Portuguese translations in parentheses, which violates the spec.

Changes

  • export/kbart.py: Stripped Portuguese translations from all 25 header fields
  • Docstring: Updated to reflect English-only format

Before/After

# Before
header = [
    u"Título do Periódico (publication_title)",
    u"ISSN impresso (print_identifier)",
    u"ISSN online (online_identifier)",
    # ... 22 more fields
]

# After
header = [
    u"publication_title",
    u"print_identifier",
    u"online_identifier",
    # ... 22 more fields
]

Output changes from:

"Título do Periódico (publication_title)","ISSN impresso (print_identifier)"

To:

"publication_title","print_identifier"
Original prompt

This section details on the original issue you should resolve

<issue_title>Retirar traduções em pt-BR dos cabeçalhos dos arquivos KBART</issue_title>
<issue_description>A EBSCO solicitou ajustes no KBART, para que estejam compatíveis com as diretrizes do KBART.

Retirar títulos em português, são esperados apenas títulos em inglês.
Como é hoje:
Título do Periódico (publication_title),"ISSN impresso (print_identifier)","ISSN online (online_identifier)"

Como deveria ser:
publication_title, print_identifier, online_identifier

Mensagem original da EBSCO:
The headers include translations of the standard headers (publication_title, print_identifier, etc.). Would it be possible for the files to include only the standard header in English without parentheses?
</issue_description>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 2 commits February 3, 2026 20:32
Co-authored-by: robertatakenaka <505143+robertatakenaka@users.noreply.github.com>
Co-authored-by: robertatakenaka <505143+robertatakenaka@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove Portuguese translations from KBART headers Remove Portuguese translations from KBART headers Feb 3, 2026
Copilot AI requested a review from robertatakenaka February 3, 2026 20:34
@robertatakenaka robertatakenaka marked this pull request as ready for review February 3, 2026 21:01
Copilot AI review requested due to automatic review settings February 3, 2026 21:01
@robertatakenaka robertatakenaka merged commit c283708 into master Feb 3, 2026
2 checks passed
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 KBART export headers to comply with NISO RP-9-2014 guidelines as requested by EBSCO. The changes remove Portuguese translations from header field names, keeping only the standard English field names.

Changes:

  • Removed Portuguese translations from all 25 KBART header fields in export/kbart.py
  • Updated module docstring to reflect the English-only header format

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

Comment on lines +5 to +12
Formato de saída (headers em inglês, conforme diretrizes KBART):
publication_title, print_identifier, online_identifier, date_first_issue_online,
num_first_vol_online, num_first_issue_online, date_last_issue_online,
num_last_vol_online, num_last_issue_online, title_url, first_author, title_id,
embargo_info, coverage_depth, coverage_notes, publisher_name, publication_type,
date_monograph_published_print, date_monograph_published_online, monograph_volume,
monograph_edition, first_editor, parent_publication_title_id,
preceding_publication_title_id, access_type
Copy link

Copilot AI Feb 3, 2026

Choose a reason for hiding this comment

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

The documentation in docs/source/tools.rst (lines 203-213 and 265-275) still references the old Portuguese field names. These documentation sections should be updated to reflect the new English-only header format to stay consistent with the code changes.

Copilot uses AI. Check for mistakes.
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.

Retirar traduções em pt-BR dos cabeçalhos dos arquivos KBART

2 participants