Problem/Motivation

The README.txt file don't use the proposed template https://www.drupal.org/docs/develop/documenting-your-project/readme-temp...

Proposed resolution

Edit the README.txt file according to the https://www.drupal.org/docs/develop/documenting-your-project/readme-temp...

Remaining tasks

None.

User interface changes

None.

API changes

None.

Data model changes

None.

CommentFileSizeAuthor
#7 interdiff.txt369 bytesadriancid
#7 use_the_readme_template-2865907-7.patch2.69 KBadriancid
#3 use_the_readme_template-2865907-2.patch2.68 KBadriancid
#2 how_to_access_to_the_user_info_on_the_search_page-2662674-14.patch481 bytesadriancid
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

adriancid created an issue. See original summary.

adriancid’s picture

adriancid
Primary language Spanish
Location Montreal, Canada
CreditAttribution: adriancid as a volunteer commented
Status: Active » Needs review
FileSize
how_to_access_to_the_user_info_on_the_search_page-2662674-14.patch481 bytes
adriancid’s picture

adriancid
Primary language Spanish
Location Montreal, Canada
CreditAttribution: adriancid as a volunteer commented
FileSize
use_the_readme_template-2865907-2.patch2.68 KB

Sorry I just uploaded in #2 the wrong patch

The last submitted patch, 2: how_to_access_to_the_user_info_on_the_search_page-2662674-14.patch, failed testing.

Aanal.addweb’s picture

Aanal.addweb CreditAttribution: Aanal.addweb at AddWeb Solution Pvt. Ltd. commented

@adriancid, Thanks for the updated ReadMe file it is as per the standard & works well for me as i checked it manually by applying in module file.

Aanal.addweb’s picture

Aanal.addweb CreditAttribution: Aanal.addweb at AddWeb Solution Pvt. Ltd. commented
Status: Needs review » Reviewed & tested by the community
adriancid’s picture

adriancid
Primary language Spanish
Location Montreal, Canada
CreditAttribution: adriancid as a volunteer commented
FileSize
use_the_readme_template-2865907-7.patch2.69 KB
interdiff.txt369 bytes

I just fixed a problem with the maintainers info, in the case of Kaushal I used the short url to have his info in only one line.

See the interdiff.txt file.

Aanal.addweb’s picture

Aanal.addweb CreditAttribution: Aanal.addweb at AddWeb Solution Pvt. Ltd. commented

@adriancid , Thanks for the corrected patch, i think in last patch only we have to replace "httpswww.drupal.org/u/kaushalkishorejaiswal" to https://www.drupal.org/u/kaushalkishorejaiswal' but current added link for maintainer is also true.

Thanks again for correcting me!..

adriancid’s picture

adriancid
Primary language Spanish
Location Montreal, Canada
CreditAttribution: adriancid as a volunteer commented

If we use this:

* Kaushal Kishore (kaushalkishorejaiswal) - https://www.drupal.org/u/kaushalkishorejaiswal

The line will have more than 80 characters and we will need to split it in two lines, I think that is better use one line and the short url.

  • adriancid committed ee89c07 on 7.x-2.x 
    Issue #2865907 by adriancid, dhwani.addweb: Use the README.txt template
adriancid’s picture

adriancid
Primary language Spanish
Location Montreal, Canada
CreditAttribution: adriancid as a volunteer commented
Status: Reviewed & tested by the community » Fixed

  • adriancid committed 2de0ed7 on 7.x-2.x 
    Issue #2865907 by adriancid, dhwani.addweb: Use the README.txt template
adriancid’s picture

adriancid
Primary language Spanish
Location Montreal, Canada
CreditAttribution: adriancid as a volunteer commented
Issue tags: +Novice

Status: Fixed » Closed (fixed)

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

adriancid’s picture

adriancid
Primary language Spanish
Location Montreal, Canada
CreditAttribution: adriancid for Ville de Montréal commented