While evaluating this module, I noticed it lacked a readme.
According to Drupal standards, modules should include a useful README.file.
https://www.drupal.org/docs/develop/documenting-your-project/module-docu....

https://www.drupal.org/docs/develop/documenting-your-project/readme-temp...

Patch to follow, thanks!

CommentFileSizeAuthor
#4 interdiff-3013358-2-4.txt530 bytesvolkswagenchick
#4 content_export_csv-readme-3013358-4.patch1.41 KBvolkswagenchick
#3 Screen Shot 2018-11-13 at 9.35.35 AM.png27.13 KBalonaoneill
#2 content_export_csv-README-3013358-2.patch1.41 KBvolkswagenchick

Comments

volkswagenchick created an issue. See original summary.

volkswagenchick’s picture

volkswagenchick
she/her
Primary language English
Location San Francisco Bay Area
commented
Assigned: volkswagenchick » Unassigned
Status: Active » Needs review
StatusFileSize
new content_export_csv-README-3013358-2.patch1.41 KB

I uploaded a path with a useful readme. Thanks!!

alonaoneill’s picture

alonaoneill commented
Status: Needs review » Needs work
StatusFileSize
new Screen Shot 2018-11-13 at 9.35.35 AM.png27.13 KB 
+++ b/README.txt
@@ -1 +1,54 @@
+    2. Navigate to Administration > Webservices > Content export for

Schould be Web services
Screen added.
Thanks

volkswagenchick’s picture

volkswagenchick
she/her
Primary language English
Location San Francisco Bay Area
commented
Status: Needs work » Needs review
StatusFileSize
new content_export_csv-readme-3013358-4.patch1.41 KB
new interdiff-3013358-2-4.txt530 bytes

I corrected the path and also caught another typo. New patch uploaded along with an interdiff, thanks.

echo15’s picture

echo15 commented
Status: Needs review » Reviewed & tested by the community

Looks good.
README file is now according to Drupal standards
Thank you for fixing.

kothandapani48’s picture

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

Status: Fixed » Closed (fixed)

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