Navigation Menu

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

Allow Conversion API to be disabled #2020

Open
rcstr opened this issue Jun 14, 2021 · 2 comments
Open

Allow Conversion API to be disabled #2020

rcstr opened this issue Jun 14, 2021 · 2 comments
Labels
feature: pixel events & conversion tracking Related to Facebook pixel, events, and conversion tracking. priority: low The issue/PR is low priority—not many people are affected or there’s a workaround, etc. type: enhancement The issue is a request for an enhancement.

Comments

@rcstr
Copy link
Member

rcstr commented Jun 14, 2021

WP.org thread: https://wordpress.org/support/topic/purchase-event-missing-some-deduplication-parameters-2/

Feature requests

Based on Conversions API docs the event is part of the Pixel implementation, which can be disabled via filter (facebook_for_woocommerce_integration_pixel_enabled) but this won't disable the requests to Conversion API.

It would be great to allow customers to disable Conversion API while Pixel API stills enable.

The code in charge of sending the events to Conversion API

public function send_pixel_events( $pixel_id, array $events ) {
$request = new API\Pixel\Events\Request( $pixel_id, $events );
$this->set_response_handler( Response::class );
return $this->perform_request( $request );
}

@rcstr rcstr added the type: enhancement The issue is a request for an enhancement. label Jun 14, 2021
@haszari haszari added the feature: pixel events & conversion tracking Related to Facebook pixel, events, and conversion tracking. label Jun 15, 2021
@haszari
Copy link
Member

haszari commented Jun 15, 2021

Just noting that this was feature request raised as a workaround for a related issue(s): duplicate events, or event deduplication not working correctly (see forum thread and possibly related issues #1668 #1722). Based on this I'm tagging as low priority for now.

@haszari haszari added the priority: low The issue/PR is low priority—not many people are affected or there’s a workaround, etc. label Jun 15, 2021
@josephbonanno74
Copy link

My client forwarded this issue to me today:

Issue: Missing Purchase Value Parameter
One or more of your Purchase events is missing a value parameter. This may affect your ability to see return on ad spend calculations.

AND

Issue: InitiateCheckout Event Missing Some Deduplication Parameters
To deduplicate identical events received by both your pixel and the Conversions API, we need to identify them by cross-checking a matching deduplication key in both sources.
Some of your InitiateCheckout events don't have a deduplication key, so we're unable to cross-check whether they're duplicate events. This means they're potentially being incorrectly counted twice and leading to inaccurate ad attribution.

AND

Issue: Purchase Event Missing Some Deduplication Parameters
To deduplicate identical events received by both your pixel and the Conversions API, we need to identify them by cross-checking a matching deduplication key in both sources.
Some of your Purchase events don't have a deduplication key, so we're unable to cross-check whether they're duplicate events. This means they're potentially being incorrectly counted twice and leading to inaccurate ad attribution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature: pixel events & conversion tracking Related to Facebook pixel, events, and conversion tracking. priority: low The issue/PR is low priority—not many people are affected or there’s a workaround, etc. type: enhancement The issue is a request for an enhancement.
Projects
None yet
Development

No branches or pull requests

3 participants