-
Notifications
You must be signed in to change notification settings - Fork 512
Update pt-BR.json5 translation for DSpace 9.x compatibility #4390
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
Conversation
|
Hi @t-rodrigues, |
|
Hi @tdonohue , can I review this PR? I am from Brazil and I have just updated these missing translations in the custom theme of my organization's DSpace instance. |
|
@danielcoelhocgu : Yes, we welcome reviews from anyone and everyone. So, please do review this PR. |
|
|
||
| // "item.edit.bitstreams.edit.live.select": "{{ bitstream }} is selected.", | ||
| // TODO New key - Add a translation | ||
| "item.edit.bitstreams.edit.live.select": "{{ bitstream }} is selected.", | ||
| "item.edit.bitstreams.edit.live.select": "{{ bitstream }} está selecionados.", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"está selecionado."
|
|
||
| // "submission.sections.describe.relationship-lookup.search-tab.tab-title.JournalIssue": "Local Journal Issues ({{ count }})", | ||
| "submission.sections.describe.relationship-lookup.search-tab.tab-title.JournalIssue": "Números de Revista Local ({{ count }})", | ||
| "submission.sections.describe.relationship-lookup.search-tab.tab-title.JournalIssue": "Números de Revistas Locais ({{ count }})", | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Em journalissue.page.titleprefix você traduziu "Journal Issue" como "Edição da Revista". Não seria melhor também usar o termo edição aqui?
| @@ -8898,22 +8712,22 @@ | |||
| "submitter.empty": "N/D", | |||
|
|
|||
| // "subscriptions.title": "Subscriptions", | |||
| "subscriptions.title": "Assinaturas", | |||
| "subscriptions.title": "Inscrições", | |||
|
|
|||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ao trocar o termo "assinatura" por "inscrição" é preciso revisar algumas outras chaves também. Exemplos: confirmation-modal.delete-subscription.header, confirmation-modal.delete-subscription.info, nav.subscriptions, subscriptions.modal.delete.success, entre outras.
|
|
||
| // "search.filters.queue_status.4,authority": "Failed", | ||
| // TODO New key - Add a translation | ||
| "search.filters.queue_status.4,authority": "Failed", | ||
| "search.filters.queue_status.4,authority": "Falha", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Não seria melhor "Falhou" ?
|
@danielcoelhocgu There is a more up-to-date pull request. I will review it, and if possible, could you review it as well? #4962 |
Update DSpace pt-BR translation to DSpace 9.x
List of changes in this PR:
Checklist
mainbranch of code (unless it is a backport or is fixing an issue specific to an older branch).npm run lintnpm run check-circ-deps)package.json), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.