🌐 Add French translation for docs/fr/docs/advanced/additional-status-code.md
#5477
Conversation
Codecov ReportBase: 100.00% // Head: 100.00% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## master #5477 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 540 540
Lines 13969 13946 -23
=========================================
- Hits 13969 13946 -23
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
|
📝 Docs preview for commit a1a7f72 at: https://63428b33fbdff17456e4b188--fastapi.netlify.app |
|
Hi @axel584, you also need to update the |
|
📝 Docs preview for commit 8658e0a at: https://634328d8d6cd7b5b90f491c9--fastapi.netlify.app |
|
Le breaking change in the unit test will be resolve by this PR : #5456 |
|
📝 Docs preview for commit aa195d8 at: https://634668bcf6f89332a8126c53--fastapi.netlify.app |
|
📝 Docs preview for commit aeb6243 at: https://6346797a4e841a37b7c34ede--fastapi.netlify.app |
|
📝 Docs preview for commit a94bfa5 at: https://6346edeee15dea004f46d100--fastapi.netlify.app |
|
📝 Docs preview for commit 9efb26a at: https://6346f37f0775e9008d2e2df2--fastapi.netlify.app |
There was a problem hiding this comment.
Small remarks (do not hesitate to challenge them) but this is nice 🙌
Thanks !
|
|
||
| Si vous renvoyez directement des codes HTTP et des réponses supplémentaires, ils ne seront pas inclus dans le schéma OpenAPI (la documentation de l'API), car FastAPI n'a aucun moyen de savoir à l'avance ce que vous allez renvoyer. | ||
|
|
||
| Mais vous pouvez documenter cela dans votre code, en utilisant : [Réponses supplémentaires dans OpenAPI](additional-responses.md){.internal-link target=_blank}. |
There was a problem hiding this comment.
| Mais vous pouvez documenter cela dans votre code, en utilisant : [Réponses supplémentaires dans OpenAPI](additional-responses.md){.internal-link target=_blank}. | |
| Mais vous pouvez documenter cela dans votre code, en utilisant : [Reponses supplémentaires](additional-responses.md){.internal-link target=_blank}. |
There was a problem hiding this comment.
I don't understand why you want to remove the "é" in "Réponses"... and I prefer use the whole title which specify "dans OpenAPI"...
Co-authored-by: Julian Maurin <julian.maurin.perso@pm.me>
Co-authored-by: Julian Maurin <julian.maurin.perso@pm.me>
|
📝 Docs preview for commit 26deb29 at: https://634a4e93735def621336e05c--fastapi.netlify.app |
|
📝 Docs preview for commit e05d84f at: https://634c4f9c83d30f3235e20306--fastapi.netlify.app |
|
@JulianMaurin / @rjNemo : Does anyone know why my MR did not receive the "approved-2" label? |
docs/fr/docs/advanced/additional-status-code.mddocs/fr/docs/advanced/additional-status-code.md
|
Thank you @axel584! 🍰 And thanks for the reviews @JulianMaurin, @emmrichard, @rjNemo ☕ |
Here is the PR to translate the
advanced/additional-status-code.mdSee the french translation tracking issue #1972.
Thanks for the reviews