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
#5 3067345-5.patch1.29 KBvuil
#3 Group Bulk OperationsAfterpatch-1.png27.66 KBkrina.addweb
#3 Group Bulk Operations-after patch2.png102.47 KBkrina.addweb
#3 Group Bulk Operations-before patch.png25.11 KBkrina.addweb
#2 group_bulk_operations-hook_help-3067345-2.patch1.53 KBalonaoneill
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

alonaoneill created an issue. See original summary.

alonaoneill’s picture

alonaoneill CreditAttribution: alonaoneill at Hook 42 commented
Assigned: alonaoneill » Unassigned
Status: Active » Needs review
FileSize
group_bulk_operations-hook_help-3067345-2.patch1.53 KB
krina.addweb’s picture

krina.addweb CreditAttribution: krina.addweb at AddWeb Solution Pvt. Ltd. commented
FileSize
Group Bulk Operations-before patch.png25.11 KB
Group Bulk Operations-after patch2.png102.47 KB
Group Bulk OperationsAfterpatch-1.png27.66 KB

@alonaoneill, Thanks for the patch. Your patch is working fine as I tested the mentioned changes using simplytest.me. PFA

krina.addweb’s picture

krina.addweb CreditAttribution: krina.addweb at AddWeb Solution Pvt. Ltd. commented
Status: Needs review » Reviewed & tested by the community
vuil’s picture

vuil
Location Bulgaria 🇧🇬 🇪🇺 🌍
CreditAttribution: vuil at trio-group I.AM commented
Status: Reviewed & tested by the community » Active
FileSize
3067345-5.patch1.29 KB
vuil’s picture

vuil
Location Bulgaria 🇧🇬 🇪🇺 🌍
CreditAttribution: vuil at trio-group I.AM commented
Status: Active » Needs review
alonaoneill’s picture

alonaoneill CreditAttribution: alonaoneill at Hook 42 commented
Status: Needs review » Reviewed & tested by the community

This issue was already RTBCed.
Can you please explain why you changed it to Needs review and uploaded different patch?
Changing it back to RTBC

wangshy’s picture

wangshy CreditAttribution: wangshy at Northern Commerce commented
Status: Reviewed & tested by the community » Fixed

Thanks, fixed on 1.x-dev

Status: Fixed » Closed (fixed)

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

vuil’s picture

vuil
Location Bulgaria 🇧🇬 🇪🇺 🌍
CreditAttribution: vuil at Bulpros Consulting commented