- Issue created by @Jay Jangid
- @jay-jangid opened merge request.
- Issue was unassigned.
- Status changed to Needs review
almost 2 years ago 6:20am 21 February 2023 - 🇮🇳India Jay Jangid
Created MR for this issue , please review.
Thank you. - Status changed to Needs work
almost 2 years ago 10:42am 21 February 2023 - 🇮🇹Italy apaderno Brescia, 🇮🇹
The MR does not say more than a sentence the project page says. In order to be worth adding it, it needs to say more.
- First commit to issue fork.
- Status changed to Needs review
over 1 year ago 5:46am 28 February 2023 - 🇮🇳India arunkumark Coimbatore
The hook_help document has been updated and raised in the PR.
https://git.drupalcode.org/project/commerce_reports/-/merge_requests/10/...
- 🇮🇳India Mahima_Mathur23
Reviewed the MR. Attaching the screenshots: before_patch and after_patch.
- 🇮🇳India Mahima_Mathur23
Please ignore previous comment "#8". Posted in the wrong issue.
- 🇮🇳India kavitha@specbee.com
Verified the MR10 on 9.5.9 & help() is added. Added screenshots for reference & it can be moved to RTBC
Thank you.
- Status changed to RTBC
over 1 year ago 11:50am 17 July 2023 - Status changed to Needs work
over 1 year ago 2:44pm 17 July 2023 - 🇮🇹Italy apaderno Brescia, 🇮🇹
It is useless to add a screenshot that shows the hook implementation has been recognized by Drupal. It is sufficient to check the hook implementation has the correct name.
- Assigned to nitin_lama
- last update
over 1 year ago 22 pass - Issue was unassigned.
- Status changed to Needs review
over 1 year ago 6:08am 18 July 2023 - Status changed to Needs work
over 1 year ago 7:59am 18 July 2023 - 🇮🇳India mukhtarm
Hi @apaderno Please review the patch. Addressed 17 📌 Implement hook_help() Needs review . Thanks
- 🇮🇹Italy apaderno Brescia, 🇮🇹
Since a MR has been already created, it would be better to keep using the MR.
- Assigned to nitin_lama
- last update
over 1 year ago 22 pass - Issue was unassigned.
- Status changed to Needs review
over 1 year ago 9:28am 18 July 2023 - last update
over 1 year ago PHPLint Failed - last update
over 1 year ago 22 pass