Skip to content
This repository has been archived by the owner on May 16, 2020. It is now read-only.

Email notification #277

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

Email notification #277

wants to merge 11 commits into from

Conversation

kracekumar
Copy link
Contributor

  • send_newsletter.py should be run as cron job like

* 23 * * * cd /path/to/hacknight/repo && python send_newsletter.py production

@kracekumar
Copy link
Contributor Author

So I did git push -f, before that author name and email was reset with the help of http://stackoverflow.com/questions/3042437/change-commit-author-at-one-specific-commit. This has side effects. All comments to this pull request is gone. So its better have wrong author name and email and move on.

Comments

Jace: @kracekumar you have to pop before executing. Also, where is this function getting called from so as to not have a request context?

Kracekumar: @ jace : Function is called from main, send_newsletter.py is intended to run as cron job.

@jace
Copy link
Member

jace commented Dec 31, 2013

There was only one comment, so it's okay to lose it.

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

Successfully merging this pull request may close these issues.

2 participants