We could be using README.md instead of README.txt to make use of the nicer rendering via markdown.
Now that we've moved to GitLab, we should probably use Markdown instead of plain .txt files for our documentation.

Follow the pattern in the README.md template.

CommentFileSizeAuthor
#8 Currency Taxonomy.png73.05 KBnupur badola
#4 3331747-4.patch2.76 KBshubham rathore

Issue fork currency_taxonomy-3331747

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

Rashmi Gaurana created an issue. See original summary.

ressa’s picture

ressa
he/him
commented
Issue summary: View changes
shubham rathore’s picture

shubham rathore commented
Assigned: Unassigned » shubham rathore
shubham rathore’s picture

shubham rathore commented
Assigned: shubham rathore » Unassigned
Status: Active » Needs review
StatusFileSize
new 3331747-4.patch2.76 KB

Hi i create patch for replace README.txt to README.md. Issue fork can't created becouse module have no dev branch.
Please review.
Thanks

Sonal Gyanani’s picture

Sonal Gyanani commented
Assigned: Unassigned » Sonal Gyanani

Sonal Gyanani opened merge request !1

Sonal Gyanani’s picture

Sonal Gyanani commented
Assigned: Sonal Gyanani » Unassigned

Thanks @Shubham Rathore, for the patch I created merge request for easy preview and commenting.
Please review

nupur badola’s picture

nupur badola commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new Currency Taxonomy.png73.05 KB

Patch is working and marking it to RTBC.

gisle’s picture

gisle
he/him
Primary language Norwegian Bokmål
Location Norway
commented
Status: Reviewed & tested by the community » Needs review

Nupur Badola: Please don't upload screenshots of your terminal showing that a patch applies, followed by a directory listing. Anyone can observe what the automated tools do. Posting a screenshot of the output of it here just adds noise to the issue.

Before an issue can be marked RTBC the changes need to be reviewed, this wasn't done.

Please read up on Issue Etiquette. Pay attention on #11 and #14 on the "Don't" list.

Your comment here is spam and you've been tagged as a spammer. I'm setting the status back to "Needs review".

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

thakurnishant_06’s picture

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

I have reviewed the readme file and it appears to comply with recommended Drupal readme.md template.
Moving to RTBC.

Thank you.

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

AstonVictor opened merge request !4

AstonVictor closed merge request !1

astonvictor’s picture

astonvictor commented
Status: Reviewed & tested by the community » Fixed

Merged the MR with updated Readme file.

Status: Fixed » Closed (fixed)

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