Problem/Motivation

The help text refers to "Drupal" where it shouldn't.

Proposed resolution

Reword "output for Drupal sites" to "output for your website". The second reference to Drupal should stay, because it refers to the modules that are part of the Drupal distribution.

Remaining tasks

User interface changes

API changes

CommentFileSizeAuthor
#3 updated-hook_help-text-2471024-3.patch2.12 KBpjbaert
#3 interdiff-updated-hook_help-text.txt2.12 KBpjbaert
#1 updated-hook_help-text-2471024-1.patch2.13 KBpjbaert

Comments

pjbaert’s picture

pjbaert
Primary language Dutch
commented
Status: Active » Needs review
StatusFileSize
new updated-hook_help-text-2471024-1.patch2.13 KB

I updated the help text with the proposed solution.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Needs review » Needs work

Actually, I think we should just say "the core modules" and "the core themes" and omit the word Drupal there too.

Thanks!

pjbaert’s picture

pjbaert
Primary language Dutch
commented
Status: Needs work » Needs review
StatusFileSize
new interdiff-updated-hook_help-text.txt2.12 KB
new updated-hook_help-text-2471024-3.patch2.12 KB

I updated the patch with the remarks mentioned by @jhodgdon in #2

ifrik’s picture

ifrik
she/her
commented
Status: Needs review » Reviewed & tested by the community

Thanks pjbart,

the text is changed and does not refer to "Drupal" anymore, but generically to core or the website.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.0.x. Thanks!

  • webchick committed 33f0f0f on 8.0.x 
    Issue #2471024 by pjbaert, ifrik: Update hook_help text for RDF module

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.