-
Notifications
You must be signed in to change notification settings - Fork 12k
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
Added Code for Python Bot for scheduling mail #2007
base: master
Are you sure you want to change the base?
Conversation
Email_Scheduler.py
Outdated
@@ -0,0 +1,56 @@ | |||
import smtplib, ssl |
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.
A README.md file, with a screenshot embedded for the program would be a good add on, for it to be a masterpiece. I would say.
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.
Embedded screenshots would like require me to reveal my email address, which I would not be very comfortable in.
I can add a README.md for this, though I have added comments to this code.
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.
Yeah, README.md would be fine.
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.
Just add a README.md file. I just review codes. Merging will be done by @geekcomputers in few days.
Email_Scheduler.py
Outdated
@@ -0,0 +1,56 @@ | |||
import smtplib, ssl |
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.
Yeah, README.md would be fine.
@NitkarshChourasia @geekcomputers |
@NitkarshChourasia |
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.
Thanks a lot for contributing.
It will soon be merged by @geekcomputers , Thanks a lot.
Solved Issue #2006
Could you please review the code and merge it?
@NitkarshChourasia