Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Port the readme file.
Comment | File | Size | Author |
---|---|---|---|
#11 | port_readme_file-2853744-11.patch | 445 bytes | kunalkursija |
Comments
Comment #2
shylajaphp CreditAttribution: shylajaphp commentedComment #3
shylajaphp CreditAttribution: shylajaphp at Iksula commentedComment #4
shylajaphp CreditAttribution: shylajaphp at Iksula commentedComment #5
kunalkursija CreditAttribution: kunalkursija at Iksula, Blisstering Solutions commentedHi @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.
Comment #6
kunalkursija CreditAttribution: kunalkursija at Iksula, Blisstering Solutions commentedAdding Patch For Review.
Comment #7
kunalkursija CreditAttribution: kunalkursija at Iksula, Blisstering Solutions commentedComment #8
riddhi.addweb CreditAttribution: riddhi.addweb at AddWeb Solution Pvt. Ltd. commented@kunal.kursija,Thanks for the patch but as per drupal standard format for ReadMe file "* Requirements & * Maintainers" details were missing from the file.
Comment #9
kunalkursija CreditAttribution: kunalkursija at Iksula, Blisstering Solutions commentedHi @jigar,
Agreed with the missing "requirements" section. However, the maintainers part is the optional one.
Will work on this.
Comment #11
kunalkursija CreditAttribution: kunalkursija at Iksula, Blisstering Solutions commentedAdding patch with updated Readme file.
Comment #12
kunalkursija CreditAttribution: kunalkursija at Iksula, Blisstering Solutions commentedComment #13
dhruveshdtripathi CreditAttribution: dhruveshdtripathi at DevsAdda for OpenSense Labs commentedPatch applied cleanly and REQUIREMENTS section added.
Comment #14
polaki_viswanath CreditAttribution: polaki_viswanath at Kellton Tech Solutions Ltd commented