Add Dockerfile and Refactor Weather Alert Application: User Input for Dynamic Weather Alerts
Developer Checklist
- [x] Adhered to the guidelines outlined in the README & Contributing file.
- [x] Maintained the correct directory structure (e.g., ProjectName/...yourfiles).
- [x] Please ensure to include a README file for enhanced project clarity.
- [x] Starred ⭐ the repository (optional).
Summary
This pull request introduces a Dockerfile for the weather alert application, streamlining the deployment process and ensuring consistent environments across different systems. Additionally, it refactors the application to include a user input form for city names, enabling dynamic weather alert checks based on user-defined locations. The application retrieves weather data in real-time, enhancing user engagement and usability.
However, the application is currently hosted on localhost, limiting its accessibility. Future work should focus on deploying the application to a cloud platform to allow users from various locations to access the service. Additionally, implementing AJAX could allow for real-time updates without requiring a page refresh, making the application more dynamic.
Suggestion: Consider developing this application using Django for better structure and scalability. I would be happy to assist in making improvements and implementing best practices in that framework.
Screenshot
👋 @Amitabh-DevOps 👋
We're delighted to have your pull request! Please take a moment to check our contributing guidelines and ensure you've filled out the PR template for a smooth process. We will review it soon.
Yes i have done with changes which you have said , now review it
On Fri, Oct 25, 2024 at 10:58 PM Aman Kumar @.***> wrote:
@.**** requested changes on this pull request.
Hi @Amitabh-DevOps https://github.com/Amitabh-DevOps,
Thank you for your valuable contribution. We already have the project available in this repository. If you would like to contribute to the same project, please create a directory within the existing Weather Alert directory. I recommend using the following structure: Weather Alert/your-usrName/project-files (excluding the project name, as that directory is already defined).
Thank you!
— Reply to this email directly, view it on GitHub https://github.com/king04aman/All-In-One-Python-Projects/pull/85#pullrequestreview-2395953371, or unsubscribe https://github.com/notifications/unsubscribe-auth/BKAIEHVFCZZZHQ4EZP7VBPLZ5J5SDAVCNFSM6AAAAABQS765RCVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDGOJVHE2TGMZXGE . You are receiving this because you were mentioned.Message ID: @.*** com>
If there is any changes you want to suggest it please go ahead and review it if it is ok then proceed it
i have did not modified the existing code , i just created a dir with my username as you have told some how i have to copy the existing code of Weather Alert to get back the existing with seprately because i have changed the origanl README.md and all so make sure to check that also
I think it is so long time done , please check my PR