This module is still using old README.md template format in README.md file.

Follow the pattern in the README.md template.

Issue fork menu_fast_edit-3335134

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

Charchil Khandelwal created an issue. See original summary.

Charchil Khandelwal opened merge request !2

charchil khandelwal’s picture

charchil khandelwal commented
Assigned: charchil khandelwal » Unassigned
Status: Active » Needs review

Created MR, please review.
Thank You

nupur badola’s picture

nupur badola commented

It should be ##Table of content instead of ## Contents of this file.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Priority: Normal » Minor
Status: Needs review » Needs work

sakthi_dev made their first commit to this issue’s fork.

sakthi_dev’s picture

sakthi_dev commented
Status: Needs work » Needs review

Please review.

nupur badola’s picture

nupur badola commented

Reviewed MR!2, the patch is not cleanly applied.

1. In the Table of Contents section, use - (a hyphen) instead of * (an asterisk).
2. How to use and how it works are not merged.
elber’s picture

elber
Primary language Portuguese, Brazil
Location Brazil
commented
Status: Needs review » Reviewed & tested by the community

Hi I reviewed the changes.

MR applied cleanly.

All the errors mentioned previously has been fixed.

README file is followind drupal standards.

Moving to RTBC.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Status: Reviewed & tested by the community » Needs work

thakurnishant_06 made their first commit to this issue’s fork.

thakurnishant_06’s picture

thakurnishant_06 commented
Status: Needs work » Needs review

Kindly review the changes.

nupur badola’s picture

nupur badola commented
Status: Needs review » Needs work

Reviewed MR!2, the patch is not cleanly applied.
1. In Supporting organization, Hyphen (-)is not required as there is only one support organization.

thakurnishant_06’s picture

thakurnishant_06 commented
Status: Needs work » Needs review
kenyoowen’s picture

kenyoowen commented
Status: Needs review » Reviewed & tested by the community

Hi thakurnishant_06

I reviewed your changes and the issue mentioned previously was resolved and README.md file is following the README.md template.

Moving to RTBC
Thank you.

torfj made their first commit to this issue’s fork.

torfj’s picture

torfj
Location Seattle, WA
commented
Status: Reviewed & tested by the community » Fixed

Thanks everyone! I made a few improvements and merged the changes to the dev branch.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.