- π¬π§United Kingdom catch
It's been more than a few days, so I'm going to go ahead and remove the tag. We have warnings for deprecated modules, so if an experimental module ends up on its way out of core, people will be notified that way. The info level item is still a reminder you're running something that's not fully stable yet.
- πΊπΈUnited States xjm
Let's bring it up in the RM channel so that folks have a chance to discuss and agree.
- π¬π§United Kingdom catch
Could use screenshots of the info-level message in the status report. The test coverage asserts that the warning doesn't appear, but should that change to a positive assertion that an info appears?
- πΊπΈUnited States xjm
Via @quietone:
+++ b/core/modules/system/system.install @@ -98,7 +98,7 @@ function system_requirements($phase) { $requirements['experimental_modules'] = [ 'title' => t('Experimental modules enabled'), 'value' => t('Experimental modules found: %module_list. <a href=":url">Experimental modules</a> are provided for testing purposes only. Use at your own risk.', ['%module_list' => implode(', ', $experimental_modules), ':url' => 'https://www.drupal.org/core/experimental']), - 'severity' => REQUIREMENT_WARNING, + 'severity' => REQUIREMENT_INFO, ]; } // Warn if any deprecated modules are installed. @@ -147,7 +147,7 @@ function system_requirements($phase) { @@ -147,7 +147,7 @@ function system_requirements($phase) { $requirements['experimental_themes'] = [ 'title' => t('Experimental themes enabled'), 'value' => t('Experimental themes found: %theme_list. Experimental themes are provided for testing purposes only. Use at your own risk.', ['%theme_list' => implode(', ', $experimental_themes)]), - 'severity' => REQUIREMENT_WARNING, + 'severity' => REQUIREMENT_INFO, ];
This patch is going too far. The scope should be limited to runtime warnings being turned to info mesages. We should still be warning about them during installation/update.
I am also not sure the warning should be removed from the help pages.
- Status changed to Needs work
almost 2 years ago 3:17pm 10 February 2023 - π³πΏNew Zealand quietone
This was discussed by the release managers yesterday. We agree with the direction in #60 and I have changed the Issue Summary to reflect that.
Also, the proposed resolution includes, "Add a better install notification/confirmation when installing an experimental module via the UI. This has basic support, but a handbook link or link to the status of experimental modules would be helpful." The current message does have a link to
Experimental modules and themes in Drupal core β . I think that covers this point, or am I missing something?Removing the tag.