Skip to content
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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Develop #407

Open
wants to merge 46 commits into
base: main
Choose a base branch
from
Open

Develop #407

wants to merge 46 commits into from

Conversation

Anton-Konyk
Copy link

No description provided.

Anton Konyk added 30 commits December 4, 2024 13:57
create main.py with def root for first test run
c72c434
create requirements.txt
be778f3
create db package
c791308
create engine.py for project and for work with library.db
2784606
add new requirements
07c73be
rename from engine.py
0fcc960
create models.py with DBAuthor and DBBook models
5107a67
create schemas.py for data validation and serialization
4e8b286
add fild bio to DBAuthor
26365a6
remove fild author from DBBook
9daaa4b
create schemas.py with AuthorBase class
65727d6
create AuthorCreate class
ef93be2
create Author class
7c46901
create BookBase class
2ac267a
create BookCreate class
b538589
create Book class
0bdaaee
create crud.py
df6e6d6
create function create_author
b056211
create function create_book
f222250
init alembic
3c9b24a
add alembic
c2a6847
add actual sqlalchemy.url
4bd7aad
add target_metadata value for current project
c58759f
add migration
2ddf399
add library.db
4ccc2e1
add uvicorn
12c164a
add relationship author to DBBook model
8c9612e
create get_author_by_name function
989e0cb
create get_db for Session
90dbb08
create function create_author
6a3f505
mateacademy-ai-mentor
mateacademy-ai-mentor approved these changes Dec 7, 2024
View reviewed changes
Copy link

@mateacademy-ai-mentor mateacademy-ai-mentor left a comment

Choose a reason for hiding this comment

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

Great job! 🎉 Your code meets all the task requirements and constraints perfectly. The project structure is well-organized, and all functionalities are implemented correctly. Keep up the excellent work! 🚀

✨ Please rate this review in the Platform's notification center. I will be grateful for your feedback! ✨

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mateacademy-ai-mentor mateacademy-ai-mentor mateacademy-ai-mentor approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Anton-Konyk @mateacademy-ai-mentor