According to Drupal standards, modules should include a useful README.file.

CommentFileSizeAuthor
#8 3070796-8.patch1.91 KBshubham.prakash
#7 3070796-7.patch1.15 KBravi.shankar
#2 3070796-2.patch1.13 KBravi.shankar

Comments

anmolgoyal74 created an issue. See original summary.

ravi.shankar’s picture

ravi.shankar commented
Status: Active » Needs review
StatusFileSize
new 3070796-2.patch1.13 KB

I have fixed it.

alonaoneill’s picture

alonaoneill commented
Status: Needs review » Needs work

Configuration part is not complete. Needs more information.
Thanks

anmolgoyal74’s picture

anmolgoyal74 commented

I think the configuration part is not even required. There is nothing to configure.

alonaoneill’s picture

alonaoneill commented
Issue summary: View changes

According to Drupal standards, modules should include a useful README.file.

alonaoneill’s picture

alonaoneill commented
Issue summary: View changes

The configuration section (required) is necessary even when little configuration is required. Use this section to list special notes about the configuration of this module – including but not limited to permissions. This section is particularly important if the module requires additional configuration outside of the Drupal UI.

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

ravi.shankar’s picture

ravi.shankar commented
Status: Needs work » Needs review
StatusFileSize
new 3070796-7.patch1.15 KB

I have modified.

shubham.prakash’s picture

shubham.prakash commented
StatusFileSize
new 3070796-8.patch1.91 KB

This patch should fix the issue.

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!

anmolgoyal74’s picture

anmolgoyal74 commented
anmolgoyal74’s picture

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

Status: Fixed » Closed (fixed)

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