Skip to content

Added WordPress installation guide for no-data and tracking code pages #20630

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 16 commits into from
Apr 30, 2023

Conversation

AltamashShaikh
Copy link
Contributor

Description:

Added WordPress installation guide for no-data and tracking code pages.
Fixes: #PG-2737

Review

@AltamashShaikh AltamashShaikh marked this pull request as draft April 25, 2023 04:54
@AltamashShaikh AltamashShaikh marked this pull request as ready for review April 27, 2023 06:03
@AltamashShaikh AltamashShaikh added the Needs Review PRs that need a code review label Apr 27, 2023
@AltamashShaikh AltamashShaikh requested review from snake14 and bx80 April 27, 2023 06:03
Copy link
Contributor

@snake14 snake14 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. The main issue I saw was that the Matomo Url is partially unreadable:
image

@@ -158,6 +158,31 @@
"SiteWithoutDataGoogleTagManagerFollowStep9": "Preview your changes by clicking the \"Preview\" button.",
"SiteWithoutDataGoogleTagManagerFollowStep10": "Check if the Matomo tag is firing as expected.",
"SiteWithoutDataGoogleTagManagerFollowStep11": "Publish your changes by clicking the \"Submit\" button and then the \"Publish\" button at the top right.",
"SiteWithoutDataGoogleTagManagerFollowStepCompleted": "%1$sCongratulations!%2$s You have successfully installed the Matomo Analytics tracking code via Google Tag Manager. To verify that hits are being tracked, visit your website and check that this data is visible in your Matomo instance."
"SiteWithoutDataGoogleTagManagerFollowStepCompleted": "%1$sCongratulations!%2$s You have successfully installed the Matomo Analytics tracking code via Google Tag Manager. To verify that hits are being tracked, visit your website and check that this data is visible in your Matomo instance.",
"WordpressDetected": "Wordpress was detected on your website. Did you know you can use the Matomo easy WordPress plugin to track the website data? %1$sLearn more in this guide%2$s.",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The phrase the Matomo easy WordPress plugin sounds a little funny. I think that the easy Matomo WordPress plugin or maybe you can easily use the Matomo plugin for WordPress.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep, this would be great to have reworded.
I would also suggest to ask the product team to review the whole content by providing them with a screenshot of the UI in the internal ticket.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@snake14 Updated as suggested and @michalkleiner The text was provided by the product team only

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. The main issue I saw was that the Matomo Url is partially unreadable: image

@snake14 This looks specific to your setup works fine for me, even the UI screenshot seems to display it correctly

Copy link
Contributor

@bx80 bx80 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left a couple of optional minor comments, but other than that this looks good to me 👍

@bx80 bx80 merged commit 6174cb7 into 5.x-dev Apr 30, 2023
@bx80 bx80 deleted the PG-2737-wordpress-no-data branch April 30, 2023 21:25
AltamashShaikh added a commit that referenced this pull request May 2, 2023
#20630)

* Added Wordpress installation guide for no-data and tracking code pages, #PG-2737

* UI screenshot updated

* Updated UI steps as per design

* Screenshot updated

* Removed instructions in integration tab for WP

* UI screenshot updated

* Steps text updated

* Final textual changes

* Fixes test cases

* PR changes applied

* UI screenshot updated

* PR changes

* Refactired usages of wordpressUsed to cms

* Updated screenshot

* Updated screenshot

* Update UI screenshot
bx80 pushed a commit that referenced this pull request May 3, 2023
#20649)

* Added WordPress installation guide for no-data and tracking code pages (#20630)

* Added Wordpress installation guide for no-data and tracking code pages, #PG-2737

* UI screenshot updated

* Updated UI steps as per design

* Screenshot updated

* Removed instructions in integration tab for WP

* UI screenshot updated

* Steps text updated

* Final textual changes

* Fixes test cases

* PR changes applied

* UI screenshot updated

* PR changes

* Refactired usages of wordpressUsed to cms

* Updated screenshot

* Updated screenshot

* Update UI screenshot

* Added WordPress installation guide for no-data and tracking code pages, #PG-2737

* Update FAQ link

* Update link

---------

Co-authored-by: Ben <ben.burgess@innocraft.com>
@sgiehl sgiehl added this to the 5.0.0 milestone May 16, 2023
@sgiehl sgiehl added not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org. and removed Needs Review PRs that need a code review labels May 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org.
Development

Successfully merging this pull request may close these issues.

5 participants