search

hotwired / turbo-rails

Use Turbo in your Ruby on Rails app
https://turbo.hotwired.dev
MIT License
2.13k stars 329 forks source link

Restore support for `broadcast_refreshes` with Sidekiq's default `strict_args` #683

Closed AnalyzePlatypus closed 2 months ago

AnalyzePlatypus commented 2 months ago

Restores support for broadcast_refreshes when using Sidekiq in the default strict_args mode.

Temporary Workaround: Until merge, this issue can be bypassed by disabling strict_args.

Original solution suggested by @jdelStrother here. Based on PR 642 by @michaelbaudino.

Fixes #522, fixes #535. See discussion on PR 642 with @michaelbaudino and @dhh

AnalyzePlatypus commented 2 months ago

Should I add a test for this issue to the suite? Or is that not necessary?