Thanks to visit codestin.com
Credit goes to github.com

Skip to content

📝 Update image in body-nested-model docs#11063

Merged
alejsdev merged 6 commits into
fastapi:masterfrom
untilhamza:dev-update-image-for-nested-body-docs
Jan 3, 2025
Merged

📝 Update image in body-nested-model docs#11063
alejsdev merged 6 commits into
fastapi:masterfrom
untilhamza:dev-update-image-for-nested-body-docs

Conversation

@untilhamza

@untilhamza untilhamza commented Jan 31, 2024

Copy link
Copy Markdown
Contributor

What is this PR for:

  • This PR changes the image in the nested-body model's docs to use the HttpUrl imported from pydantic instead of UrlStr imported from pydantic.types as illustrated in the related docs.
    • This changes makes the code in the image consistent with the code in the docs

Old image
image01

New image
Screenshot 2024-01-31 at 8 21 13 PM

@tiangolo

Copy link
Copy Markdown
Member

📝 Docs preview for commit 04550ee at: https://54317ac8.fastapitiangolo.pages.dev

@alejsdev alejsdev changed the title docs: Update image in body-nested-model docs 📝 Update image in body-nested-model docs Jan 31, 2024
@alejsdev alejsdev added docs Documentation about how to use FastAPI p4 labels Jan 31, 2024
@tiangolo

Copy link
Copy Markdown
Member

📝 Docs preview for commit ad9695d at: https://e8d70064.fastapitiangolo.pages.dev

@alejsdev alejsdev self-assigned this Aug 5, 2024
@github-actions

Copy link
Copy Markdown
Contributor

@alejsdev alejsdev removed their assignment Jan 3, 2025
@github-actions

github-actions Bot commented Jan 3, 2025

Copy link
Copy Markdown
Contributor

@alejsdev alejsdev merged commit 8416e3e into fastapi:master Jan 3, 2025
@alejsdev

alejsdev commented Jan 3, 2025

Copy link
Copy Markdown
Member

Nice, thanks! (I made a small tweak to the image)
Thanks for your patience and help! @untilhamza 🙇‍♀️

s-rigaud pushed a commit to s-rigaud/fastapi that referenced this pull request Jan 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation about how to use FastAPI p4

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants