- πͺπΈSpain penyaskito Seville π, Spain πͺπΈ, UTC+2 πͺπΊ
The block itself might need tests.
-
+++ b/core/modules/system/src/Plugin/Block/CronStatusBlock.php @@ -0,0 +1,159 @@ + * Provides a block to display 'Site branding' elements.
Fix comment.
-
+++ b/core/modules/system/src/Plugin/Block/CronStatusBlock.php @@ -0,0 +1,159 @@ + * forms = { + * "settings_tray" = "Drupal\system\Form\SystemBrandingOffCanvasForm", + * },
This needs to be removed.
-
+++ b/core/modules/system/src/Plugin/Block/CronStatusBlock.php @@ -0,0 +1,159 @@ + $cron_last = \Drupal::state()->get('system.cron_last'); ... + $cron_last = \Drupal::state()->get('install_time', 0);
Inject state?
-
+++ b/core/modules/system/src/Plugin/Block/CronStatusBlock.php @@ -0,0 +1,159 @@ + $request_time = \Drupal::time()->getRequestTime();
Inject time()?
-
+++ b/core/modules/system/src/Plugin/Block/CronStatusBlock.php @@ -0,0 +1,159 @@ + $summary = t('Last run @time ago', ['@time' => \Drupal::service('date.formatter')->formatTimeDiffSince($cron_last)]);
Inject date.formatter?
-
+++ b/core/modules/system/src/Plugin/Block/CronStatusBlock.php @@ -0,0 +1,159 @@ + $x['cron'] = [
This needs to be $data, or to be removed.
-
+++ b/core/modules/system/src/Plugin/Block/CronStatusBlock.php @@ -0,0 +1,159 @@ + '#title' => t('more information'), ... + '#title' => t('Run cron'),
We need StringTranslationTrait and $this->t()
-
+++ b/core/modules/system/src/Plugin/Block/CronStatusBlock.php @@ -0,0 +1,159 @@ + return Cache::mergeTags( + parent::getCacheTags(), + $this->configFactory->get('system.cron')->getCacheTags() + );
We need CacheableMetadata->createFrom() to ensure we don't leave anything behind.
-
- πͺπΈSpain penyaskito Seville π, Spain πͺπΈ, UTC+2 πͺπΊ
Discussed with @lauriii. Splitting the big status report template into smaller pieces it's useful no matter what happens with dashboards. They eventually might become sdc components. So it makes sense to propose that change in core already.
- πͺπΈSpain penyaskito Seville π, Spain πͺπΈ, UTC+2 πͺπΊ
On second thoughts, we agreed that it's not that important atm, and we can avoid that for now. Probably by the time we plan to integrate this that might have been converted to SDC already.
- Merge request !6552Issue #3327827: Provide a block for running cron from a dashboard β (Open) created by plopesc
- Status changed to Needs review
10 months ago 3:48pm 12 February 2024 - Status changed to RTBC
10 months ago 11:39pm 12 February 2024 - πΊπΈUnited States smustgrave
Tested this out by manually verifying the block appears in the block layout list
Placed the block at the top of the content region.
Verified it worked as expectCR reads well.
- π¬π§United Kingdom catch
Double checked in CronForm whether it includes the cron status/last stufff, and it does. This looks good to me.
- Status changed to Needs work
9 months ago 9:01am 29 February 2024 - Status changed to Needs review
9 months ago 3:34pm 29 February 2024 - πͺπΈSpain plopesc Valladolid
Thank you for your review @alexpott.
Addressed the comments MR.
Moving it back to Needs Review.
- Status changed to RTBC
9 months ago 2:35pm 1 March 2024 - π¬π§United Kingdom catch
π± Enhance user experience with customizable dashboards Active is still waiting for product manager approval, and this block will only make sense with that, so tagging for product managers.
- Status changed to Postponed
9 months ago 2:45pm 1 March 2024