- Issue created by @dev2.addweb
- Issue was unassigned.
- Status changed to Needs review
10 months ago 8:40am 1 July 2024 - ๐ฎ๐ณIndia dev2.addweb
Hi,
I have updated README.md as per standard template. Please review. - Status changed to Needs work
10 months ago 4:12am 2 July 2024 - ๐ต๐ญPhilippines kenyoOwen
Hi nilesh.addweb
I reviewed your MR!2 and for configuration no need to say
Go to Administration ยป Extend and Enable module.
since that is covered in the installation section and the #3 in configuration this should be big lettervisit
to "Visit"Thanks.
- Status changed to Needs review
10 months ago 5:38am 2 July 2024 - ๐ฎ๐ณIndia dev2.addweb
Thanks for review @kenyoowen,
I have updated README.md as per #4, Please review it. - ๐ฎ๐ณIndia riddhi.addweb
I have tested the Read Me file, need to change the Navigate spelling from Navigage to Navigate. Attaching Screenshots for the same. Moving in Needs Work.
- Status changed to Needs work
10 months ago 1:03pm 3 July 2024 - Status changed to Needs review
10 months ago 1:11pm 3 July 2024 - ๐ฎ๐ณIndia dev2.addweb
Hi,
I have modified README.md as per #6, Please review it. - ๐ฎ๐ณIndia riddhi.addweb
I have tried and tested the Read Me file
I have compared the Read Me file with the Doc, attaching the doc link for the same.
Doc Link:- https://www.drupal.org/docs/develop/managing-a-drupalorg-theme-module-or... โEverything seemed alright when we compared it in the README.md file, attaching the URL for the same. And moving it to RTBC.
README.md File URL:- https://git.drupalcode.org/issue/sa11y-3458222/-/blob/3458222-update-rea... - Status changed to RTBC
10 months ago 1:22pm 3 July 2024 - First commit to issue fork.
-
markconroy โ
committed 5920b6a8 on 3.x authored by
nilesh.addweb โ
#3458222 updated README.md.
-
markconroy โ
committed 5920b6a8 on 3.x authored by
nilesh.addweb โ
- ๐ฎ๐ชIreland markconroy
Merged, thanks very much for working on this.
---
Thanks to The Confident for sponsoring my time to work on this. Automatically closed - issue fixed for 2 weeks with no activity.