Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Feat (I18n - pt-BR): Started Brazilian Portuguese localization #3146

Closed
wants to merge 6 commits into from
Closed

Feat (I18n - pt-BR): Started Brazilian Portuguese localization #3146

wants to merge 6 commits into from

Conversation

ItaloPussi
Copy link
Contributor

Description

I've started translating the website into the Brazilian Portuguese language. As of now:

  • Configurated and localized the locale file pt-br.json
  • Localized content/about/*
  • Localized content/homepage/*

(Should disable the enabled flag before merging it)

Related Issues

No related issue

Check List

  • I have read the Contributing Guidelines and made commit messages that follow the guideline.
  • I have run npm run lint:js -- --fix and/or npm run lint:md -- --fix for my JavaScript and/or Markdown changes.
    • This is important as most of the cases your code changes might not be correctly linted
  • I have run npm run test to check if all tests are passing, and/or npm run test -- -u to update snapshots if I created and/or updated React Components.
  • I have checked that the build works locally and that npm run build work fine.
  • I've covered new added functionality with unit tests if necessary.

@AugustinMauroy
Copy link
Contributor

CC @nodejs/nodejs-pt

@AugustinMauroy AugustinMauroy added the create-preview Generate preview on staging.nodejs.dev label Jan 8, 2023
@github-actions github-actions bot removed the create-preview Generate preview on staging.nodejs.dev label Jan 8, 2023
@github-actions
Copy link

github-actions bot commented Jan 8, 2023

Please find a preview at: https://staging.nodejs.dev/3146/

Copy link
Member

@ovflowd ovflowd left a comment

Choose a reason for hiding this comment

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

This is good stuff. Going to give a better 👀 afterwards but thank you for the translation contribution!

content/homepage/index.pt-br.md Outdated Show resolved Hide resolved
src/i18n/locales/pt-br.json Show resolved Hide resolved
@ovflowd
Copy link
Member

ovflowd commented Jan 8, 2023

BTW, @ItaloPussi the unit tests are failing, could you double-check?

@codecov-commenter
Copy link

codecov-commenter commented Jan 8, 2023

Codecov Report

Base: 66.02% // Head: 66.39% // Increases project coverage by +0.36% 🎉

Coverage data is based on head (628b1cf) compared to base (fd57b87).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3146      +/-   ##
==========================================
+ Coverage   66.02%   66.39%   +0.36%     
==========================================
  Files         118      145      +27     
  Lines        1351     1702     +351     
  Branches      342      392      +50     
==========================================
+ Hits          892     1130     +238     
- Misses        422      528     +106     
- Partials       37       44       +7     
Impacted Files Coverage Δ
src/templates/blog.tsx 0.00% <0.00%> (-100.00%) ⬇️
util-node/getNodeReleasesData.js 91.30% <0.00%> (-6.32%) ⬇️
src/__fixtures__/page.tsx 94.11% <0.00%> (-5.89%) ⬇️
src/pages/index.tsx 100.00% <0.00%> (ø)
src/templates/api.tsx 0.00% <0.00%> (ø)
src/templates/learn.tsx 100.00% <0.00%> (ø)
util-node/createSlug.js 100.00% <0.00%> (ø)
util-node/getNvmData.js 100.00% <0.00%> (ø)
src/hooks/useOnScreen.tsx 100.00% <0.00%> (ø)
src/pages/about/index.tsx 100.00% <0.00%> (ø)
... and 185 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

Signed-off-by: Italo Pussi <italopussi@gmail.com>
@ItaloPussi
Copy link
Contributor Author

BTW, @ItaloPussi the unit tests are failing, could you double-check?

Augustin apparently got it fixed. Thank you!

Copy link

@TiagoDanin TiagoDanin left a comment

Choose a reason for hiding this comment

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

Thank you for your contributions
I left some suggestions

---
title: about
displayTitle: Sobre
description: 'About | Node.js'

Choose a reason for hiding this comment

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

We can also translate here

não precisam se preocupar com _deadlock_ de processos, pois não existem _locks_.
Quase nenhuma função no Node.js realiza diretamente operações de E/S, por essa
razão o processo nunca bloqueia exceto quando a operação de E/S é realizada
através de métodos sincronos nativos. Por não existirem operações bloqueantes,
Copy link

@TiagoDanin TiagoDanin Jan 10, 2023

Choose a reason for hiding this comment

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

Suggested change
através de métodos sincronos nativos. Por não existirem operações bloqueantes,
através de métodos síncronos nativos. Por não existirem operações bloqueantes,

tempo que permite que bibliotecas terceiras sejam atualizadas. Depois desse período, versões impares (9, 11, etc.) se tornam obsoletas e versões pares (10, 12, etc.)
entram em _LTS_ e estão prontas para uso geral. Versões _LTS_ possuem um tempo de suporte
de 30 meses cujo garante a correção de BUGs críticos nesse período. Aplicações
produtivas somente devem utilizar versões _LTS_.

Choose a reason for hiding this comment

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

What do you think about replacing "produtivas" (productive) to "de produção" (production)?

Por favor, revise a [Política de Marca Registrada](https://trademark-policy.openjsf.org/)
para informação sobre o uso aceitável de logos e da marca Node.js®

A lista de requirementos para utilização da marca Node.js estão disponíveis em [Visual Guidelines](/static/documents/foundation-visual-guidelines.pdf).

Choose a reason for hiding this comment

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

Suggested change
A lista de requirementos para utilização da marca Node.js estão disponíveis em [Visual Guidelines](/static/documents/foundation-visual-guidelines.pdf).
A lista de requerimentos para utilização da marca Node.js estão disponíveis em [Visual Guidelines](/static/documents/foundation-visual-guidelines.pdf).


## Recebendo atualizações de segurança

Atualizações de segurança serão distribuidas através dos seguintes métodos:

Choose a reason for hiding this comment

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

Suggested change
Atualizações de segurança serão distribuidas através dos seguintes métodos:
Atualizações de segurança serão distribuídas através dos seguintes métodos:

---
title: homepage
displayTitle: Rode JavaScript em qualquer lugar.
subTitle: 'Node.js é um runtime JavaScript gratuito, open-source e multiplaforma que permite que desenvolvedores escrevam CLIs e execute JavaScript no servidor sem a utilização de um browser.'

Choose a reason for hiding this comment

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

What do you think about replacing "browser" to "navegador"?

Suggested change
subTitle: 'Node.js é um runtime JavaScript gratuito, open-source e multiplaforma que permite que desenvolvedores escrevam CLIs e execute JavaScript no servidor sem a utilização de um browser.'
subTitle: 'Node.js é um runtime JavaScript gratuito, open-source e multiplaforma que permite que desenvolvedores escrevam CLIs e execute JavaScript no servidor sem a utilização de um navegador.'

@@ -0,0 +1,99 @@
{
"blog.title": "Blog Node.js",
"blog.description": "As últimas noticias, tutoriais, recursos e exemplos do NodeJS.",

Choose a reason for hiding this comment

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

Suggested change
"blog.description": "As últimas noticias, tutoriais, recursos e exemplos do NodeJS.",
"blog.description": "As últimas noticias, tutoriais, recursos e exemplos do Node.js.",

"blog.categories.release.description": "Anúncios sobre novas versões do Node.Js",
"blog.categories.vulnerability.description": "Anúncios envolvendo Vulnerabilidades de Segurança",
"components.downloadToggle.ltsVersions": "Versões LTS",
"components.downloadToggle.currentVersions": "Versões atuais"

Choose a reason for hiding this comment

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

let's keep the style pattern in the menu

Suggested change
"components.downloadToggle.currentVersions": "Versões atuais"
"components.downloadToggle.currentVersions": "Versões Atuais"

* Uma data de publicação é escolhida e uma _CVE (Common
Vulnerabilities and Exposures (CVE®))_ é requisitada para a vulnerabilidade.

* No dia de publicação, os colaboradores de segurança do Node.Js recebem uma cópia do
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
* No dia de publicação, os colaboradores de segurança do Node.Js recebem uma cópia do
* No dia de publicação, os colaboradores de segurança do Node.js recebem uma cópia do

@ovflowd
Copy link
Member

ovflowd commented Jan 30, 2023

@ItaloPussi any update on here?

@github-actions
Copy link

github-actions bot commented Mar 9, 2023

This pull request is stale because it has been open for 30 days with no activity.

@github-actions github-actions bot added the stale label Mar 9, 2023
@ovflowd ovflowd closed this Mar 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
6 participants