Skip to content
This repository was archived by the owner on May 25, 2026. It is now read-only.

FastAPI and testing example with supadantic (docs)#87

Merged
makridenko merged 8 commits into
makridenko:masterfrom
AlexPozh:docs/40
Jul 3, 2025
Merged

FastAPI and testing example with supadantic (docs)#87
makridenko merged 8 commits into
makridenko:masterfrom
AlexPozh:docs/40

Conversation

@AlexPozh

Copy link
Copy Markdown
Contributor

closes #40

Comment thread docs/example_usage/fast_api_usage.md Outdated
Comment thread docs/example_usage/fast_api_usage.md Outdated
Comment thread docs/example_usage/fast_api_usage.md Outdated
Comment thread docs/example_usage/fast_api_usage.md Outdated
Comment thread docs/example_usage/fast_api_usage.md Outdated
Comment thread docs/example_usage/fast_api_usage.md Outdated
Comment thread docs/example_usage/fast_api_usage.md Outdated
Comment thread docs/example_usage/fast_api_usage.md Outdated
Comment thread docs/example_usage/fast_api_usage.md Outdated
@AlexPozh AlexPozh changed the title FastAPI example with supadantic (docs) FastAPI and testing example with supadantic (docs) Jun 22, 2025

@makridenko makridenko left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we make endpoints more RESTful?

  • GET /authors/ (Authors list)
  • GET /authors/:id/ (Retrieve author)
  • POST /authors/ (Create author)
  • PATCH /authors/:id/ (Update author)
  • DELETE /authors/:id/ (Delete author and books)
  • GET /authors/:id/books/ (Author's book list)
  • GET /authors/:id/books/:book_id/ (Retrieve book)
  • POST /authors/:id/books/ (Create book)
  • PATCH /authors/:id/books/:book_id/ (Update book)
  • DELETE /authors/:id/books/:book_id/ (Delete book)

Also, in books endpoints we can catch two different exceptions, Author.DoesNotExist and Book.DoesNotExist. It would be nice if we will show it. Two different responses - Author not found or book not found

Comment thread docs/fast_api_usage.md Outdated
Comment thread docs/fast_api_usage.md Outdated
Comment thread docs/fast_api_usage.md Outdated
Comment thread docs/fast_api_usage.md
Comment thread docs/fast_api_usage.md Outdated
Comment thread docs/fast_api_usage.md Outdated
Comment thread docs/fast_api_usage.md Outdated
Comment thread docs/fast_api_usage.md Outdated
Comment thread docs/fast_api_usage.md Outdated
Comment thread docs/fast_api_usage.md Outdated
Comment thread docs/fast_api_usage.md Outdated
Comment thread docs/fast_api_usage.md Outdated
Comment thread docs/fast_api_usage.md Outdated
Comment thread docs/fast_api_usage.md Outdated
Comment thread docs/fast_api_usage.md Outdated
Comment thread docs/fast_api_usage.md Outdated
Comment thread docs/fast_api_usage.md Outdated
Comment thread docs/fast_api_usage.md
Comment thread docs/fast_api_usage.md
Comment thread docs/fast_api_usage.md Outdated
Comment thread docs/fast_api_usage.md Outdated
Comment thread docs/fast_api_usage.md Outdated
Comment thread docs/fast_api_usage.md
Comment thread docs/fast_api_usage.md Outdated
Comment thread docs/fast_api_usage.md Outdated
@makridenko makridenko merged commit e87646e into makridenko:master Jul 3, 2025
9 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add usage section to documentation

2 participants