Skip to content

Rain Alert Notification #1070

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

Merged
merged 32 commits into from
Oct 31, 2021
Merged

Rain Alert Notification #1070

merged 32 commits into from
Oct 31, 2021

Conversation

vkb20
Copy link
Contributor

@vkb20 vkb20 commented Oct 31, 2021

Related Issue

  • Using python script to navigate to https://openweathermap.org/api and then getting 12 hour weather forecast and then emailing the registered user about the weather.

  • Hacktoberfest2021 Participant

  • Contributor

Closes: #936

Describe the changes you've made

I did all the tasks from scratch. Created the rain_alert_notification.py file for script. Created README.md file for explaining the project. Added relevant screenshots and added requirement.txt file.

Type of change

What sort of change have you made:

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Code style update (formatting, local variables)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

I ran the script in the command prompt and received the weather updates on my email ID.

Checklist:

  • My code follows the guidelines of this project.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly whereever it was hard to understand.
  • I have made corresponding changes to the documentation.
  • My changes generate no new warnings.
  • I have added tests that prove my fix is effective or that my feature works.
  • Any dependent changes have been merged and published in downstream modules.

Screenshots

rain_alert_notificaiton

@vkb20
Copy link
Contributor Author

vkb20 commented Oct 31, 2021

@prathimacode-hub check this.

Copy link
Owner

@prathimacode-hub prathimacode-hub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No changes needed. @vkb20

Copy link
Owner

@prathimacode-hub prathimacode-hub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR approved. Well done. Welcome to repo officially. 👍✨@vkb20

@prathimacode-hub prathimacode-hub added HACKTOBERFEST This issue is applicable for Hacktoberfest 2022 HACKTOBERFEST-ACCEPTED This PR is approved and accepted for Hacktoberfest labels Oct 31, 2021
@prathimacode-hub prathimacode-hub merged commit 0b50533 into prathimacode-hub:main Oct 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
HACKTOBERFEST This issue is applicable for Hacktoberfest 2022 HACKTOBERFEST-ACCEPTED This PR is approved and accepted for Hacktoberfest
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Rain Alert Notification
2 participants