Scheduling morning emails with Python and Celery (opens in new tab)

(cucumbertown.com)

56 pointsdeepakprakash13y ago17 comments

17 comments

Isn't the ScheduleTime class doing redundant work? Why bother keeping a dictionary, dealing with ints and strings when you can directly use strftime/strptime?

Except for the friday/fri thing, which, to be honest, I don't see the point of. I'd rather have uniformity in my code.

aqeel13y ago

Agree. Its not the most useful class.

est13y ago

alternatively: use uWSGI cron

http://uwsgi-docs.readthedocs.org/en/latest/PythonDecorators...

aqeel13y ago

Good one! People relying on Cron could benefit from this.

yeison13y ago

Did anyone else read this title as: 'Scheduling morning meals with Python and Celery'?

aqeel13y ago

Haha, its so funny! especially considering the fact that http://www.cucumbertown.com is a recipe site :)

clicks13y ago

While we're on the topic -- there's also http://mailchimp.com/ if you want to outsource the job of sending subscription e-mail.

th13y ago

Would this create a problem around Daylight Saving Time until the user logs in again to reset their time offset?

aqeel13y ago

getTimezoneOffset() would return the value after taking care of DST. So there shouldn't be any DST related inconsistencies. Now if the user moves to another timezone, and never hits one of our page, there is no direct way of tracking his/her change in location.

th13y ago

I mean that starting mid-March a user who had previously been in a -8:00 UTC offset would then be in a -7:00 UTC offset and you wouldn't know the timezone offset changed until they visited the website again.

1 more reply

ieze13y ago

This doesn't handle daylight saving time.

aqeel13y ago

It does take care of DST: http://www.w3schools.com/jsref/jsref_gettimezoneoffset.asp

cyriacthomas13y ago

Wouldn't it be a better idea to store the timezone and then account for the DST at the server side? https://bitbucket.org/pellepim/jstimezonedetect

getTimezoneOffset() gives you the offset with the DST accounted and will be a different time when DST is off.

1 more reply

ericcholis13y ago

Better Reference (W3 Schools sucks: http://w3fools.com/):

https://developer.mozilla.org/en-US/docs/JavaScript/Referenc...

j / k navigate · click thread line to collapse

17 comments

manish_gill13y ago

Isn't the ScheduleTime class doing redundant work? Why bother keeping a dictionary, dealing with ints and strings when you can directly use strftime/strptime?

Except for the friday/fri thing, which, to be honest, I don't see the point of. I'd rather have uniformity in my code.

aqeel13y ago

Agree. Its not the most useful class.

est13y ago

alternatively: use uWSGI cron

http://uwsgi-docs.readthedocs.org/en/latest/PythonDecorators...

aqeel13y ago

Good one! People relying on Cron could benefit from this.

yeison13y ago

Did anyone else read this title as: 'Scheduling morning meals with Python and Celery'?

aqeel13y ago

Haha, its so funny! especially considering the fact that http://www.cucumbertown.com is a recipe site :)

clicks13y ago

While we're on the topic -- there's also http://mailchimp.com/ if you want to outsource the job of sending subscription e-mail.

th13y ago

Would this create a problem around Daylight Saving Time until the user logs in again to reset their time offset?

aqeel13y ago

th13y ago

1 more reply

ieze13y ago

This doesn't handle daylight saving time.

aqeel13y ago

It does take care of DST: http://www.w3schools.com/jsref/jsref_gettimezoneoffset.asp

cyriacthomas13y ago

Wouldn't it be a better idea to store the timezone and then account for the DST at the server side? https://bitbucket.org/pellepim/jstimezonedetect

getTimezoneOffset() gives you the offset with the DST accounted and will be a different time when DST is off.

1 more reply

ericcholis13y ago

Better Reference (W3 Schools sucks: http://w3fools.com/):

https://developer.mozilla.org/en-US/docs/JavaScript/Referenc...

j / k navigate · click thread line to collapse