Per the Module documentation guidelines, a module should Provide help text in the Drupal UI. This module is missing an implementation of hook_help().
Documentation:
https://www.drupal.org/docs/develop/documenting-your-project/module-docu...
https://www.drupal.org/node/632280
Patch to follow, thanks!

CommentFileSizeAuthor
#4 interdiff_2-4.txt1.45 KBzrpnr
#4 jquery_ui_draggable-hook_help-3071214-4.patch1.22 KBzrpnr
#3 Screenshot from 2019-08-01 15-11-45.png56.6 KBWilltg
#2 jquery_ui_draggable-hook_help-3071214-2.patch1.08 KBalonaoneill

Comments

alonaoneill created an issue. See original summary.

alonaoneill’s picture

alonaoneill commented
Assigned: alonaoneill » Unassigned
Status: Active » Needs review
StatusFileSize
new jquery_ui_draggable-hook_help-3071214-2.patch1.08 KB
Willtg’s picture

Willtg commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new Screenshot from 2019-08-01 15-11-45.png56.6 KB

Code looks good, patch tested with STM and it worked well. Adding a screenshot below for confirmation.
Marking RTBC
Thank you for working on the module!

zrpnr’s picture

zrpnr
he/him
Primary language English
Location UTC-7
commented
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new jquery_ui_draggable-hook_help-3071214-4.patch1.22 KB
new interdiff_2-4.txt1.45 KB

Added the link to the change record and made small edits to line endings and to remove whitespace.

bnjmnm’s picture

bnjmnm
he/him
Primary language English
Location Ann Arbor, MI
commented
Status: Needs review » Reviewed & tested by the community

Reviewed the changes in #4 this is RTBC.

  • zrpnr authored c6a5e5e on 8.x-1.x 
    Issue #3071214 by zrpnr, alonaoneill, Willtg, bnjmnm: Hook Help is...
bnjmnm’s picture

bnjmnm
he/him
Primary language English
Location Ann Arbor, MI
commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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