- Issue created by @shubham rathore
- @shubham-rathore-0 opened merge request.
- Issue was unassigned.
- Status changed to Needs review
almost 2 years ago 9:29am 7 February 2023 - 🇮🇳India shubham rathore Jaipur
Created MR for the issue.
Please review.
Thanks - Assigned to Charchil Khandelwal
- Issue was unassigned.
- Status changed to RTBC
almost 2 years ago 1:52pm 7 February 2023 - 🇮🇳India Charchil Khandelwal
MR looks good to me.
So moving it to RTBC.Thanks.
- Status changed to Needs work
almost 2 years ago 2:47pm 24 February 2023 - 🇺🇸United States DamienMcKenna NH, USA
Thank you for working on this.
Two things.
First off - the problem is not that hook_help() is missing, it just hasn't been added. So it's more correct to indicate that it needs to be added.
Secondly, hook_help should be used for more than simply adding the description from the info file, which is also available from the modules page; it should instead be used to provide documentation, which is why a lot of people just render the README.md file.
Setting this to "needs work" as the documentation in hook_help() needs improvements.
- First commit to issue fork.
- Assigned to dharmeshbarot
- Status changed to Needs review
almost 2 years ago 10:05am 27 February 2023 - 🇮🇳India dharmeshbarot Gujarat
I made changes in the .module file and created a .help.yml file. need review!
- Status changed to RTBC
over 1 year ago 3:21pm 24 March 2023 - 🇵🇭Philippines clarkssquared
Hi Shubham Rathore,
I applied the MR d3eeed9a and 90f41d96 in my local with drupal core version 9.5.6 and confirmed that the help page documentation improved.
Please look at the screen shot attached for your reference
Thank you.
- Open on Drupal.org →Core: 10.0.7 + Environment: PHP 7.4 & MySQL 5.7last update
over 1 year ago Not currently mergeable. - @damienmckenna opened merge request.
- last update
over 1 year ago Composer require failure - 🇺🇸United States DamienMcKenna NH, USA
There were some coding standards mistakes in the merge request. Also, I haven't seen anything about a *.help.yml file, so for now am excluding that. I've also added a link to the service provider.
- Issue was unassigned.
- Status changed to Fixed
over 1 year ago 7:47pm 5 August 2023 -
DamienMcKenna →
committed b80b0b45 on 8.x-1.x
Issue #3339865 by Shubham Rathore, dharmeshbarot, DamienMcKenna,...
-
DamienMcKenna →
committed b80b0b45 on 8.x-1.x
Automatically closed - issue fixed for 2 weeks with no activity.