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

Background page publishing #2067

Merged
merged 2 commits into from
Apr 13, 2021
Merged

Conversation

tvdeyen
Copy link
Member

@tvdeyen tvdeyen commented Apr 13, 2021

What is this pull request for?

This will allow to background page publishing.
It will run in parallel if no background job processor
has been defined in the host app.

Apps using sidekiq or delayed job will benefit from
running this in the background, making large pages
publish in the pbackground.

Checklist

  • I have followed Pull Request guidelines
  • I have added a detailed description into each commit message
  • I have added tests to cover this change

tvdeyen added 2 commits April 13, 2021 13:19
This will help us adding background jobs
This will allow to background page publishing.
It will run in parallel if no background job processor
has been defined in the host app.

Apps using sidekiq or delayed job will benefit from
running this in the background, making large pages
publish in the pbackground.
@tvdeyen tvdeyen requested a review from mamhoff April 13, 2021 11:22
@tvdeyen tvdeyen added this to the 6.0 milestone Apr 13, 2021
@tvdeyen tvdeyen merged commit 6f6f477 into AlchemyCMS:main Apr 13, 2021
@tvdeyen tvdeyen deleted the background-puage-publish branch April 13, 2021 12:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants