Added Docker support and fixed dependency issues with flask and Werkzeug#13
Open
Awalamoo7 wants to merge 4 commits into
Open
Added Docker support and fixed dependency issues with flask and Werkzeug#13Awalamoo7 wants to merge 4 commits into
Awalamoo7 wants to merge 4 commits into
Conversation
Owner
|
Hey @Awalamoo7! Just reviewed your PR and I have to say this is exactly the kind of contribution this project needed. Clean commit history, proper description, no unnecessary files. Really appreciate the effort and quality here. I'll run some local tests to make sure nothing breaks with the dependency changes (Flask/Werkzeug downgrade) and to verify the full Docker setup works end to end, but rest assured your contribution is great and I'll be merging this very soon. 🙌 Stay tuned! |
Author
|
Hi @Remy349 , Thank you. This means a lot. Kindly let me know if there are adjustments you need me to make to my commit before you merge |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR adds Docker support to simplify running the application.
Changes:
You can now run the app with:
docker compose up --build
Frontend: http://localhost:8080
Backend: http://localhost:5001
API Docs: http://localhost:5001/docs