Problem/Motivation

When you enable the module, the method hook_help is not implemented.

Steps to reproduce

Download and activate the module and visit its help page.

Proposed resolution

Implement the hook_help and describe the module's functionality and usage.

CommentFileSizeAuthor
#4 code.png147.17 KBraveen_thakur51
#4 after-patch.png15.77 KBraveen_thakur51
#4 before-patch.png14.34 KBraveen_thakur51

Issue fork video_filter-3360305

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

vimal_nadar created an issue. See original summary.

vimal_nadar opened merge request !6

vimal_nadar’s picture

vimal_nadar commented
Assigned: vimal_nadar » Unassigned
Status: Active » Needs review

Created MR. Please review it.

raveen_thakur51’s picture

raveen_thakur51 commented
StatusFileSize
new before-patch.png14.34 KB
new after-patch.png15.77 KB
new code.png147.17 KB

@vimal,
I have reviewed the MR! and applied the code changes on to my website. The hook_help was implemented properly.
I am using Drupal version - 9.5, and PHP version - 8.1. Composer - 2.6.
The hook_help was implemented properly. I am attaching my testing screenshots for reference.
Please review & Thank You.
Can be moved to RTBC!

sourabhjain’s picture

sourabhjain
Primary language English
commented
Status: Needs review » Reviewed & tested by the community

RTBC +1

DamienMcKenna made their first commit to this issue’s fork.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Parent issue: » #3358722: Plan for Video Filter 8.x-1.3

Thank you for putting this together.

FYI screenshots of code in a text editor aren't really useful, but the screenshots of the module's info from the Extend admin page was helpful in confirming the changes.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Reviewed & tested by the community » Fixed

Committed.

Status: Fixed » Closed (fixed)

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