Skip to content

This is New Feature & upgration in 2025 #11

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ankush21sali
Copy link

Title :
Updated routes.py, requirements.txt, and added Dockerfile for better app setup

Description :

What I did:
In this pull request, I made several updates to improve the application setup and dependencies. Here are the main changes:

  • Updated routes.py: Improved routing logic for better application flow.
  • Updated requirements.txt: Removed old versions of libraries and added new ones to ensure compatibility with the latest features.
  • Added Dockerfile: Created a Dockerfile to containerize the app, making it easier to deploy and run the application in different environments.

Changes made :

  • Refactored routes in routes.py to streamline request handling.
  • Updated and optimized libraries in requirements.txt.
  • Created a Dockerfile to allow for easy setup and deployment of the app in a Docker container.

How I tested it :

  • Ran the application locally to ensure the new routes and dependencies work properly.
  • Built and tested the Docker container to confirm that the app runs smoothly inside the container.

Next Steps :

  • You can build the Docker image using docker build -t flask-app . and run it with docker run -p 5000:5000 flask-app.
  • Please test the new features and let me know if there are any issues!

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.

1 participant