Port the readme file.

CommentFileSizeAuthor
#11 port_readme_file-2853744-11.patch445 byteskunalkursija
#6 port_readme_file-2853744-6.patch1.35 KBkunalkursija
#2 README.txt1.19 KBshylajaphp

Comments

kunal.kursija created an issue. See original summary.

shylajaphp’s picture

shylajaphp commented
StatusFileSize
new README.txt1.19 KB
shylajaphp’s picture

shylajaphp commented
Assigned: Unassigned » shylajaphp
shylajaphp’s picture

shylajaphp commented
Assigned: shylajaphp » Unassigned
Status: Active » Needs review
kunalkursija’s picture

kunalkursija commented
Assigned: Unassigned » kunalkursija
Status: Needs review » Needs work

Hi @shylajaphp ,

Thank you very much for the readme file, The file looks good.

I Will create a patch once I have a 8.x branch created by the author. So changing the status.

kunalkursija’s picture

kunalkursija commented
Assigned: kunalkursija » Unassigned
Status: Needs work » Needs review
StatusFileSize
new port_readme_file-2853744-6.patch1.35 KB

Adding Patch For Review.

kunalkursija’s picture

kunalkursija commented
Version: 7.x-1.0 » 8.x-1.x-dev
riddhi.addweb’s picture

riddhi.addweb commented
Status: Needs review » Needs work

@kunal.kursija,Thanks for the patch but as per drupal standard format for ReadMe file "* Requirements & * Maintainers" details were missing from the file.

kunalkursija’s picture

kunalkursija commented
Assigned: Unassigned » kunalkursija

Hi @jigar,

Agreed with the missing "requirements" section. However, the maintainers part is the optional one.

Will work on this.

kunalkursija’s picture

kunalkursija commented
StatusFileSize
new port_readme_file-2853744-11.patch445 bytes

Adding patch with updated Readme file.

kunalkursija’s picture

kunalkursija commented
Status: Needs work » Needs review
dhruveshdtripathi’s picture

dhruveshdtripathi commented
Assigned: kunalkursija » Unassigned
Status: Needs review » Reviewed & tested by the community

Patch applied cleanly and REQUIREMENTS section added.

polaki_viswanath’s picture

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

Status: Fixed » Closed (fixed)

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