The Readme must must be formatted as specified by README template : https://www.drupal.org/docs/develop/documenting-your-project/readme-temp...

CommentFileSizeAuthor
#6 3069876-6.patch2.22 KBshubham.prakash
#4 3069876-4.patch2.21 KBshubham.prakash
#2 3069876-2.patch2.1 KBshubham.prakash
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

shubham.prakash created an issue. See original summary.

shubham.prakash’s picture

shubham.prakash CreditAttribution: shubham.prakash at OpenSense Labs commented
Assigned: shubham.prakash » Unassigned
Status: Active » Needs review
FileSize
3069876-2.patch2.1 KB

This patch will fix the issue.

alonaoneill’s picture

alonaoneill CreditAttribution: alonaoneill at Hook 42 commented
Status: Needs review » Needs work 
  1. +++ b/Readme.md
@@ -1,26 +1,55 @@
 MSG91 SMS provides a block to send SMS form Drupal website.It provides easy integration of msg91 to be used in India.
...
+ * Install this module as you would normally install a contributed Drupal module.
...
+   3. Go to admin/configuration/smsbymsg91 and enter values of Authentication Key,

    Text should manually word-wrap within around 80 cols. 

  2. +++ b/Readme.md
@@ -1,26 +1,55 @@
+   2. Install and enable this module. ¶

    Extra space should be removed

shubham.prakash’s picture

shubham.prakash CreditAttribution: shubham.prakash at OpenSense Labs commented
Status: Needs work » Needs review
FileSize
3069876-4.patch2.21 KB

Thanks for the review, this patch should fix the issue.

alonaoneill’s picture

alonaoneill CreditAttribution: alonaoneill at Hook 42 commented
Status: Needs review » Needs work 
+++ b/Readme.md
@@ -1,26 +1,56 @@
+ * Install this module as you would normally install a contributed Drupal module.
...
+   3. Go to admin/configuration/smsbymsg91 and enter values of Authentication Key,

Text should manually word-wrap within around 80 cols.

shubham.prakash’s picture

shubham.prakash CreditAttribution: shubham.prakash at OpenSense Labs commented
Status: Needs work » Needs review
FileSize
3069876-6.patch2.22 KB
alonaoneill’s picture

alonaoneill CreditAttribution: alonaoneill at Hook 42 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!

gaurav.kapoor’s picture

gaurav.kapoor CreditAttribution: gaurav.kapoor commented
Status: Reviewed & tested by the community » Fixed
gaurav.kapoor’s picture

gaurav.kapoor CreditAttribution: gaurav.kapoor commented

Status: Fixed » Closed (fixed)

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