Implement hook_help()

Created on 22 January 2019, almost 6 years ago
Updated 29 June 2023, over 1 year ago

The module should implement hook_help().

📌 Task
Status

Fixed

Version

1.0

Component

Documentation

Created by

🇮🇳India lomasr

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • 🇮🇳India Bushra Shaikh

    I applied patch #7, patch failed got below errors:

    error: patch failed: README.txt:8
    error: README.txt: patch does not apply
    error: patch failed: node_title_help_text.module:1
    error: node_title_help_text.module: patch does not apply
    
  • Assigned to nitin_lama
  • @nitin_lama opened merge request.
  • Status changed to Needs review over 1 year ago
  • 🇮🇳India nitin_lama India

    None of the patches applies to 8.x-1.x, Created MR for 8.x-1.x. Since the version of the issue is 8.x-1.x.

  • 🇮🇳India Raveen Kumar

    Hello Guys,
    I have created a path for version - 8.x-1.x-dev. The hook_help has been implemented fine after applying the patch.
    I am adding my screenshots and patches for reference.
    It would be great if someone can test my patch.
    And Thank you.

  • 🇮🇹Italy apaderno Brescia, 🇮🇹
  • Issue was unassigned.
  • Status changed to Needs work over 1 year ago
  • 🇮🇹Italy apaderno Brescia, 🇮🇹
    +    // Main module help for the custom_page module.
    +    case 'help.page.node_title_help_text':

    That comment is using the wrong module machine name. It is also not necessary, as it is clear what the code does.

    + $output .= '<p>' . t('This module allows administrator to add help text to node title field. You can set different help text for each content type. The configuration is available at content type edit form. It helps content editors understand the title field easily') . '</p>';

    The period at the end of the sentence is missing.

  • 🇮🇳India Raveen Kumar

    The patch is updated as per #14. Please review.

  • Status changed to Needs review over 1 year ago
  • Status changed to Needs work over 1 year ago
  • 🇮🇹Italy apaderno Brescia, 🇮🇹

    + $output .= '<p>' . t('This module allows administrator to add help text to node title field. You can set different help text for each content type. The configuration is available at content type edit form. It helps content editors understand the title field easily.') . '</p>';

    Those sentences need some corrections.

    This module allows administrators to add a help text to the node title field. You can set a different help text for each content type. The configuration is available on the content type edit form. It helps content editors to easily understand the title field.

  • Status changed to Needs review over 1 year ago
  • Status changed to RTBC over 1 year ago
  • 🇮🇹Italy apaderno Brescia, 🇮🇹

    The last patch is good to go.

  • 🇮🇳India Raveen Kumar

    Thanks for confirmation @apaderno

  • Status changed to Fixed over 1 year ago
  • 🇺🇸United States vinmassaro
  • Automatically closed - issue fixed for 2 weeks with no activity.

Production build 0.71.5 2024