Skip to content

🌐 Add Ukrainian translation for docs/uk/docs/tutorial/path-params.md#13354

Merged
tiangolo merged 5 commits into
fastapi:masterfrom
valentinDruzhinin:translation-uk/path-params
Feb 28, 2025
Merged

🌐 Add Ukrainian translation for docs/uk/docs/tutorial/path-params.md#13354
tiangolo merged 5 commits into
fastapi:masterfrom
valentinDruzhinin:translation-uk/path-params

Conversation

@valentinDruzhinin

@valentinDruzhinin valentinDruzhinin commented Feb 12, 2025

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions github-actions Bot added the lang-all Translations label Feb 12, 2025
@github-actions

github-actions Bot commented Feb 12, 2025

Copy link
Copy Markdown
Contributor

📝 Docs preview for commit f60637a at: https://a1eca1c9.fastapitiangolo.pages.dev

Modified Pages

@alejsdev alejsdev changed the title Add ukrainian translation for path-params.md 🌐 Update Ukrainian translation for docs/uk/docs/tutorial/path-params.md Feb 12, 2025
@alejsdev alejsdev changed the title 🌐 Update Ukrainian translation for docs/uk/docs/tutorial/path-params.md 🌐 Add Ukrainian translation for docs/uk/docs/tutorial/path-params.md Feb 12, 2025
@alejsdev alejsdev added awaiting-review lang-vi Vietnamese translations labels Feb 12, 2025
@valentinDruzhinin

valentinDruzhinin commented Feb 12, 2025

Copy link
Copy Markdown
Contributor Author

Hi @alejsdev could you please change lang-vi label on lang-uk? Because it's ukrainian translation.

@alejsdev alejsdev added lang-uk Ukrainian translations and removed lang-vi Vietnamese translations labels Feb 12, 2025
@rostik1410

rostik1410 commented Feb 18, 2025

Copy link
Copy Markdown
Contributor

@valentinDruzhinin awesome work. Same ask as for other PRs about Ви but not required because it already has 2 approves :)

@valentinDruzhinin valentinDruzhinin force-pushed the translation-uk/path-params branch from f60637a to 52c582d Compare February 18, 2025 21:54
@github-actions

github-actions Bot commented Feb 18, 2025

Copy link
Copy Markdown
Contributor

📝 Docs preview for commit 52c582d at: https://0478d5f2.fastapitiangolo.pages.dev

Modified Pages

@valentinDruzhinin

valentinDruzhinin commented Feb 18, 2025

Copy link
Copy Markdown
Contributor Author

@valentinDruzhinin awesome work. Same ask as for other PRs about Ви but not required because it already has 2 approves :)

Thanks, added fix :)

svlandeg
svlandeg reviewed Feb 24, 2025
View reviewed changes

@svlandeg svlandeg left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

As usual @valentinDruzhinin this is looking good from a structural point of view. Thanks for all your hard work translating these pages! 🙏 🚀 🤩

Comment thread docs/uk/docs/tutorial/path-params.md Outdated
valentinDruzhinin and others added 2 commits February 24, 2025 18:16
@valentinDruzhinin @svlandeg
Update docs/uk/docs/tutorial/path-params.md
21ff430 
Co-authored-by: Sofie Van Landeghem <svlandeg@users.noreply.github.com>
@valentinDruzhinin
Merge branch 'master' into translation-uk/path-params
5d798df
@github-actions

github-actions Bot commented Feb 24, 2025

Copy link
Copy Markdown
Contributor

📝 Docs preview for commit 5d798df at: https://2f000ed5.fastapitiangolo.pages.dev

Modified Pages

tiangolo
tiangolo approved these changes Feb 28, 2025
View reviewed changes

@tiangolo tiangolo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks @valentinDruzhinin! 🍰

And thanks all for the reviews ☕

@tiangolo tiangolo merged commit ee729d4 into fastapi:master Feb 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@svlandeg svlandeg svlandeg left review comments
@tiangolo tiangolo tiangolo approved these changes
+3 more reviewers
@rostik1410 rostik1410 rostik1410 approved these changes
@mezgoodle mezgoodle mezgoodle approved these changes
@SofiiaTrufanova SofiiaTrufanova SofiiaTrufanova approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

approved-2 lang-all Translations lang-uk Ukrainian translations

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@valentinDruzhinin @rostik1410 @tiangolo @svlandeg @mezgoodle @SofiiaTrufanova @alejsdev