Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
The link (https://www.drupal.org/modules/help_topics) provided in the help_topics
module in hook_help()
is not available in drupal.org.
Proposed resolution
Point link to https://www.drupal.org/documentation/modules/help_topics
Remaining tasks
commit
User interface changes
link points to stub page for the module
API changes
no
Data model changes
no
Release notes snippet
n/a
Comment | File | Size | Author |
---|---|---|---|
#6 | 3113236.patch | 1.43 KB | jhodgdon |
Screenshot 2020-02-13 at 2.25.12 AM.png | 358.8 KB | rksyravi |
Comments
Comment #2
xjmThanks @rksyravi! The Help Topics module is experimental, so its documentation might not be completed until it is closer to stable. That said, we should probably at least create a stub page or check that the URL is correct.
Comment #3
xjmComment #4
jhodgdonDoes it actually make sense to make that page? The plan is for the Help Topics module to be merged into the Help module, so we do not really want that documentation to exist when the module is stable. We can make a stub page for now but we would want it to go away...
Comment #5
jhodgdonWe could also put in a redirect from that URL to the modules/help docs page, and a note at the top of that page about Help Topics beta?
Comment #6
jhodgdonOK, there are actually two problems here.
First: The link in the hook_help for help topics module should be https://www.drupal.org/documentation/modules/help_topics, not https://www.drupal.org/modules/help_topics as it is now.
Second: the correct link didn't go anywhere... I think it should probably be redirected to
https://www.drupal.org/docs/develop/documenting-your-project/help-topic-...
which is a docs page about the new help topics system. So, I've put that redirect in.
We also need a patch to fix the link in help_topics.module, so here is that.
Comment #7
daffie CreditAttribution: daffie commentedLooks good to me.
Comment #8
andypost+1 rtbc, https://www.drupal.org/documentation/modules/help_topics
Fixed IS
Comment #10
jhodgdonUnrelated test failure. I assume this one is known:
Drupal\KernelTests\Core\Action\SaveActionTest::testGetDerivativeDefinitionsassertArraySubset() is deprecated and will be removed in PHPUnit 9.
/var/www/html/vendor/phpunit/phpunit/src/Framework/TestResult.php:691
Comment #11
alexpottCommitted and pushed 63cdee3511 to 9.0.x and c179a28bb0 to 8.9.x and 6e9b99da12 to 8.8.x. Thanks!
Comment #17
quietone CreditAttribution: quietone at PreviousNext commentedClosed a duplicate, adding credit here.