- ๐ฎ๐ณIndia Anjali Mehta
Anjali Mehta โ made their first commit to this issueโs fork.
- ๐ฎ๐ณIndia abhishek_virasat
abhishek_gupta1 โ made their first commit to this issueโs fork.
- Status changed to Needs review
10 months ago 5:19am 14 February 2024 - ๐ฎ๐ณIndia abhishek_virasat
@sureshcj , i have created readme.md file. please kindly review the MR.
- Status changed to Needs work
10 months ago 7:34am 14 February 2024 - ๐ต๐ญPhilippines kenyoOwen
Hi abhishek_gupta1
I reviewed the MR!2 and it seems didn't follow the README.md template โ .
These are the sections that Drupal recommendsProject name and introduction (required) Table of contents (optional) Requirements (required) Recommended modules (optional) Installation (required, unless a separate INSTALL.md is provided) Configuration (required) Troubleshooting & FAQ (optional) Maintainers (optional)
Thank you
- Status changed to Needs review
10 months ago 7:57am 14 February 2024 @Anjali Mehta, the "Ppayment" spelling is incorrect in Reame.md file and it should be "Payments".
- Issue was unassigned.
- Status changed to Needs work
10 months ago 9:58am 21 February 2024 - First commit to issue fork.
- Status changed to Needs review
10 months ago 1:11pm 21 February 2024 - Status changed to RTBC
10 months ago 1:50pm 21 February 2024 - Status changed to Needs work
10 months ago 7:04am 22 February 2024 - ๐ต๐ญPhilippines kenyoOwen
Hi sakthi_dev
I reviewed the MR!2 and the "Project name and introduction" section the text
The Multiple Payments Module
should be change toThe Commerce Multiple Payments module
and according to the README.md templateText manually word-wrapped within around 80 cols
.Thank you.
- Assigned to nitin_lama
- Status changed to Needs review
10 months ago 1:03pm 22 February 2024 - Issue was unassigned.
- Status changed to RTBC
10 months ago 6:15am 26 February 2024 - ๐ต๐ญPhilippines kenyoOwen
Hi nitin_lama
I reviewed your changes in MR!2 and it addressed comment #14, and the README.md file is following the README.md template.
Thank you