-
Notifications
You must be signed in to change notification settings - Fork 104
Add Dockerfile and Refactor Weather Alert Application: User Input for Dynamic Weather Alerts #85
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
Conversation
👋 @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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @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!
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
Uh oh!
There was an error while loading. Please reload this page.
Developer Checklist
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