search

aldryn / aldryn-blog

aldryn-blog IS DEPRECATED. PLEASE USE aldryn-newsblog!
Other
20 stars 35 forks source link

publication_date should be a DateTimeField #3

Closed beniwohli closed 11 years ago

beniwohli commented 11 years ago

To allow publications at a certain time of the day

szuliq commented 11 years ago

@piquadrat Isn't it supposed to be a dead simple blog application? Is it a common use case to publish at certain time of the day when you're not a newspaper?

It adds complexity and probably more then 95% usages will have time set to 00:00.

beniwohli commented 11 years ago

@szuliq "dead simple" doesn't mean "useless"... with no specific publication time, this app couldn't be used for such things as press releases or announcements. And I don't really see how it adds any complexity. Just set the default date to "today, midnight".

szuliq commented 11 years ago

"Certain time of the day" concept brings a whole new level of complexity, namely timezones.

Is Aldryn timezone aware at all? Is it planned to be?

The basic use case is covered: you don't want to publish the article yet because it is being created or want to do so on some more-less specific date.

If we're going to introduce datetime as a publication time, I opt for renaming publication_date to publication_time to avoid mistakes and make sure users will be able to set time in their respective timezones.