FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

馃摑 Tweak wording in `docs/tutorial/fastapi/multiple-models.md` by tiangolo 路 Pull Request #674 路 fastapi/sqlmodel 路 GitHub

  1. fastapi
  2. sqlmodel

Repository navigation

馃摑 Tweak wording in docs/tutorial/fastapi/multiple-models.md #674

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.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
tiangolo merged 4 commits into from
Oct 23, 2023
Conversation 0 Commits 4 Checks 0 Files changed
Merged

馃摑 Tweak wording in docs/tutorial/fastapi/multiple-models.md #674

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
1d2ff1d
Replace usages of 'compromise' and use assertive language
lbenitez000 Sep 27, 2023
8bce484
Titleize
lbenitez000 Sep 27, 2023
1d08683
Merge branch 'main' into doc-compromise-rephrase
tiangolo Oct 23, 2023
b4e16b0
馃摑 Tweak wording
tiangolo Oct 23, 2023
File filter

Filter by extension

Filter by extension .md  (1) All 1 file type selected
Viewed files
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Unified
Split
Hide whitespace
Diff view
Unified
Split
Hide whitespace
6 changes: 3 additions & 3 deletions docs/tutorial/fastapi/multiple-models.md
Show comments
View file
Open in desktop
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Original file line number Diff line number Diff line change
Expand Up @@ -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**.

Expand Down
Back | FazBrowse Home | New Git URL