Skip to content

Covid-19 Real Time notification#969

Merged
kaustubhgupta merged 6 commits into
avinashkranjan:masterfrom
sayantani11:covid
May 24, 2021
Merged

Covid-19 Real Time notification#969
kaustubhgupta merged 6 commits into
avinashkranjan:masterfrom
sayantani11:covid

Conversation

@sayantani11
Copy link
Copy Markdown
Contributor

@sayantani11 sayantani11 commented Apr 26, 2021 •
edited by kaustubhgupta
Loading

Description

This a real time notification that keeps the user updated about covid related information of a particular state

Fixes #938

Replace issue_no with the issue number which is fixed in this PR

Have you read the Contributing Guidelines on Pull Requests?

  • Yes
  • No

Type of change

Please delete options that are not relevant.

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

Checklist:

  • My code follows the style guidelines(Clean Code) of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have created a helpful and easy to understand README.md
  • My documentation follows Template for README.md
  • My changes generate no new warnings
  • I have added tests/screenshots(if any) that prove my fix is effective or that my feature works.

Project Metadata

If there is no-file/nothing to fill the below fields with, then type: none

Example: If no requirements.txt needed/present, then type none in Requirments.

Category:

  • Calculators
  • AI/ML
  • Scrappers
  • Social_Media
  • PDF
  • Image_Processing
  • Video_Processing
  • Games
  • Networking
  • OS_Utilities
  • Automation
  • Cryptography
  • Computer_Vision
  • Fun
  • Others

Title: REAL TIME COVID-19 OUTBREAK NOTIFICATION

Folder: Covid-19_Real-time_Notification

Requirements: requirements.txt

Script: covid.py

Arguments: none

Contributor: sayantani11

Description: A script that notifies the user about the situation of covid-19 in the states specified at intervals

@sayantani11
Copy link
Copy Markdown
Contributor Author

sayantani11 commented Apr 26, 2021

@kaustubhgupta can you tell me the error?

kaustubhgupta
kaustubhgupta suggested changes Apr 27, 2021
View reviewed changes
Comment thread Covid-19 _Real-time_Notification/readME.md Outdated
Comment thread Covid-19 _Real-time_Notification/readME.md Outdated
Comment thread Covid-19 _Real-time_Notification/readME.md Outdated
Comment thread Covid-19 _Real-time_Notification/readME.md Outdated
Comment thread Covid-19 _Real-time_Notification/readME.md Outdated
Comment thread Covid-19 _Real-time_Notification/Covid.py Outdated
Comment thread Covid-19 _Real-time_Notification/Covid.py Outdated
Comment thread Covid-19 _Real-time_Notification/Covid.py Outdated
@kaustubhgupta kaustubhgupta added the bug Something isn't working label Apr 27, 2021
@sayantani11
Copy link
Copy Markdown
Contributor Author

sayantani11 commented Apr 27, 2021

@kaustubhgupta had to change the link because the previous one kept showing the error

kaustubhgupta
kaustubhgupta suggested changes Apr 28, 2021
View reviewed changes
Comment thread Covid-19 _Real-time_Notification/Covid.py Outdated
Comment thread Covid-19 _Real-time_Notification/Covid.py Outdated
Comment thread Covid-19 _Real-time_Notification/Covid.py Outdated
Comment thread Covid-19 _Real-time_Notification/Covid.py
Comment thread Covid-19 _Real-time_Notification/readME.md Outdated
@sayantani11
Copy link
Copy Markdown
Contributor Author

sayantani11 commented Apr 28, 2021

Ran it on jupyter lab works fine

kaustubhgupta
kaustubhgupta suggested changes Apr 29, 2021
View reviewed changes
Comment thread Covid-19 _Real-time_Notification/Covid.py
Comment thread Covid-19 _Real-time_Notification/Covid.py Outdated
@kaustubhgupta kaustubhgupta added next review needed Approved by some mentors, more approvals needed and removed bug Something isn't working labels Apr 29, 2021
@kaustubhgupta kaustubhgupta added Approved PR Approved and Ready to Merge gssoc23 Issues created for/by the GirlScript Summer of Code'23 Participants level1 Wiki or Major Documentation Work and removed next review needed Approved by some mentors, more approvals needed labels May 1, 2021
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 9, 2021

Due to inactivity this pull request has been marked as stale.

@github-actions github-actions Bot added the Stale PRs with no updates label May 9, 2021
@kaustubhgupta kaustubhgupta removed the Stale PRs with no updates label May 9, 2021
kaustubhgupta
kaustubhgupta suggested changes May 11, 2021
View reviewed changes
Copy link
Copy Markdown
Contributor

@kaustubhgupta kaustubhgupta left a comment

Choose a reason for hiding this comment

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

@sayantani11 Add the requirements.txt file and update the PR template with project metadata. Look at this PR for reference: #1002

@kaustubhgupta kaustubhgupta added hold Needs a second thought and removed Approved PR Approved and Ready to Merge labels May 11, 2021
@sayantani11
Copy link
Copy Markdown
Contributor Author

sayantani11 commented May 11, 2021

@kaustubhgupta Done.....Please check

@kaustubhgupta kaustubhgupta added Approved PR Approved and Ready to Merge and removed hold Needs a second thought labels May 12, 2021
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 20, 2021

Due to inactivity this pull request has been marked as stale.

@github-actions github-actions Bot added the Stale PRs with no updates label May 20, 2021
@kaustubhgupta kaustubhgupta removed the Stale PRs with no updates label May 20, 2021
@kaustubhgupta kaustubhgupta merged commit bd9f3c2 into avinashkranjan:master May 24, 2021
@kaustubhgupta kaustubhgupta added the manual-db-update-req The PR was merged but db was not updated. Need to update the database manully label May 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

3 more reviewers

@kaustubhgupta kaustubhgupta kaustubhgupta approved these changes

@antrikshmisri antrikshmisri antrikshmisri approved these changes

@santushtisharma10 santushtisharma10 santushtisharma10 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

Approved PR Approved and Ready to Merge gssoc23 Issues created for/by the GirlScript Summer of Code'23 Participants level1 Wiki or Major Documentation Work manual-db-update-req The PR was merged but db was not updated. Need to update the database manully

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

COVID-19 outbreak notification

4 participants

@sayantani11 @kaustubhgupta @antrikshmisri @santushtisharma10