- Issue created by @noorulshameera
- Status changed to Needs review
almost 2 years ago 12:27pm 30 January 2023 - ๐ฎ๐ณIndia noorulshameera
Adding patch to implement hook_help() function.
- ๐ฎ๐ณIndia __i.am.prashant__
Hi @noorulshameera,
patch on comment #2 works fine for me on drupal 9.5. - ๐ฎ๐ณIndia Charchil Khandelwal
Charchil Khandelwal โ made their first commit to this issueโs fork.
- @charchil-khandelwal opened merge request.
- ๐ฎ๐ณIndia Charchil Khandelwal
Created MR for this issue.
Please review. - Status changed to Needs work
over 1 year ago 5:31am 2 March 2023 - ๐ฎ๐ณIndia prashant.c Dharamshala
Please take the text from the README file of the module in the hook_help().
- ๐ฎ๐ณIndia adminazhar
I disagree with you @Prashant.c, the help page shows after installation, so there is no need to add the installation instructions etc in the help page.
It would be better to have a better help page by cleanup up few headings from readme and add better text. - ๐ฎ๐ณIndia prashant.c Dharamshala
@adminazhar README file does not have only "installation" instructions it also contains "configuration" and "troubleshooting" instructions as well therefore it is a good idea to display the content from the README file for consistency also.
Moreover, there is no harm in showing "installation" instructions also along with other valuable details.However, if it is not read from the README file then please add all the possible configuration details provided by the module otherwise help page will not be beneficial to someone.
- Status changed to RTBC
over 1 year ago 9:55am 16 March 2023 - ๐ฎ๐ณIndia prashant.c Dharamshala
We can push it in the next release along with some other features.
-
Prashant.c โ
committed 3b5a0889 on 3.x authored by
noorulshameera โ
Issue #3337580 by Charchil Khandelwal, noorulshameera, himanshu_jhaloya...
-
Prashant.c โ
committed 3b5a0889 on 3.x authored by
noorulshameera โ
- Status changed to Fixed
over 1 year ago 6:16am 28 March 2023 Automatically closed - issue fixed for 2 weeks with no activity.