- ๐ฎ๐ณIndia Rinku Jacob 13 Kerala
Rinku Jacob 13 โ made their first commit to this issueโs fork.
- First commit to issue fork.
- Status changed to Needs review
about 1 year ago 9:02pm 19 October 2023 - ๐ฆ๐ทArgentina tguerineau
Hi, I have worked on the issue of replacing the README.txt file with a README.md file to take advantage of Markdown's formatting capabilities, improving the readability and visual appeal of the project documentation.
Here are the steps I took to resolve this issue:
- Renamed the existing README.txt to README.md.
- Converted the plain text formatting to Markdown syntax for enhanced rendering on GitLab.The changes have been pushed to the issue fork.
Please review the MR and let me know if any further changes or adjustments are required.
- Status changed to Needs work
about 1 year ago 12:09am 8 November 2023 - ๐ต๐ญPhilippines kenyoOwen
Hi tguerineau
I reviewed your changes on the README.md . This should be added in the Project name and introduction section
For a full description of the module, visit the [project page](https://www.drupal.org/project/admin_menu). Submit bug reports and feature suggestions, or track changes in the [issue queue](https://www.drupal.org/project/issues/admin_menu).
as this is recommended on the README.md template โ . Also, an installation section should be added.
Thank you.
- First commit to issue fork.
- Status changed to Needs review
about 1 year ago 4:27pm 8 November 2023 - ๐ฆ๐ทArgentina tguerineau
Hi @kenyoOwen,
I wanted to provide an update on the progress of this issue. I have reviewed the latest commit by @thakurnishant_06, and it appears that they have addressed the feedback provided. The commit includes the addition of the "Project name and introduction" section along with a clear "Installation" section as requested.
The issue status was not updated to "Needs review" after this commit, so I have gone ahead and updated it to reflect the current state of the issue for review.
I believe the issue is now ready for the next steps in the review process. Please let me know if any modifications are required.
- Issue was unassigned.
- Status changed to Needs work
about 1 year ago 5:07am 9 November 2023 - thakurnishant_06 India
Hello, @tguerineau. Thanks for working on this. There are still a few minor things that need to be changed. Please follow the Drupal readme.md template โ .
changing status back to NW - Status changed to Needs review
about 1 year ago 10:32pm 9 November 2023 - ๐ฆ๐ทArgentina tguerineau
Hi @thakurnishant_06,
I have updated the README.md file following the Drupal README.md template provided in the documentation guidelines.
The sections for Troubleshooting and FAQ were not included, as they are optional and there's currently no information to add for those categories.
The updated README is now pushed to the repository. I have changed the issue status to "Needs review" for the latest updates to be examined.
- Status changed to Needs work
about 1 year ago 3:38am 13 November 2023 - ๐ต๐ญPhilippines kenyoOwen
Hi tguerineau
Small letter on "contents" in the "TOC" section
Table of contents
. Table of contents format is only## Table of contents - Requirements - Recommended modules - Installation - Configuration - Troubleshooting - FAQ - Maintainers
based on README.md template. I think the installation section text should be
Install as you would normally install a contributed Drupal module. For further information, see
but change module to theme.
- Status changed to Needs review
about 1 year ago 3:58pm 13 November 2023 - ๐ฆ๐ทArgentina tguerineau
Hi @kenyoOwen,
Thank you for your feedback. I have made the change to the README.md as suggested.
The changes have been committed and pushed. The issue status has been updated to "Needs review" for the latest revision to be checked.
- Status changed to Needs work
about 1 year ago 1:11am 14 November 2023 - ๐ต๐ญPhilippines kenyoOwen
Hi tguerineau
You missed the formatting issue I raised, which is the table of content formats.
- Requirements - Recommended modules - Installation - Configuration - Troubleshooting - FAQ - Maintainers
If you're still confused about the format I raised, you can visit the README.md template โ .
Thank you.
- Status changed to Needs review
about 1 year ago 3:40pm 14 November 2023 - ๐ฆ๐ทArgentina tguerineau
Hi,
I've updated the README.md file to align with the format specified in the Drupal README.md template. The Table of Contents now follows the plain text format without hyperlinks.
This update has been committed and pushed, and the issue status is set to "Needs review.".
Additionally, I wanted to briefly mention that implementing a clickable Table of Contents could enhance user navigation for longer documents. This is a common feature in many open-source projects. I understand the importance of following Drupal's standards, but I thought it worthwhile to suggest this potential improvement for future consideration.
- Status changed to Needs work
about 1 year ago 5:23am 21 November 2023 - ๐ต๐ญPhilippines kenyoOwen
Hi tguerineau
I reviewed your changes and last thing I noticed that Project name and introduction section it has introduction text, it should be remove and replaced with Project name.
Thank you.
- Merge request !16Issue #3245899 by anagomes, beatrizrodrigues, shweta__sharma, Gauravmahlawat:... โ (Open) created by thakurnishant_06
- Status changed to Needs review
about 1 year ago 7:10am 21 November 2023 - Status changed to RTBC
12 months ago 5:41am 22 November 2023 - ๐ต๐ญPhilippines kenyoOwen
Hi thakurnishant_06
I reviewed your MR!16 and it has the changes and follows the README.md template.
Thank you
- ๐ฆ๐ทArgentina tguerineau
Hi @kenyoOwen,
Thank you for the correction @thakurnishant_06.
If anything else needs to be corrected, let me know.