- Merge request !15Add new event that we can subscribe after batch is complete. → (Open) created by mohit_aghera
- 🇮🇳India arunkumark Coimbatore
Verified patch applying manually working. Need to re-roll the patch to work with the latest version.
- last update
about 1 year ago 33 pass, 4 fail - Status changed to Needs review
about 1 year ago 5:15pm 5 December 2023 - 🇮🇳India arunkumark Coimbatore
Re-rolled the patch and additionally added Event start subscriber.
- 🇺🇸United States DamienMcKenna NH, USA
This needs to be updated to apply to the 6.0.x branch.
- First commit to issue fork.
- Merge request !68Issue #3269373 by joegraduate, mohit_aghera, arunkumark: Implement new event for batch complete action → (Open) created by joegraduate
- 🇺🇸United States joegraduate Arizona, USA
Created a new MR based on 6.0.x because I couldn't change the target branch of the existing MR.
- 🇺🇸United States joegraduate Arizona, USA
For the event that is dispatched when a batch finishes, I wonder if it would be better to dispatch the event outside of the check to see if the batch was successful or not since I imagine event subscribers would want to be able to respond to the batch finishing even if it wasn't successful. It also seems to me that it'd be more useful to only dispatch the event once for the entire batch and include the full results in the event rather than dispatch one event for each of the items in the results array.