This isn't correct as the module is called "Content Synchronization".

Currently Says "Congratulations! You have successfully installed the Contact Sync module."

Should say "Congratulations! You have successfully installed the Content Sync module."

Spotted against 8.x-2.x

Thanks

Tom

CommentFileSizeAuthor
#8 patch.png12.74 KBvandna b
#6 After-install-the-module-I-see-Congratulations!-You-have-successfully-installed-the-Contact-Sync-module-2966514-6.patch697 bytestom m fallon
#3 After-install-the-module-I-see-Congratulations!-You-have-successfully-installed-the-Contact-Sync-module-2966514-2.patch708 bytestom m fallon

Comments

Tom M Fallon created an issue. See original summary.

tom m fallon’s picture

tom m fallon commented
Issue summary: View changes
tom m fallon’s picture

tom m fallon commented
StatusFileSize
new After-install-the-module-I-see-Congratulations!-You-have-successfully-installed-the-Contact-Sync-module-2966514-2.patch708 bytes

Hey

Here is a small patch to correct the name.

Thanks

Tom

tom m fallon’s picture

tom m fallon commented
Assigned: tom m fallon » Unassigned
tom m fallon’s picture

tom m fallon commented
Status: Active » Needs review
tom m fallon’s picture

tom m fallon commented
StatusFileSize
new After-install-the-module-I-see-Congratulations!-You-have-successfully-installed-the-Contact-Sync-module-2966514-6.patch697 bytes

Hey

In fact redoing patch. It should say "Contact Sync" as in the rest of the code its referred to as that.

Example of where we are referring to it as that.

"You have successfully updated to the 8.x-2.0 release of the Content Sync module. Learn more"

Updating issue to reflect this

tom m fallon’s picture

tom m fallon commented
Issue summary: View changes
vandna b’s picture

vandna b commented
StatusFileSize
new patch.png12.74 KB

#6 I have tested your patch which is working fine

john cook’s picture

john cook commented
Status: Needs review » Reviewed & tested by the community

I’ve had a look at the code and it looks good.

As it’s been tested by @vandna b I’m setting this to RTBC.

blanca.esqueda’s picture

blanca.esqueda commented

Good catch!
And thank you for the patch. It'll be applied to 8.x-2.x-dev.

blanca.esqueda’s picture

blanca.esqueda commented
Status: Reviewed & tested by the community » Fixed
blanca.esqueda’s picture

blanca.esqueda commented
Status: Fixed » Closed (fixed)