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

🌐 Add French translation for docs/fr/docs/tutorial/body-multiple-params.md #11796

Merged

Conversation

pe-brian
Copy link
Contributor

@pe-brian pe-brian commented Jul 4, 2024

Bonjour 🇫🇷 🤗

Here is the PR to translate the "Body multiple params" tutorial.
See the french translation discussion.

Thanks for the reviews 🙏

@tiangolo
Copy link
Member

tiangolo commented Jul 4, 2024

📝 Docs preview for commit dd53c66 at: https://18c01939.fastapitiangolo.pages.dev

@tiangolo
Copy link
Member

tiangolo commented Jul 4, 2024

📝 Docs preview for commit 8bad0b2 at: https://6a27f490.fastapitiangolo.pages.dev

@tiangolo
Copy link
Member

tiangolo commented Jul 4, 2024

📝 Docs preview for commit c45642a at: https://7c0c0d31.fastapitiangolo.pages.dev

@estebanx64 estebanx64 added docs Documentation about how to use FastAPI lang-fr French translations lang-all Translations awaiting-review labels Jul 4, 2024
Copy link

@GDemay GDemay left a comment

Choose a reason for hiding this comment

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

Good job!

docs/fr/docs/tutorial/body-multiple-params.md Outdated Show resolved Hide resolved
@pe-brian
Copy link
Contributor Author

Thank you !

@alejsdev alejsdev removed the docs Documentation about how to use FastAPI label Aug 5, 2024
@alejsdev
Copy link
Member

alejsdev commented Aug 5, 2024

Hi @GDemay, would you approve it? do you think is ready to be merged? 😄

Copy link

@GDemay GDemay left a comment

Choose a reason for hiding this comment

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

Good, ready to merge from my perspective :)

@alejsdev alejsdev merged commit dc384c4 into fastapi:master Aug 5, 2024
44 checks passed
@alejsdev
Copy link
Member

alejsdev commented Aug 5, 2024

Thank you for the help! @pe-brian @GDemay 🤓 ✨

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting-review lang-all Translations lang-fr French translations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants