Skip to content
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

Pinterest Conversions API #787

Merged

Conversation

message-dimke
Copy link
Contributor

@message-dimke message-dimke commented Jun 30, 2023

Changes proposed in this Pull Request:

Closes #771 .

Adds Conversions API implementation as well as Pinterest Tag implementation refactoring. Now we have a general Tracking class which hooks into events of interest and had trackers attacked to handle action accordingly.

e.g. Tracking has two tracker implementations: Tag and Conversions. Tracking class hooks into add to cart event and when action is fired it notifies trackers about the event so the last could perform their corresponding actions.

Conversions API events are logged under pinterest-for-woocommerce-conversions-<date> log files so the user could see what was logged.

WooCommerce_status_‹_WordPress_Pinterest_—_WordPress

Pinterest has events log which shows events receive through Tag and Conversion APIs. It is located under Ads -> Conversions

_1__Pinterest Cursor_and__1__Pinterest

Unfortunately I was not able to find where Pinterest stores any event details we report. Could be it is not yet implemented or something.

Detailed test instructions:

  1. Checkout add/pinterest-conversions branch
  2. Connect to your Pinterest Account.
  3. Navigate the website, visit product pages, perform product searches, add to cart and checkout events.
  4. Check WooCommerce conversion logs and Pinterest Tag Helper while navigating the website (logs will show you data reported through Conversion APIs, Pinterest Tag Helper will show you data reported through it).
  5. Visit Pinterest Dashboard, the page under Ads > Conversions to see events reported.

Additional details:

Changelog entry

Add - Pinterest Conversions API implementation.

@message-dimke message-dimke self-assigned this Jun 30, 2023
@github-actions github-actions bot added changelog: add A new feature, function, or functionality was added. type: enhancement The issue is a request for an enhancement. labels Jun 30, 2023
src/Tracking.php Outdated Show resolved Hide resolved
@message-dimke message-dimke requested a review from a team August 8, 2023 18:05
@message-dimke message-dimke marked this pull request as ready for review August 8, 2023 19:51
class-pinterest-for-woocommerce.php Outdated Show resolved Hide resolved
class-pinterest-for-woocommerce.php Outdated Show resolved Hide resolved
src/Tracking/Conversions.php Outdated Show resolved Hide resolved
src/Tracking/Conversions.php Outdated Show resolved Hide resolved
src/Tracking/Conversions.php Outdated Show resolved Hide resolved
src/Tracking/Data/Category.php Outdated Show resolved Hide resolved
src/Tracking/Data/Checkout.php Outdated Show resolved Hide resolved
src/Tracking/Data/Product.php Outdated Show resolved Hide resolved
src/Tracking/Tag.php Show resolved Hide resolved
src/Tracking/Tag.php Outdated Show resolved Hide resolved
@budzanowski
Copy link
Collaborator

Looks very nice after all of the refactors. Great job.

@budzanowski
Copy link
Collaborator

We can merge!

@message-dimke message-dimke merged commit ff326f6 into pinterest-v5-integration-branch Aug 30, 2023
10 checks passed
@message-dimke message-dimke deleted the add/pinterest-conversions branch August 30, 2023 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog: add A new feature, function, or functionality was added. type: enhancement The issue is a request for an enhancement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants