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!

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

volkswagenchick created an issue. See original summary.

volkswagenchick’s picture

Assigned: volkswagenchick » Unassigned
Status: Active » Needs review

I uploaded a patch that adds a readme to the project. Thanks!

volkswagenchick’s picture

Helps if i attach the patch :)

Status: Needs review » Needs work

The last submitted patch, 3: snapengage-readme-3012372-2.patch, failed testing. View results

alonaoneill’s picture

Status: Needs work » Reviewed & tested by the community

Reviewed the patch for spelling and grammar. Formatting aligns to Drupal documentation standards. Everything looks good. Patch applied.
Marking as RTBCed.
Thank you for working on module!

arnested’s picture

Status: Reviewed & tested by the community » Fixed

Thank you for the patch!

Sorry to have missed it for so long. The patch is applied.

Status: Fixed » Closed (fixed)

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