search

jrgifford / delayed_paperclip

Process your Paperclip attachments in the background with delayed_job or Resque.
http://www.jstorimer.com/ruby/2010/01/30/delayed-paperclip.html
MIT License
402 stars 155 forks source link

fixed delayed job upgrade and cleaned up dependencies #32

Closed ScotterC closed 11 years ago

ScotterC commented 11 years ago

The main fix is adding the queue column to DelayedJob's testing table. DelayedJob 3.0.4 requires it otherwise it will silently rase an unknown attribute :queue error when enqueuing it.

I also removed duplication of the dependencies and loosened the paperclip version requirement to better reflect the gemspec's intentions.

jrgifford commented 11 years ago

restarted the build, since it errored out.

ScotterC commented 11 years ago

Looks like all but rubinius. What do you think about allowing failures there?

Scott Carleton Artsicle, Co-Founder / CTO scott@artsicle.com C. 914 400 5085

On Mon, Jun 3, 2013 at 10:45 AM, James Gifford notifications@github.comwrote:

restarted the build, since it errored out.

— Reply to this email directly or view it on GitHubhttps://github.com/jrgifford/delayed_paperclip/pull/32#issuecomment-18846010 .

jrgifford commented 11 years ago

I'm ok with allowing failures for rbx.