Support from Acquia helps fund testing for Drupal Acquia logo

Comments

soaratul created an issue. See original summary.

soaratul’s picture

Here is the patch for this.

soaratul’s picture

Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 2: message_notify-docblock-typo-2836818-2.patch, failed testing.

Shashwat Purav’s picture

Assigned: soaratul » Shashwat Purav
Shashwat Purav’s picture

Assigned: Shashwat Purav » Unassigned
Status: Needs work » Needs review
FileSize
505 bytes

Created patch. Fixed typo.

soaratul’s picture

@Shashwat Purav,

I found that both patch files posted by me and you are identical, still your patch results in pass but mine got failed.

Can you please review and suggest the mistakes that I made during creating and posting the patch, It will really be helpful for me.

zalak.addweb’s picture

@Shashwat Purav, Thanks for typo correction, but I found 1 more typo in message_notify.module file, I've attached the patch with the correction of "successded" to "succeeded".

Hope this helps too!

volkswagenchick’s picture

Status: Needs review » Reviewed & tested by the community

Patch applied via simplytest.me and the typos have been corrected.

Marking RTBC

bluegeek9’s picture

Status: Reviewed & tested by the community » Closed (outdated)