Skip to content
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

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

Arcturus22
Copy link

Solved Issue #2006
Could you please review the code and merge it?
@NitkarshChourasia

@@ -0,0 +1,56 @@
import smtplib, ssl
Copy link
Contributor

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.

Copy link
Author

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.

Copy link
Contributor

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.

Email_Scheduler.py Outdated Show resolved Hide resolved
Copy link
Contributor

@NitkarshChourasia NitkarshChourasia left a 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.

@@ -0,0 +1,56 @@
import smtplib, ssl
Copy link
Contributor

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.

@Arcturus22 Arcturus22 closed this Oct 12, 2023
@Arcturus22 Arcturus22 reopened this Oct 12, 2023
@Arcturus22
Copy link
Author

@NitkarshChourasia @geekcomputers
I have added the README.md as you asked for.
Please review the code and the README.md and merge it.

@Arcturus22
Copy link
Author

@NitkarshChourasia
Could you please merge this as I have made the suggested changes that you had approved of?
Suggest other changes(if any)

Copy link
Contributor

@NitkarshChourasia NitkarshChourasia left a 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants