search

jonge-democraten / website

JD website
https://jongedemocraten.nl
MIT License
6 stars 2 forks source link

NewsletterToList date not set in migration #131

Closed trolando closed 8 years ago

trolando commented 8 years ago

I think the actual date that the newsletter is sent (ie newslettertolist date) should be set in the migration. I changed the field from auto_now to default=timezone.now to allow this to be set.

Pi2048 commented 8 years ago

Isn't this set currently? I think that both the date of the Newsletter and the NewsletterToList are set to the max(create_date, send_date) of the mailing. This is to compensate for the fact that many mailings have a wrong date (1970) set for one of the two.

Pi2048 commented 8 years ago

Yes, line 338 and 346 of jd_finalize.py.

trolando commented 8 years ago

Ah then the next migration call will pick this up.