- ๐ฎ๐ณIndia Manoj Raj.R Chennai
Updated the Readme File as per the latest format.
https://www.drupal.org/docs/develop/managing-a-drupalorg-theme-module-or... โPlease review.
- Issue was unassigned.
- Status changed to Needs review
about 1 year ago 12:02pm 6 April 2023 - Status changed to RTBC
about 1 year ago 12:13pm 15 June 2023 - Status changed to Needs review
12 months ago 7:06pm 14 July 2023 - ๐ฎ๐นItaly apaderno Brescia, ๐ฎ๐น
It would be better if somebody else reviews the MR.
- ๐ฎ๐ณIndia Nupur Badola
Reviewed 3319899-4.patch, the patch is not cleanly applied.
1. Introduction format is incorrect and not as per readme.md file template. 2. Requirement section is missing. 3. Two lines prior to ##/### headings.
- Status changed to Needs work
10 months ago 12:03pm 23 August 2023 - ๐ฎ๐นItaly apaderno Brescia, ๐ฎ๐น
The format to use in a README.md file and which information must be provided is described in README.md template โ .
- Assigned to nitin_lama
- last update
10 months ago Composer require failure - Issue was unassigned.
- Status changed to Needs review
10 months ago 6:45am 28 August 2023 - Status changed to Needs work
10 months ago 7:34am 28 August 2023 - ๐ฎ๐ณIndia Anjali Mehta
Anjali Mehta โ made their first commit to this issueโs fork.
- last update
10 months ago Composer require failure - Status changed to Needs review
10 months ago 11:59am 14 September 2023 - Status changed to Needs work
10 months ago 1:58pm 15 September 2023 - last update
10 months ago Composer require failure - Status changed to Needs review
10 months ago 5:01am 18 September 2023 - Status changed to Needs work
10 months ago 1:11pm 18 September 2023 - First commit to issue fork.
- last update
10 months ago Composer require failure - Status changed to Needs review
10 months ago 2:01pm 18 September 2023 - Status changed to Needs work
9 months ago 3:33am 25 September 2023 - ๐ต๐ญPhilippines kenyoOwen
Hi sakthi_dev
I applied and reviewed your MR!9, but the configuration section is still not fixed. As per Apaderno "It is not necessary to say In menu, which is also missing a definite article." maybe you can change it to something like "Navigate to /admin/people/login-and-logout-redirect-per-role and set up login or logout tables for redirect URL or what to set in the configuration page.". Please see the screenshots attached.
Thank you.
- ๐ฎ๐นItaly apaderno Brescia, ๐ฎ๐น
I also added further comments in another review I have done.
- Status changed to Needs review
7 months ago 6:33am 13 December 2023