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.

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

[pt] Add translations for /pt/docs/concepts/glossary #5183

Merged
merged 31 commits into from
Oct 10, 2024

Conversation

vitorvasc
Copy link
Contributor

Add:

  • glossary.md

Issue: #4922

@opentelemetrybot opentelemetrybot requested review from a team September 11, 2024 11:47
@emdneto emdneto requested a review from a team September 11, 2024 12:46
@edsoncelio edsoncelio mentioned this pull request Sep 11, 2024
26 tasks
@opentelemetrybot opentelemetrybot requested a review from a team September 11, 2024 14:03
@emdneto
Copy link
Member

emdneto commented Sep 25, 2024

@vitorvasc gentle remind. ainda tá trabalhando nesta issue ou alguém pode assumir a partir daqui?

@opentelemetrybot opentelemetrybot requested review from a team September 27, 2024 10:41
@vitorvasc
Copy link
Contributor Author

@vitorvasc gentle remind. ainda tá trabalhando nesta issue ou alguém pode assumir a partir daqui?

opa! ainda estou. estive de férias nos últimos dias e acabei ficando offline, porém consigo retomar neste final de semana e seguir para uma próxima! valeu pelo comentário.

@opentelemetrybot opentelemetrybot requested review from a team and krol3 and removed request for a team October 2, 2024 16:50
@opentelemetrybot opentelemetrybot requested a review from a team October 3, 2024 11:45
@opentelemetrybot opentelemetrybot requested a review from a team October 9, 2024 22:37
@emdneto emdneto removed the lang:es label Oct 9, 2024
@svrnm
Copy link
Member

svrnm commented Oct 10, 2024

happy to merge this, but I see some open discussions, anything that still needs to be addressed?

@vitorvasc
Copy link
Contributor Author

happy to merge this, but I see some open discussions, anything that still needs to be addressed?

@svrnm actually, @EzzioMoreira's comment is about keeping the original wording for "Span link", since it's a feature from the API and SDK. Just an observation with no necessary action.

Did I understand that right, @EzzioMoreira?

@svrnm
Copy link
Member

svrnm commented Oct 10, 2024

@vitorvasc understood. based on the approvals, this is good to go, so I am happy to merge this PR then, we can have follow up changes as needed

@svrnm svrnm added this pull request to the merge queue Oct 10, 2024
Merged via the queue into open-telemetry:main with commit c5c1d02 Oct 10, 2024
17 checks passed
@vitorvasc vitorvasc deleted the ptbr_translate_glossary branch October 10, 2024 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

5 participants