- Issue created by @Pracheth
- Issue was unassigned.
- Status changed to Needs review
over 2 years ago 7:38am 27 January 2023 - Status changed to RTBC
over 2 years ago 8:09am 27 January 2023 - 🇮🇳India sahilgidwani Jaipur
I have reviewed the PR and it looks perfect to me.
Moving the issue to RTBC. - Status changed to Needs work
over 2 years ago 10:45am 27 January 2023 - 🇮🇹Italy apaderno Brescia, 🇮🇹
No, showing the README.txt file content is not what that hook should do, since:
- The README.txt file contains information that is only helpful before installing the module
- The text is not translatable
- First commit to issue fork.
- Status changed to Needs review
over 2 years ago 11:54am 27 January 2023 - 🇧🇷Brazil elber Brazil
Hi I added a suggestion to hook help, please revise.
- Status changed to Needs work
over 2 years ago 12:11pm 27 January 2023 - 🇮🇹Italy apaderno Brescia, 🇮🇹
The text explain how to use the module, but it does not describe what the module does.
- Status changed to Needs review
over 2 years ago 12:19pm 27 January 2023 - First commit to issue fork.
- 🇫🇷France dydave
It seems development has moved to the 2.1.x branch now, therefore:
Created the new merge request MR !8 above at #11 based on the previous merge request but rebased for branch 2.1.x.
Updated the previous merge request to be hidden.
This MR could use a round of textual/content review and overall testing.
Any feedback, comments, reviews or suggestions would be greatly appreciated.
Thanks in advance!