The README file for this module needs to be updated so I updated the the README file with the new README file format. I t is very important to keep the README updated especially in the new Drupal 8. I included the new README.txt file and also a patch for the README file so that you may choose which ever way best to implement it in the module.
This was a task done for Google code In.

CommentFileSizeAuthor
#13 interdiff.txt3.25 KBpfrenssen
#13 2843368-13.patch3.52 KBpfrenssen
#11 2843368-11.patch4.03 KBPooja Ganjage
#7 2843368-7.patch5.77 KBPooja Ganjage
#2 2843368-readme-update-1.patch4.06 KBlewiking
README.txt2.25 KBlewiking
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Phillip Junior created an issue. See original summary.

lewiking’s picture

lewiking
Primary language English
CreditAttribution: lewiking at Google Code-In commented
FileSize
2843368-readme-update-1.patch4.06 KB
lewiking’s picture

lewiking
Primary language English
CreditAttribution: lewiking at Google Code-In commented
Issue tags: +GCI16
keshavv’s picture

keshavv CreditAttribution: keshavv at gai Technologies Pvt Ltd for gai Technologies Pvt Ltd commented

#2 Patch will cover most of the documentation standard.

keshavv’s picture

keshavv CreditAttribution: keshavv at gai Technologies Pvt Ltd for gai Technologies Pvt Ltd commented
Status: Needs review » Reviewed & tested by the community
pfrenssen’s picture

pfrenssen
Location Sofia
CreditAttribution: pfrenssen at Randstad Digital for European Commission and European Union Institutions, Agencies and Bodies commented
Status: Reviewed & tested by the community » Needs work

Thanks for working on this!

  1. https://drupal.org/documentation/install/modules-themes/modules-7
    This link leads to the documentation for Drupal 7.
  2. * Read more about installing modules at http://drupal.org/node/70151
    Let's remove this, this is for Drupal 5 and 6. 
  3. TROUBLESHOOTING
---------------

 * This consist of issues that are asked frequently which are fixed.

FAQ
---

* This consist of evidence of the issues that are asked frequently which are
  fixed.

MAINTAINERS
-----------

Current Maintainers:
 *
 
This project has been sponsored by:
 *

    Let's not add empty placeholders in the readme, these can either be filled in, or removed.

Pooja Ganjage’s picture

Pooja Ganjage CreditAttribution: Pooja Ganjage at Asentech LLC commented
FileSize
2843368-7.patch5.77 KB

Hi,

I am updating patch as per the comment #6 given.

Please review the patch.

Thanks.

Pooja Ganjage’s picture

Pooja Ganjage CreditAttribution: Pooja Ganjage at Asentech LLC commented
Status: Needs work » Needs review
Webbeh’s picture

Webbeh
he/him/his
Location Georgia, USA
CreditAttribution: Webbeh at Georgia Institute of Technology commented

Patch in #7 fails:

Hunk #1 FAILED at 1.
Hunk #2 FAILED at 6.
2 out of 2 hunks FAILED -- saving rejects to file README.txt.rej
Webbeh’s picture

Webbeh
he/him/his
Location Georgia, USA
CreditAttribution: Webbeh at Georgia Institute of Technology commented
Status: Needs review » Needs work
Pooja Ganjage’s picture

Pooja Ganjage CreditAttribution: Pooja Ganjage at Asentech LLC commented
FileSize
2843368-11.patch4.03 KB

Uploading new patch.

Pooja Ganjage’s picture

Pooja Ganjage CreditAttribution: Pooja Ganjage at Asentech LLC commented
Status: Needs work » Needs review
pfrenssen’s picture

pfrenssen
Location Sofia
CreditAttribution: pfrenssen at Randstad Digital for European Commission and European Union Institutions, Agencies and Bodies commented
Status: Needs review » Reviewed & tested by the community
FileSize
2843368-13.patch3.52 KB
interdiff.txt3.25 KB

Thanks! I am proposing a few small changes.

  • pfrenssen committed a922718 on 8.x-1.x 
    Issue #2843368 by Pooja Ganjage, lewiking, pfrenssen, Webbeh, keshav.k:...
pfrenssen’s picture

pfrenssen
Location Sofia
CreditAttribution: pfrenssen at Randstad Digital for European Commission and European Union Institutions, Agencies and Bodies commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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