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

Update .env #60

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Update .env #60

wants to merge 3 commits into from

Conversation

Shashanknan
Copy link

No description provided.

abhilashkulkarniofficial and others added 3 commits May 2, 2021 14:56
* Changed email body

Added html email template to the email body to display the slots information in a cleaner template.

* Removed bug in creating template

* Updated Readme to add new email template
Revert was done as createTemplate method iterates on a string, and throws errors.

This reverts commit f04624d.
@abhilashkulkarniofficial
Copy link
Contributor

@Shashanknan please remove the password. Also if it is associated with your gmail account, please reset your password.

@Shashanknan
Copy link
Author

Shashanknan commented May 4, 2021 via email

@msvamp
Copy link

msvamp commented May 5, 2021

If you are assuming that updating the .env file and sending a pull request will make the script run online, that is not going to happen. You have to run the code locally on your computer.

By uploading the updated file online, you are publicly disclosing your email address and password to everyone, and only compromising your account's security and privacy.

@Shashanknan
Copy link
Author

Shashanknan commented May 5, 2021 via email

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.

4 participants