Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

🌐 Add Portuguese translation fordocs/pt/docs/advanced/events.md#9326

Merged
tiangolo merged 9 commits intofastapi:masterfrom
oandersonmagalhaes:docs/pt_advanced-events
May 8, 2023
Merged

🌐 Add Portuguese translation fordocs/pt/docs/advanced/events.md#9326
tiangolo merged 9 commits intofastapi:masterfrom
oandersonmagalhaes:docs/pt_advanced-events

Conversation

@oandersonmagalhaes
Copy link
Contributor

No description provided.

@github-actions
Copy link
Contributor

📝 Docs preview for commitfcd966f at:https://6423813dd08506199aa8e03f--fastapi.netlify.app

@KludexKludex added lang-allTranslations lang-ptPortuguese translations awaiting-review labelsMar 30, 2023
@github-actions
Copy link
Contributor

📝 Docs preview for commit6968bfe at:https://642b3e1536db80082ae410bc--fastapi.netlify.app

Copy link
Contributor

@fcrozettafcrozetta left a comment

Choose a reason for hiding this comment

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

Suggestions:

  • One small grammar change (My spell check keeps pointing the accent should not be there 🤷🏻‍♂️)
  • the phrase in parenthesis was not translated to BR🇧🇷

oandersonmagalhaes reacted with thumbs up emoji
Co-authored-by: Fernando Crozetta <fernando@czetta.com>
@github-actions
Copy link
Contributor

📝 Docs preview for commit43aff4f at:https://6431d4d42e231e767ec8c459--fastapi.netlify.app

@github-actions
Copy link
Contributor

📝 Docs preview for commit5a8102d at:https://644667e10c8fa210b18c2ed5--fastapi.netlify.app

@oandersonmagalhaes
Copy link
ContributorAuthor

@LorhanSohaky and@fabioueno, could you take a look at this PR, please? 😅

@github-actions
Copy link
Contributor

📝 Docs preview for commit26e928c at:https://64487e6f116d705f2e58be4e--fastapi.netlify.app

@LorhanSohaky
Copy link
Contributor

@LorhanSohaky and@fabioueno, could you take a look at this PR, please? 😅

At the end of the day I will review

Copy link
Contributor

@LorhanSohakyLorhanSohaky left a comment

Choose a reason for hiding this comment

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

I fixed some typos, standardized some terms used throughout the text and
I think that with these suggestions the reading becomes more fluid

oandersonmagalhaes reacted with thumbs up emoji
Co-authored-by: Lorhan Sohaky <16273730+LorhanSohaky@users.noreply.github.com>
@github-actions
Copy link
Contributor

📝 Docs preview for commit5a5d05a at:https://644abbf094090e07d7d72287--fastapi.netlify.app

Copy link
Contributor

@LorhanSohakyLorhanSohaky left a comment

Choose a reason for hiding this comment

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

Nice!!!

oandersonmagalhaes reacted with laugh emoji
@github-actions
Copy link
Contributor

📝 Docs preview for commitc4c8480 at:https://6451712516dd18096c35e217--fastapi.netlify.app

@tiangolo
Copy link
Member

Awesome, thanks@oandersonmagalhaes! 🚀

And thanks for the reviews@LorhanSohaky and@fcrozetta 🙇

oandersonmagalhaes and LorhanSohaky reacted with rocket emoji

@tiangolotiangolo merged commitf2d01d7 intofastapi:masterMay 8, 2023
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

2 more reviewers

@fcrozettafcrozettafcrozetta approved these changes

@LorhanSohakyLorhanSohakyLorhanSohaky approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

approved-2lang-allTranslationslang-ptPortuguese translations

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@oandersonmagalhaes@LorhanSohaky@tiangolo@fcrozetta@Kludex

Comments


[8]ページ先頭

©2009-2026 Movatter.jp