- Issue created by @andypost
- Status changed to Needs review
over 1 year ago 12:51am 28 July 2023 - last update
over 1 year ago 29,885 pass - Assigned to Amber Himes Matz
- last update
over 1 year ago 29,885 pass - π«π·France andypost
There's 2 more places to update in documentation (probably) or keep it as is
I fixed second mention to look for examples in the help module's topicscore/modules/help/help.module:39: $output .= '<p>' . t('Help topics provided by modules and themes are also part of the Help module. If the core Search module is enabled, these topics are searchable. For more information, see the <a href=":online">online documentation for Help Topics</a>.', [':online' => 'https://www.drupal.org/documentation/modules/help_topics']) . '</p>'; core/modules/help/help.module:49: $output .= '<dd>' . t("Modules and themes can provide help topics as Twig-file-based plugins in a project sub-directory called <em>help_topics</em>; plugin meta-data is provided in YAML front matter within each Twig file. Plugin-based help topics provided by modules and themes will automatically be updated when a module or theme is updated. Use the plugins in <em>core/modules/help_topics/help_topics</em> as a guide when writing and formatting a help topic plugin for your theme or module.") . '</dd>';
the first link redirects to https://www.drupal.org/docs/develop/managing-a-drupalorg-theme-module-or... β
- π«π·France andypost
@Amber the link to https://www.drupal.org/documentation/modules/help_topics β is primary question of replacement
- Status changed to Needs work
over 1 year ago 4:23pm 26 August 2023 - πΊπΈUnited States smustgrave
Seen this one laying around so thought I'd take a look.
Should the patches #2 and #3 be combined?
- Issue was unassigned.
- Status changed to Needs review
over 1 year ago 10:01pm 31 August 2023 - last update
over 1 year ago 30,128 pass, 4 fail - πΊπΈUnited States Amber Himes Matz Portland, OR USA
Alright, here's a patch that adds to the one in #3, updating the URL and link text to the Help Topic Standards docs page (without the redirect).
- π«π·France andypost
@Amber thank you! please add patch from #2 to final one
- last update
over 1 year ago 30,130 pass - πΊπΈUnited States Amber Himes Matz Portland, OR USA
Thanks for the clarification, @andypost. This patch adds the patch from #2 (the changes to core/modules/help/tests/src/Kernel/HelpSearchPluginTest.php). Interdiff included.
Ok, I think this is ready for review!
The last submitted patch, 8: 3377604-8.patch, failed testing. View results β
- Status changed to RTBC
over 1 year ago 11:22pm 31 August 2023 - last update
over 1 year ago 30,135 pass - last update
over 1 year ago 30,136 pass 38:10 36:55 Running- last update
over 1 year ago 30,146 pass - last update
over 1 year ago 30,146 pass - last update
over 1 year ago 30,150 pass -
quietone β
committed d6f9f1d4 on 11.x
Issue #3377604 by Amber Himes Matz, andypost, smustgrave: Remove...
-
quietone β
committed d6f9f1d4 on 11.x
- π³πΏNew Zealand quietone
I applied the patch and searched core for "\help_topics" outside of the help_topics module. Only references to test modules that have names beginning with that string are found.
$ grep -ri "\\\help_topics" core | grep -v modules/help_topics/ | grep help_topics core/modules/help/tests/src/Functional/HelpTopicSearchTest.php: // \Drupal\help_topics_test\Plugin\HelpSection\TestHelpSection. core/modules/help/tests/src/Functional/HelpTopicsSyntaxTest.php:use Drupal\help_topics_twig_tester\HelpTestTwigNodeVisitor; core/modules/help/tests/modules/help_topics_test/src/Plugin/HelpTopic/TestHelpTopicPlugin.php:namespace Drupal\help_topics_test\Plugin\HelpTopic; core/modules/help/tests/modules/help_topics_test/src/Plugin/HelpSection/TestHelpSection.php:namespace Drupal\help_topics_test\Plugin\HelpSection; core/modules/help/tests/modules/help_topics_test/src/Plugin/Deriver/TestHelpTopicDeriver.php:namespace Drupal\help_topics_test\Plugin\Deriver; core/modules/help/tests/modules/help_topics_test/src/Plugin/Deriver/TestHelpTopicDeriver.php: 'class' => 'Drupal\\help_topics_test\\Plugin\\HelpTopic\\TestHelpTopicPlugin', core/modules/help/tests/modules/help_topics_test/src/Controller/HelpTopicsTestController.php:namespace Drupal\help_topics_test\Controller; core/modules/help/tests/modules/help_topics_test/help_topics_test.help_topics.yml: class: 'Drupal\help_topics_test\Plugin\HelpTopic\TestHelpTopicPlugin' core/modules/help/tests/modules/help_topics_test/help_topics_test.help_topics.yml: deriver: 'Drupal\help_topics_test\Plugin\Deriver\TestHelpTopicDeriver' core/modules/help/tests/modules/help_topics_test/help_topics_test.routing.yml: _controller: '\Drupal\help_topics_test\Controller\HelpTopicsTestController::testPage' core/modules/help/tests/modules/help_topics_twig_tester/help_topics_twig_tester.services.yml: class: Drupal\help_topics_twig_tester\HelpTestTwigExtension core/modules/help/tests/modules/help_topics_twig_tester/src/HelpTestTwigExtension.php:namespace Drupal\help_topics_twig_tester; core/modules/help/tests/modules/help_topics_twig_tester/src/HelpTestTwigNodeVisitor.php:namespace Drupal\help_topics_twig_tester;
- Status changed to Fixed
over 1 year ago 4:30am 15 September 2023 Automatically closed - issue fixed for 2 weeks with no activity.