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
#8 user_sanitize-updated-readme-3008368-8.patch1.69 KBneel24
#2 user_sanitize-readme-3008368-2.patch1.71 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 user_sanitize-readme-3008368-2.patch1.71 KB

I uploaded a patch that adds a README to the project, thanks!!

mattgill’s picture

mattgill commented

Unfortunately I have very little time to maintain my Drupal modules any more.

Because of this, I'll only be merging patches which add functionality or address bugs. With respect, I will not be merging these patches as they are purely textual (readmes, changes to one letter capitalisation etc)

If you wish to contribute functionality or fixes I'll gladly merge these patches along with it later.

mattgill’s picture

mattgill commented
Status: Needs review » Postponed
alonaoneill’s picture

alonaoneill commented

An extra space should be removed. 

+++ b/README.md
@@ -0,0 +1,64 @@
+    ¶

Extra space.

Thanks.

alonaoneill’s picture

alonaoneill commented
Status: Postponed » Needs work
dhirendra.mishra’s picture

dhirendra.mishra commented
Assigned: Unassigned » dhirendra.mishra

Working on it.

neel24’s picture

neel24 commented
StatusFileSize
new user_sanitize-updated-readme-3008368-8.patch1.69 KB

Created a new patch which adds the readme.

neel24’s picture

neel24 commented
Assigned: dhirendra.mishra » Unassigned
Status: Needs work » Needs review
alonaoneill’s picture

alonaoneill commented
Status: Needs review » 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!

andre.bonon’s picture

andre.bonon
Primary language Portuguese, Brazil
commented
Status: Reviewed & tested by the community » Fixed

Hi, we've merged your patch to the drupal 9 support ticket. So I'll close this issue and add a credit to you.
Thanks for your contribution.

Status: Fixed » Closed (fixed)

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