Hi team,

I add README.md file for this module since this is required for security advisory coverage process:
https://www.drupal.org/node/539608
https://www.drupal.org/node/1587704

CommentFileSizeAuthor
#5 draggableViews-add-readme-file-2877884-5.patch2.21 KBsimbaw
#3 1.png20.73 KBriddhi.addweb
#2 draggableViews-add-readme-file-2877884-2.patch2.25 KBsimbaw
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

simbaw created an issue. See original summary.

simbaw’s picture

simbaw CreditAttribution: simbaw commented
Status: Active » Needs review
FileSize
draggableViews-add-readme-file-2877884-2.patch2.25 KB
riddhi.addweb’s picture

riddhi.addweb CreditAttribution: riddhi.addweb at AddWeb Solution Pvt. Ltd. commented
FileSize
1.png20.73 KB

Thanks for patch Samba, but I found two issues in it. When I apply you patch manually it shows me error. After completion of installation section please add two line gap, currently it is showing just one line gap.

PFA screenshots for the same.

Also refer this link for checking standards in readme.

riddhi.addweb’s picture

riddhi.addweb CreditAttribution: riddhi.addweb at AddWeb Solution Pvt. Ltd. commented
Status: Needs review » Needs work
simbaw’s picture

simbaw CreditAttribution: simbaw commented
Status: Needs work » Needs review
FileSize
draggableViews-add-readme-file-2877884-5.patch2.21 KB

@Jigar.addweb,Your issues has been fixed in this patch.
Thanks.

riddhi.addweb’s picture

riddhi.addweb CreditAttribution: riddhi.addweb at AddWeb Solution Pvt. Ltd. commented
Status: Needs review » Reviewed & tested by the community

Thanks Simbaw for solving issues, attached patch resolves the error and also space added after installation section is accomplished.

aldibier’s picture

aldibier
Primary language Spanish
Location Manizales
CreditAttribution: aldibier at SeeD EM commented

The last part of instructions in README "Things to confirm after you saved your new view." doesn't work for 8.x version.

  • iStryker committed 5c1d8f6 on 8.x-1.x authored by simbaw 
    Issue #2877884 by simbaw, Jigar.addweb: Adding README.md file
iStryker’s picture

iStryker CreditAttribution: iStryker commented
Status: Reviewed & tested by the community » Fixed

Commit #5. After this issue closes, please open a new issue if you want to make a change.

Status: Fixed » Closed (fixed)

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