search

datamade / nyc-council-councilmatic

NYC Council version of Councilmatic
MIT License
7 stars 3 forks source link

Log #115

Closed reginafcompton closed 6 years ago

reginafcompton commented 6 years ago

@hancush - this PR adds a cronjob that runs the RTF conversion script after each data_import.

It handles the last part of this issue: https://github.com/datamade/nyc-council-councilmatic/issues/108

fgregg commented 6 years ago

It would be safer to chain the commands with an &&

On Wed, Jan 10, 2018 at 4:51 PM, Regina Compton notifications@github.com wrote:

@hancush https://github.com/hancush - this PR adds a cronjob that runs the RTF conversion script after each data_import. For production, I used flock, though by locking the same file. Then, if the conversion script takes longer than 15 minutes, a new import cannot start.

You can view, comment on, or merge this pull request online at:

https://github.com/datamade/nyc-council-councilmatic/pull/115 Commit Summary

  • Correct log for production
  • Update crontask

File Changes

Patch Links:

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/datamade/nyc-council-councilmatic/pull/115, or mute the thread https://github.com/notifications/unsubscribe-auth/AAgxbb_ZDnRHLYi9jPSCiQs0sQlYY0YSks5tJT7jgaJpZM4RaEKo .

-- 773.888.2718

reginafcompton commented 6 years ago

I do not want to merge this PR nor turn on the NYC cron tasks, until we get this sorted out: https://github.com/datamade/django-councilmatic/pull/169

@hancush, do you feel comfortable with that?