search

bikeindex / bike_index

All the code for Bike Index, because we love you
https://bikeindex.org
GNU Affero General Public License v3.0
274 stars 70 forks source link

Update Carrierwave, third try #2569

Closed sethherr closed 2 months ago

sethherr commented 2 months ago

Redo of PR https://github.com/bikeindex/bike_index/pull/2555 - which was reverted because it caused an error from Carrierwave Backgrounder.

This raises errors on production every time it's deployed.

I may just give up, because ultimately, I'd like to switch to active storage using Cloudflare R2

(which would mean I'd remove carrierwave_backgrounder, and do the migration to active storage after Rails is updated)

Here is an article on migrating, here is an article on background processing image variants

sethherr commented 2 months ago

There is this unresolved issue for:

NoMethodError: undefined methodcopy'`

For the carrierwave 3.0.7, which is the error that I've seen when this is deployed currently. So I'm trying downgrading.

When deployed with Carrierwave 2.2.6, I saw this error:

NoMethodError: undefined method `key' for nil:NilClass

This stack overflow says adding attr_writer :image_cache solved the issue.