breamware / sidekiq-batch

Sidekiq Batch Jobs Implementation
MIT License
357 stars 95 forks source link

Callback success fired too soon with ActionMailer #67

Closed KalaTheMother closed 1 year ago

KalaTheMother commented 1 year ago

Hi,

The success callback is called too soon due to ActionMailer. (class ApplicationMailer < ActionMailer::Base)

I have a complex workflow with six steps (to start). And when a mail is sent by ActionMailer in the middle of step 2, it fires success callback even the step 2 is not finished.

So, the step 3 begins whereas the step 2 is still in progress.

When I disable the sent of mail, the workflow is followed perfectly.

aleksclark commented 1 year ago

@KalaTheMother I believe #69 will actually fix this behavior

github-actions[bot] commented 1 year ago

Stale issue message