godaddy-wordpress / godaddy-wordpress/wc-plugin-framework

Remove the background job handler dependency from the batch handler

Open
#268 0 comments 0 reactions 0 assignees View on GitHub
Background Jobs Feature Refactor
Dominant language
PHP
Stars
146
Forks
43
Avg merge
21m
Merged PRs (30d)
3

Description

Right now our `SV_WP_Job_Batch_Handler` utility relies on the background/async job handler by simply processing chunks of a background job in batches. This works very well for plugins that support both types of job processing and those with a UI for listing ongoing jobs (so batches can be resumed, paused, etc...), but it can be overkill for plugins that don't really need background processing at all.

We should make the underlying [job handler optional](https://github.com/skyverge/wc-plugin-framework/blob/master/woocommerce/utilities/class-sv-wp-job-batch-handler.php#L60) so that plugins don't have to worry about any of that and can just process batches until the user navigates away.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.