- Issue created by @Kamlesh Kishor Jha
- @kamlesh-kishor-jha opened merge request.
- Issue was unassigned.
- Status changed to Needs review
almost 2 years ago 10:29am 4 February 2023 - 🇮🇳India Kamlesh Kishor Jha Jaipur
Created mr fot this issue.
Please review.
Thanks. - Status changed to Needs work
over 1 year ago 8:32am 2 June 2023 Applied the MR1 on 9.5.8, and on applying the patch it gave the error:
error: patch failed: menu_link_sync.module:8
error: menu_link_sync.module: patch does not apply- 🇮🇳India nitin_lama India
Unable to replicate comment #4. Attaching ss for reference. MR changes are correct. Please confirm.
- Status changed to Needs review
over 1 year ago 9:18am 2 June 2023 - Status changed to Needs work
over 1 year ago 9:43am 2 June 2023 - 🇵🇭Philippines clarkssquared
Hi
I applied MR !2 to my local with Drupal 9.5.9, I cannot also replicate comment #4 and the MR applied successfully on my end. But, my concern is if possible we can add more information to the help page like a quick help on how to configure the module or such. So that the help page will become reliable to the users, only if possible.
Please look at the screenshots attached for reference.
Thank you
- 🇮🇳India Raveen Kumar
I have reviewed & implemented the MR! And the hook_help has been implemented successfully.
As you can see in my attached screenshots.
Can be moved to RTBC!
Please & thank you. - Status changed to RTBC
over 1 year ago 7:31am 3 June 2023 - First commit to issue fork.
-
rutiolma →
committed 0f61ec8b on 8.x-1.x authored by
Kamlesh Kishor Jha →
Issue #3339168 by Kamlesh Kishor Jha: Implement hook_help()
-
rutiolma →
committed 0f61ec8b on 8.x-1.x authored by
Kamlesh Kishor Jha →
Automatically closed - issue fixed for 2 weeks with no activity.