- πΈπ°Slovakia poker10
Thanks for working on this.
Why are we setting
sent_time
in case the notification is not being sent?+ if (!$checker->getIsInStock($product, $locations)) { + $time = new DrupalDateTime(); + $data->set('sent_time', $time->getTimestamp())->save(); + return; + }
I think the correct behavior would be to just return, to remove the item from queue, as it is not valid anymore.
-
poker10 β
committed 5d06ab1d on 8.x-1.x
Issue #3252466: Check in queue if still in stock
-
poker10 β
committed 5d06ab1d on 8.x-1.x
- πΈπ°Slovakia poker10
I converted the patch from #3 to MR and implemented the suggestion from #4. Committed and pushed this. Thanks!