Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork8.7k
Comments
🌐 Add French translation fordocs/fr/docs/deployment/index.md#3689
🌐 Add French translation fordocs/fr/docs/deployment/index.md#3689tiangolo merged 10 commits intofastapi:masterfrom
docs/fr/docs/deployment/index.md#3689Conversation
deployment/index.mddeployment/index.mdcodecovbot commentedAug 12, 2021 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Codecov Report
@@ Coverage Diff @@## master #3689 +/- ##========================================= Coverage 100.00% 100.00% ========================================= Files 531 532 +1 Lines 13629 13672 +43 =========================================+ Hits 13629 13672 +43
Continue to review full report at Codecov.
|
📝 Docs preview for commit567c354 at:https://6114ce4eb773bd7e578cf3da--fastapi.netlify.app |
📝 Docs preview for commit37ee682 at:https://6114d0fdcc2d51748610f988--fastapi.netlify.app |
📝 Docs preview for commit710a203 at:https://61220e61f67165d21c9bb328--fastapi.netlify.app |
📝 Docs preview for commit76014e8 at:https://61f12eface347a905df6a9c7--fastapi.netlify.app |
📝 Docs preview for commit5b147b1 at:https://627b9399fc0e5463a411dcf7--fastapi.netlify.app |
arthurio left a comment• edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Looks good 👍🏻 🇫🇷
However, I just checkedhttps://github.com/tiangolo/fastapi/blob/master/docs/en/docs/deployment/index.md and looks like there is more to translate now?
arthurio left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
I didn't see that it was out of sync with current english version, but the translation was great!
rjNemo commentedJul 7, 2022
@arthurio I review the recent version as soon as possible. |
rjNemo commentedJul 7, 2022
@arthurio I pushed the latest changes. |
📝 Docs preview for commitf991d34 at:https://62c68c71c8ef8c4da59ee500--fastapi.netlify.app |
arthurio left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
2 questions, otherwise looks great 🤩
| utilisateurs**. | ||
| Pour une **API Web**, cela implique normalement de la placer sur une **machine distante**, avec un **programme serveur** | ||
| qui offre de bonnes performances, une bonne stabilité, _etc._, afin que vos **utilisateurs** puissent **accéder** à |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Why_ect._ instead of justetc.?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
I learned that Latin terms should be written in italics.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Thanks for the info 😃
Uh oh!
There was an error while loading.Please reload this page.
Co-authored-by: Arthur Rio <arthurio@users.noreply.github.com>
📝 Docs preview for commit3a54345 at:https://62c7db7569e707538f566159--fastapi.netlify.app |
arthurio left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Great work! 💯
deployment/index.mddocs/fr/docs/deployment/index.mdtiangolo commentedAug 18, 2022
…api#3689)Co-authored-by: Arthur Rio <arthurio@users.noreply.github.com>Co-authored-by: Ruidy <r.nemausat@empfohlen.de>Co-authored-by: Ruidy <rnemausat@newstore.com>Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
🤗
Here is the PR to translate the deployment intro.
See the french translation tracking issuehere.
Thanks for the reviews