]> git.ipfire.org Git - thirdparty/fastapi/sqlmodel.git/commitdiff
📝 Tweak wording in `docs/tutorial/fastapi/multiple-models.md` (#674)
authorSebastián Ramírez <tiangolo@gmail.com>
Mon, 23 Oct 2023 20:37:07 +0000 (00:37 +0400)
committerGitHub <noreply@github.com>
Mon, 23 Oct 2023 20:37:07 +0000 (20:37 +0000)
Co-authored-by: Luis Benitez <lbenitez000@gmail.com>
docs/tutorial/fastapi/multiple-models.md

index c37fad386b99d370fccb573e799b991cba7563a1..6845b9862d87a7c748ca069c0166ac775f3dcf7a 100644 (file)
@@ -53,11 +53,11 @@ Here's the weird thing, the `id` currently seems also "optional". 🤔
 
 This is because in our **SQLModel** class we declare the `id` with `Optional[int]`, because it could be `None` in memory until we save it in the database and we finally get the actual ID.
 
-But in the responses, we would always send a model from the database, and it would **always have an ID**. So the `id` in the responses could be declared as required too.
+But in the responses, we always send a model from the database, so it **always has an ID**. So the `id` in the responses can be declared as required.
 
-This would mean that our application is making the compromise with the clients that if it sends a hero, it would for sure have an `id` with a value, it would not be `None`.
+This means that our application is making the promise to the clients that if it sends a hero, it will for sure have an `id` with a value, it will not be `None`.
 
-### Why Is it Important to Compromise with the Responses
+### Why Is it Important to Have a Contract for Responses
 
 The ultimate goal of an API is for some **clients to use it**.