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

Proposal: Detect if CRON is disabled #1472

Closed
arunbasillal opened this issue Jan 10, 2019 · 6 comments
Closed

Proposal: Detect if CRON is disabled #1472

arunbasillal opened this issue Jan 10, 2019 · 6 comments
Labels
Module: dashboard type: enhancement Improvements that slightly enhance existing functionality and are fast to implement
Milestone

Comments

@arunbasillal
Copy link
Contributor

arunbasillal commented Jan 10, 2019

WP Rocket needs CRON for multiple operations. Critical Path CSS generation, Preload, Database optimization etc.

WP CRON being disabled is a common issue for why these fails. We could try and detect if WP Cron is disabled.

When to detect?

It doesn't make sense to detect unless the user is trying to use a feature that needs WP Cron. Check for WP Cron when user enables Preload, Critical Path CSS generation, or database optimization

Thoughts and notes

  • The website might have a server CRON setup, so if we display a notice to the customer and should be dismissible.
  • During the check, see if the counter for the preload, or Critical Path CSS generation is already increased. In that case, even if WP CRON is disabled, there is server cron hopefully and the notice is no longer needed.
@webtrainingwheels
Copy link

+10000

@Tabrisrp Tabrisrp added the type: enhancement Improvements that slightly enhance existing functionality and are fast to implement label Jan 11, 2019
@Tabrisrp Tabrisrp added this to the 3.2.6 milestone Feb 8, 2019
@Tabrisrp Tabrisrp removed this from the 3.2.7 milestone Feb 26, 2019
@arunbasillal
Copy link
Contributor Author

@Tabrisrp What design changes does this need? My original suggestion was for an admin notice like we have for incompatible plugins.

@Tabrisrp Tabrisrp added wontfix Indicates that work won't continue on an issue or pull request and removed needs: copywriting needs: design type: enhancement Improvements that slightly enhance existing functionality and are fast to implement labels Apr 18, 2019
@Tabrisrp
Copy link
Contributor

Good news is, it's automatically included in WP 5.2 Site Health Check tool, so we don't have to develop our own

@arunbasillal
Copy link
Contributor Author

I still think showing an admin notice would be more effective. Customers are going to enable Preload / Critical Path CSS and say it's not working and write to us. We could piggyback on 5.2's features to do this hopefully. Lets wait till 5.2 is actually here.

@GeekPress
Copy link
Contributor

I'm kindly re-open this issue. A notice to warn our users or grey out our options will be helpful

@GeekPress GeekPress reopened this May 28, 2019
@Tabrisrp Tabrisrp added this to the 3.3.7 milestone Aug 12, 2019
@Tabrisrp Tabrisrp added Module: dashboard type: enhancement Improvements that slightly enhance existing functionality and are fast to implement and removed wontfix Indicates that work won't continue on an issue or pull request labels Aug 12, 2019
@Tabrisrp Tabrisrp self-assigned this Aug 12, 2019
@arunbasillal arunbasillal reopened this Sep 3, 2019
@arunbasillal
Copy link
Contributor Author

Re-opening this issue as discussed in the QA channel.

With the 3.3.7 update, here is what we did:

  • An admin notice is displayed if a scheduled event missed it's schedule time.
  • This will accommodate for WordPress installations that use server cron as well.
  • The notice displayed on WP Rocket dashboard can be dismissed and will not come back once dismissed.
  • This is basically what WordPress health check tool uses as well

False positives

At the time of writing, we have 9 false positives reported where customers saw this notice even when cron was working correctly. This could be because of an edge case.

  • Say an event is scheduled for 15:02
  • Server cron is set to run every 5 minutes.
  • It runs at 15:00
  • Customer visits the dashboard at 15:03 and the notice will be displayed because the even scheduled for 15:02 is missed and will only be done at 15:05 when the server cron runs again.

Next Steps

We will continue to monitor tickets about this and make a decision about whether to make changes in a future update.

Related conversations:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Module: dashboard type: enhancement Improvements that slightly enhance existing functionality and are fast to implement
Projects
None yet
Development

No branches or pull requests

4 participants