Thanks to visit codestin.com
Credit goes to github.com

Skip to content

🌐 Add Portuguese translation for docs/pt/docs/advanced/security/oauth2-scopes.md#12263

Merged
alejsdev merged 8 commits into
fastapi:masterfrom
ceb10n:features/pt-oauth2-scopes
Oct 4, 2024
Merged

🌐 Add Portuguese translation for docs/pt/docs/advanced/security/oauth2-scopes.md#12263
alejsdev merged 8 commits into
fastapi:masterfrom
ceb10n:features/pt-oauth2-scopes

Conversation

@ceb10n

@ceb10n ceb10n commented Sep 25, 2024

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions github-actions Bot added the lang-all Translations label Sep 25, 2024
@github-actions

github-actions Bot commented Sep 25, 2024

Copy link
Copy Markdown
Contributor

📝 Docs preview for commit 4e591f1 at: https://e4bffa75.fastapitiangolo.pages.dev

Modified Pages

@alejsdev alejsdev added lang-pt Portuguese translations approved-1 labels Sep 25, 2024
Joao-Pedro-P-Holanda
Joao-Pedro-P-Holanda approved these changes Sep 26, 2024
View reviewed changes

@Joao-Pedro-P-Holanda Joao-Pedro-P-Holanda left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Great job, just highlighted some minor typos. Seems like the admonition blocks (///) don't need double quotes for the translation. I didn't marked all occurences to avoid polluting the pull request thread.

Comment thread docs/pt/docs/advanced/security/oauth2-scopes.md
Comment thread docs/pt/docs/advanced/security/oauth2-scopes.md Outdated
Comment thread docs/pt/docs/advanced/security/oauth2-scopes.md Outdated
Comment thread docs/pt/docs/advanced/security/oauth2-scopes.md Outdated
Comment thread docs/pt/docs/advanced/security/oauth2-scopes.md
Comment thread docs/pt/docs/advanced/security/oauth2-scopes.md Outdated
Comment thread docs/pt/docs/advanced/security/oauth2-scopes.md Outdated
Comment thread docs/pt/docs/advanced/security/oauth2-scopes.md
@ceb10n

ceb10n commented Sep 27, 2024

Copy link
Copy Markdown
Contributor Author

Great job, just highlighted some minor typos. Seems like the admonition blocks (///) don't need double quotes for the translation. I didn't marked all occurences to avoid polluting the pull request thread.

Thank you Joao 😄 Fixed it

@github-actions

github-actions Bot commented Sep 27, 2024

Copy link
Copy Markdown
Contributor

📝 Docs preview for commit 6008f65 at: https://6d659597.fastapitiangolo.pages.dev

Modified Pages

@github-actions

github-actions Bot commented Sep 28, 2024

Copy link
Copy Markdown
Contributor

📝 Docs preview for commit 2460d93 at: https://7ad5667f.fastapitiangolo.pages.dev

Modified Pages

@alejsdev alejsdev merged commit 0e7806e into fastapi:master Oct 4, 2024
@alejsdev

alejsdev commented Oct 4, 2024

Copy link
Copy Markdown
Member

Thanks @ceb10n @nazarepiedady @Joao-Pedro-P-Holanda 🚀

s-rigaud pushed a commit to s-rigaud/fastapi that referenced this pull request Jan 23, 2025
@ceb10n @s-rigaud
🌐 Add Portuguese translation for `docs/pt/docs/advanced/security/oaut…
877f81a 
…h2-scopes.md` (fastapi#12263)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers

@nazarepiedady nazarepiedady nazarepiedady approved these changes

@Joao-Pedro-P-Holanda Joao-Pedro-P-Holanda Joao-Pedro-P-Holanda approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

approved-2 lang-all Translations lang-pt Portuguese translations

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@ceb10n @alejsdev @nazarepiedady @Joao-Pedro-P-Holanda