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.
Background:
This issue is part of the task to update the hook_help texts of the Drupal 8 modules:
#1908570: [meta] Update or create hook_help() texts for D8 core modules
Tasks:
- review / write the hook_help text according to help guidelines
Comment | File | Size | Author |
---|---|---|---|
#7 | interdiff.txt | 802 bytes | batigolix |
#7 | update-hook-help-forum-2091455-7.patch | 3.59 KB | batigolix |
#6 | interdiff-2091455-2-6.txt | 2.1 KB | InternetDevels |
#6 | update-hook-help-forum-2091455-6.patch | 3.58 KB | InternetDevels |
Comments
Comment #1
berkas1 CreditAttribution: berkas1 commentedTaking it - Drupalcon Prague
Comment #2
berkas1 CreditAttribution: berkas1 commentedComment #3
petrpo CreditAttribution: petrpo commentedManualy patched module forum and all links are working right.
Comment #4
jhodgdonThanks for the patch and the testing!
Looking over this help as a whole, it doesn't really follow our guidelines on http://drupal.org/node/632280 , so the help for this module needs some more work.
a) The "For more information..." line is not our standard wording/format.
b) The Uses topics should all have headers with -ing verbs.
Other than that, I think it looks good.
Comment #5
berkas1 CreditAttribution: berkas1 commentedThanks, I'll send patch ASAP
Comment #6
InternetDevels CreditAttribution: InternetDevels commentedAdded patch.
Comment #7
batigolixI verified the patch in #6:
- Verify that all the links work --> Okay
- Verify that all mentions of pages/text within the UI match what is seen in the UI --> Okay
- Verify that the formatting is OK. --> Okay
I just found the link to d.o should be https. Attached patch fixes that
Comment #8
batigolixComment #9
jhodgdonLooks like this one is ready to commit! We're in the middle of a "only commit critical and major issues" week, so I'll get it committed on the 23rd or so.
Comment #10
jhodgdonI'm being extra careful about commit conflicts lately, and there is an issue tagged "avoid commit conflicts" that touches this file. So I plan to wait to commit this until
#1996238: Replace hook_library_info() by *.libraries.yml file
is resolved. Sorry for the delay!
Comment #11
alexpott#1996238: Replace hook_library_info() by *.libraries.yml file is postponed. Committed d78587e and pushed to 8.x. Thanks!