While evaluating this module we noticed it could use a hook_help implementation to make it easier for users to find helpful information about the module. Patch to follow.

CommentFileSizeAuthor
#3 after patch.png35.53 KBvolkswagenchick
#3 before patch.png22.85 KBvolkswagenchick
#2 conditional_message_hook_help-2966888-2.patch1.6 KBdani3lr0se

Comments

daniel_rose created an issue. See original summary.

dani3lr0se’s picture

dani3lr0se commented
Assigned: dani3lr0se » Unassigned
Status: Active » Needs review
StatusFileSize
new conditional_message_hook_help-2966888-2.patch1.6 KB

Here is a patch that adds a hook_help by pulling in the readme. Thanks for your work on this module. :)

volkswagenchick’s picture

volkswagenchick
she/her
Primary language English
Location San Francisco Bay Area
commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new before patch.png22.85 KB
new after patch.png35.53 KB

Tested using simplytest.me and patch applies.

The new help page pulls in the readme which definitely provides more useful information than the current help page.

Screen shots attached. marking RTBC

wranvaud’s picture

wranvaud commented
Status: Reviewed & tested by the community » Fixed

Thank you for your contributions! committed

  • daniel_rose authored 08e0f0c on 8.x-1.x 
    Issue #2966888 by daniel_rose, volkswagenchick: Missing hook_help...

Status: Fixed » Closed (fixed)

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