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.

Sign up for GitHub

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

Jump to bottom

[pt-br] feat: localized apis and sdks languages for pt-br #5393

Merged
merged 7 commits into from
Oct 24, 2024
Merged

[pt-br] feat: localized apis and sdks languages for pt-br #5393

merged 7 commits into from
Oct 24, 2024

Conversation

samyev
Copy link
Contributor

@samyev samyev commented Oct 11, 2024

Localized Language APIs & SDKs session for the Portuguese language.

close #5366

@samyev samyev requested a review from a team as a code owner October 11, 2024 22:27
@linux-foundation-easycla Linux Foundation: EasyCLA
Copy link

linux-foundation-easycla bot commented Oct 11, 2024 •
edited
Loading

CLA Signed

The committers listed above are authorized under a signed CLA.

@opentelemetrybot opentelemetrybot requested review from a team October 11, 2024 22:27
@samyev samyev force-pushed the feat/language-apis-and-sdks branch from 8eeb39c to 15abd31 Compare October 11, 2024 22:40
@emdneto emdneto requested a review from a team October 12, 2024 00:04
@emdneto
Copy link
Member

emdneto commented Oct 12, 2024

/fix:all

@opentelemetrybot
Copy link
Collaborator

You triggered fix:all action run at https://github.com/open-telemetry/opentelemetry.io/actions/runs/11300884719

@opentelemetrybot
Copy link
Collaborator

fix:all failed or was cancelled. For details, see https://github.com/open-telemetry/opentelemetry.io/actions/runs/11300884719.

emdneto
emdneto requested changes Oct 12, 2024
View reviewed changes
Copy link
Member

@emdneto emdneto left a comment

Choose a reason for hiding this comment

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

Obrigado pelo PR. Deixei alguns comentários. Acho que só corrigir os erros ali da CI ✌️

content/pt/docs/languages/_index.md Outdated Show resolved Hide resolved
content/pt/docs/languages/_index.md Outdated Show resolved Hide resolved
emdneto
emdneto reviewed Oct 12, 2024
View reviewed changes
content/pt/docs/languages/_index.md Outdated Show resolved Hide resolved
content/pt/docs/languages/_index.md Outdated Show resolved Hide resolved
content/pt/docs/languages/_index.md Outdated Show resolved Hide resolved
content/pt/docs/languages/_index.md Outdated Show resolved Hide resolved
content/pt/docs/languages/_index.md Outdated Show resolved Hide resolved
maryliag
maryliag reviewed Oct 12, 2024
View reviewed changes
Copy link
Contributor

@maryliag maryliag left a comment

Choose a reason for hiding this comment

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

Obrigada pela contribuição! Deixei alguns comentários

content/pt/docs/languages/_index.md Outdated Show resolved Hide resolved
content/pt/docs/languages/_index.md Outdated Show resolved Hide resolved
content/pt/docs/languages/_index.md Outdated Show resolved Hide resolved
content/pt/docs/languages/_index.md Outdated Show resolved Hide resolved
content/pt/docs/languages/_index.md Outdated Show resolved Hide resolved
content/pt/docs/languages/_index.md Outdated Show resolved Hide resolved
content/pt/docs/languages/_index.md Outdated Show resolved Hide resolved
content/pt/docs/languages/_index.md Outdated Show resolved Hide resolved
@opentelemetrybot opentelemetrybot requested a review from a team October 12, 2024 13:20
@samyev samyev requested review from maryliag and emdneto October 12, 2024 13:25
@samyev samyev force-pushed the feat/language-apis-and-sdks branch from fec829a to c8e16eb Compare October 13, 2024 12:36
@samyev
Copy link
Contributor Author

samyev commented Oct 13, 2024

@maryliag @emdneto Bom dia! Muito obrigada pelas correções, já realizei as alterações, e deixei comentários no glossário sobre a localização da palavra Registry ser alterada para Registro, e sobre a inserção da palavra Status para ser localizada para Estado pois foram dúvidas que tive e acabei me baseando por lá ^^

@samyev samyev force-pushed the feat/language-apis-and-sdks branch from c8e16eb to e1925b4 Compare October 15, 2024 23:12
maryliag
maryliag approved these changes Oct 15, 2024
View reviewed changes
Copy link
Contributor

@maryliag maryliag left a comment

Choose a reason for hiding this comment

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

Obrigada por atualizar o glossário também!

maryliag
maryliag requested changes Oct 15, 2024
View reviewed changes
Copy link
Contributor

@maryliag maryliag left a comment

Choose a reason for hiding this comment

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

Tem alguns tests falhando, você precisa rodar o linter pra arrumar estes erros

@opentelemetrybot opentelemetrybot requested a review from a team October 16, 2024 00:48
@samyev
Copy link
Contributor Author

samyev commented Oct 16, 2024

Tem alguns tests falhando, você precisa rodar o linter pra arrumar estes erros

Obrigada, ajustado! Você poderia revisar novamente?

@samyev samyev changed the title feat: localized apis and sdks languages for pt-br [pt-br] feat: localized apis and sdks languages for pt-br Oct 16, 2024
svrnm
svrnm requested changes Oct 16, 2024
View reviewed changes
content/pt/docs/languages/_index.md Outdated Show resolved Hide resolved
content/pt/docs/languages/_index.md Outdated Show resolved Hide resolved
@opentelemetrybot opentelemetrybot requested review from a team October 17, 2024 01:27
@samyev samyev force-pushed the feat/language-apis-and-sdks branch 2 times, most recently from 709cdf5 to 1f3770d Compare October 17, 2024 22:55
@emdneto emdneto requested a review from svrnm October 17, 2024 23:26
@samyev samyev force-pushed the feat/language-apis-and-sdks branch from 8040186 to 0f42a36 Compare October 23, 2024 01:01
@samyev
Copy link
Contributor Author

samyev commented Oct 23, 2024

Olá @maryliag @emdneto, tudo bem? O que ainda está faltando neste pr para ele ser aprovado? Vocês poderiam me ajudar? ><

@svrnm
Copy link
Member

svrnm commented Oct 23, 2024

@emdneto your review is still on "changes requested", can you take a look and unblock the PR? Happy to merge it

@svrnm
Copy link
Member

svrnm commented Oct 23, 2024

One more thing, the link checker is reporting the following:

  hash does not exist --- pt/docs/languages/index.html --> #status-and-releases

@samyev samyev force-pushed the feat/language-apis-and-sdks branch from 0f42a36 to 552125a Compare October 24, 2024 00:43
@opentelemetrybot opentelemetrybot requested review from a team October 24, 2024 00:43
@samyev
Copy link
Contributor Author

samyev commented Oct 24, 2024

One more thing, the link checker is reporting the following:

  hash does not exist --- pt/docs/languages/index.html --> #status-and-releases

Thanks! Could you approve the workflow execution to check if it's fixed?

@svrnm svrnm added this pull request to the merge queue Oct 24, 2024
@svrnm
Copy link
Member

svrnm commented Oct 24, 2024

geat work, thank you @samyev !

Merged via the queue into open-telemetry:main with commit 6a3fb5f Oct 24, 2024
17 checks passed
@chalin chalin mentioned this pull request Nov 5, 2024
Closed
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@emdneto emdneto emdneto approved these changes

@svrnm svrnm svrnm approved these changes

@maryliag maryliag maryliag approved these changes

Assignees
No one assigned
Labels
lang:pt
Projects
SIG Comms: Docs Translation
Status: Done
SIG Comms: PRs & Issues
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[pt] Localize content/en/docs/languages/_index.md
5 participants
@samyev @emdneto @opentelemetrybot @svrnm @maryliag