Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

DEPRECATION WARNING: default_worker_options is deprecated and will be removed from Sidekiq 7.0 (use default_job_options instead) (called from notify_agents at /Users/hackeron/Development/Tether/timeline/app/models/user.rb:303) #705

Closed
rgaufman opened this issue Apr 19, 2022 · 3 comments · Fixed by #707
Closed

DEPRECATION WARNING: default_worker_options is deprecated and will be removed from Sidekiq 7.0 (use default_job_options instead) (called from notify_agents at /Users/hackeron/Development/Tether/timeline/app/models/user.rb:303) #705

rgaufman opened this issue Apr 19, 2022 · 3 comments · Fixed by #707

Comments

@rgaufman
Copy link

Latest version throws this deprecation warning:

DEPRECATION WARNING: default_worker_options is deprecated and will be removed from Sidekiq 7.0 (use default_job_options instead)
@mhenrixon
Copy link
Owner

mhenrixon commented Apr 22, 2022 •
edited
Loading

Yes, it’s coming from sidekiq and shouldn’t be a problem I think but I’ll look into it.

i already prepared a few things for Sidekiq 7.

@mhenrixon mhenrixon mentioned this issue Apr 23, 2022
Merged
@mhenrixon
Copy link
Owner

If someone could try out the master branch that would be brilliant.

I did a bunch of renaming of worker to job. Seems that's the way sidekiq is heading.

@rgaufman
Copy link
Author

Tried master and the deprecation warnings are gone, so far so good! - Will report back if I notice any issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Prepare for Sidekiq v7 mhenrixon/sidekiq-unique-jobs
2 participants
@mhenrixon @rgaufman