All-In-One-Python-Projects icon indicating copy to clipboard operation
All-In-One-Python-Projects copied to clipboard

Add Dockerfile and Refactor Weather Alert Application: User Input for Dynamic Weather Alerts

Open Amitabh-DevOps opened this issue 1 year ago • 4 comments

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

Screenshot 2024-10-25 155527

Screenshot 2024-10-25 155151

Screenshot 2024-10-25 155207

Amitabh-DevOps avatar Oct 25 '24 10:10 Amitabh-DevOps

👋 @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.

github-actions[bot] avatar Oct 25 '24 10:10 github-actions[bot]

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>

Amitabh-DevOps avatar Oct 26 '24 03:10 Amitabh-DevOps

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

Amitabh-DevOps avatar Oct 26 '24 03:10 Amitabh-DevOps

I think it is so long time done , please check my PR

Amitabh-DevOps avatar Nov 02 '24 08:11 Amitabh-DevOps