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.

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

AL-18 Crud #53

Merged
merged 21 commits into from
Nov 18, 2021
Merged

AL-18 Crud #53

merged 21 commits into from
Nov 18, 2021

Conversation

mrharpo
Copy link
Collaborator

@mrharpo mrharpo commented Nov 16, 2021

CRUD

  • added fastapi-crudrouter for automatic routes based on Pydantic model
    • POST /thing - create new thing
    • GET /thing - get all things
    • GET /thing/{id} - get thing with id
    • PUT /thing/{id} - update thing
    • DELETE /thing/{id} - delete thing
    • DELETE /thing - delete all things
  • automatic documentation through FastAPI
  • motor backend with custom patch of fastapi-crudrouter

api

  • split functions into better file structure
    • admin.py invoicing
    • testapi.py basic (unauthenticated) api for isolated testing
  • migrated to Otto's Edl and Clip models

misc

  • Dockerfile order updates
  • passed DB_NAME as environment variable along with DB_URL
    Plus many small bugfixes

Copy link
Contributor

@subcontrabass subcontrabass left a comment

Choose a reason for hiding this comment

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

Initially got error of no odmantic installed, and after adding to reqs.txt got 'resolution impossible' due to conflicting depedancies.

@subcontrabass
Copy link
Contributor

odmantic_conflict

Copy link
Contributor

@subcontrabass subcontrabass left a comment

Choose a reason for hiding this comment

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

Looking great, well done!

@mrharpo mrharpo merged commit 9de8db4 into main Nov 18, 2021
@mrharpo mrharpo deleted the crud branch November 18, 2021 03:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants