- ๐ฎ๐ณIndia Nupur Badola
Error in the patch.
3321599-2.patch.txt:17: new blank line at EOF. + warning: 1 line adds whitespace errors.
- Status changed to Needs review
over 1 year ago 5:20am 26 May 2023 - ๐ฎ๐ณIndia mrinalini9 New Delhi
Updated patch #2 by addressing #3, please review it.
Thanks!
- Status changed to RTBC
over 1 year ago 11:40am 15 June 2023 - Status changed to Needs review
over 1 year ago 7:06pm 14 July 2023 - ๐ฎ๐นItaly apaderno Brescia, ๐ฎ๐น
It would be better if somebody else reviews the patch.
- Status changed to Needs work
over 1 year ago 3:04pm 18 July 2023 - ๐ต๐ญPhilippines kenyoOwen
Hi mrinalini9
I applied and reviewed your patch it does replace README.txt with README.md, but it doesn't follow the README.md template โ Please see the screenshots attached.
Thank you.
- Assigned to nitin_lama
- Issue was unassigned.
- Assigned to govind_giri_goswami
- Status changed to Needs review
about 1 year ago 11:37am 3 October 2023 - ๐ฎ๐ณIndia govind_giri_goswami
i have created a patch. please review it.
- Issue was unassigned.
- Status changed to Needs work
about 1 year ago 1:03pm 3 October 2023 - ๐ฎ๐นItaly apaderno Brescia, ๐ฎ๐น
+## Configuration + + +## Maintainers
Either the Configuration section is not added to the Table of contents section, or content it is provided for that section. If the module does not use configuration, the section should say that.
- Status changed to Needs review
about 1 year ago 5:52pm 4 October 2023 - ๐ฎ๐ณIndia sakthi_dev
Updated the Configuration section. Please review.
- Status changed to Needs work
about 1 year ago 8:34am 5 October 2023 - ๐ฎ๐นItaly apaderno Brescia, ๐ฎ๐น
The last patch does not seem correct.
diff --git a/README.txt b/README.txt deleted file mode 100644 index 4339ca7..0000000 --- a/README.txt +++ /dev/null @@ -1 +0,0 @@ -Doghouse Views Reference
- ๐ฎ๐ณIndia mohd sahzad
Mohd Sahzad โ made their first commit to this issueโs fork.
- Status changed to Needs review
about 1 year ago 9:51am 6 October 2023 - ๐ฎ๐ณIndia mohd sahzad
Hi @Carlopogus,
i have updated this Replace README.txt with README.md , so please review.. - Status changed to Needs work
11 months ago 10:54pm 14 December 2023 - ๐ต๐ญPhilippines kenyoOwen
Hi Mohd Sahzad
I reviewed your MR!1 and it doesn't follow the README.md template โ .
Drupal recommends the following README sections: Project 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
- First commit to issue fork.
- Status changed to Needs review
11 months ago 4:30am 18 December 2023 - Status changed to Needs work
11 months ago 7:06am 18 December 2023 - ๐ฎ๐ณIndia Nupur Badola
Reviewwd MR!1, the patch is not cleanly applied.
1. Two lines prior to ##/### headings. Lines need above Maintainers section.
- Status changed to Needs review
11 months ago 7:09am 18 December 2023 - Status changed to RTBC
11 months ago 12:02pm 19 December 2023 - ๐ฎ๐ณIndia Nupur Badola
Reviewed MR!1, the file is updated as per readme.md file template. Hence marking it to RTBC.