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.

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

Remove the background job handler dependency from the batch handler #268

Open
ChaseWiseman opened this issue Feb 27, 2018 · 0 comments
Open
Labels
Background Jobs Code concerning background jobs Feature Refactor Issues and pull requests that involve refactoring of FW components

Comments

@ChaseWiseman
Copy link
Contributor

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 so that plugins don't have to worry about any of that and can just process batches until the user navigates away.

@ChaseWiseman ChaseWiseman added Feature Refactor Issues and pull requests that involve refactoring of FW components Background Jobs Code concerning background jobs labels Feb 27, 2018
@ChaseWiseman ChaseWiseman modified the milestones: v5.2.0, v5.3.0 Feb 27, 2018
@ChaseWiseman ChaseWiseman removed this from the v5.5.0 milestone Nov 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Background Jobs Code concerning background jobs Feature Refactor Issues and pull requests that involve refactoring of FW components
Projects
None yet
Development

No branches or pull requests

1 participant