- Issue created by @deaom
- @deaom opened merge request.
- Status changed to Needs review
over 1 year ago 6:37am 11 September 2023 - Status changed to Needs work
over 1 year ago 8:11am 11 September 2023 - thakurnishant_06 India
Reviewed the changes in the MR15 it doesnt follow the drupal README.md format. →
changing status back to NW. - Status changed to Needs review
over 1 year ago 10:25am 11 September 2023 - Assigned to thakurnishant_06
- Status changed to Needs work
over 1 year ago 10:42am 11 September 2023 - Status changed to Needs review
over 1 year ago 10:51am 11 September 2023 - Issue was unassigned.
- Status changed to Needs work
over 1 year ago 11:45pm 11 September 2023 - 🇵🇭Philippines kenyoOwen
Hi thakurnishant_06
I reviewed and applied your MR, but I think you shouldn't remove the "Instapage user account that you can get on their website www.instapage.com" since you can't configure the module without an instapage account and also in the README.md template requirement section it says "The idea is to inform the users about what is required, so that everything they need can be procured and included in advance of attempting to install the module."
Thank you.
- Status changed to Needs review
over 1 year ago 12:13am 12 September 2023 - 🇵🇭Philippines kenyoOwen
Updated the requirements sections as per the README.md Template.
- Status changed to RTBC
over 1 year ago 11:22am 12 September 2023 - 🇮🇳India Akshay72
Hi Kenyon Owen,
I have tested your README.md file, and you have provided excellent documentation. Everything is working as expected based on your documentation. I have gone through every step, and everything is working fine.
Thank you,
Akshay