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.

Sign up for GitHub

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

Jump to bottom

Support for add to cart pixel on ajax add to cart #2

Closed
helgatheviking opened this issue Jul 27, 2017 · 5 comments
Closed

Support for add to cart pixel on ajax add to cart #2

helgatheviking opened this issue Jul 27, 2017 · 5 comments
Labels
help wanted

Comments

@helgatheviking
Copy link
Contributor

The pixel doesn't seem to fire after an item has been added to the cart asynchronously from the archive pages.
@sanderkie sanderkie mentioned this issue Sep 15, 2017
Closed
@dmitridr
Copy link
Contributor

We definitely don't fire the pixel on AJAX add to cart. We could support this in the future based on demand for this feature, it would be a big change for us though.

@dmitridr dmitridr mentioned this issue Sep 18, 2017
Closed
@helgatheviking
Copy link
Contributor Author

I ended up writing it into my own plugin. I don't have time for a PR right now, but I could probably paste some snippets of you wanted.

@dmitridr
Copy link
Contributor

Definitely interested in how you solved this! Would love to see some snippets.

@helgatheviking
Copy link
Contributor Author

Pulling very heavily from your plugin, I added some code into the "base pixel" that fires a second little ajax request when the ajax add to cart JS trigger fires
https://github.com/AltoLabs/snappic-woocommerce/blob/master/includes/class-snappic-pixel.php#L34-L47

That builds the new pixel event and sends it back to the script to be injected in the <head>
https://github.com/AltoLabs/snappic-woocommerce/blob/master/includes/class-snappic-pixel-display.php#L98-L109

Hope that helps!

@addedlovely addedlovely mentioned this issue Sep 22, 2017
Closed
@dmitridr
Copy link
Contributor

dmitridr commented Oct 9, 2017

Thanks for that! I would definitely like to set aside some time to merge this into our repo but we're dealing with a large number of bugs right now. If you ever have time to send that PR we could review it pretty quickly though! Please consider it.

@metuza metuza mentioned this issue Nov 30, 2017
Closed
@sm7uge sm7uge mentioned this issue Jan 26, 2018
Closed
@camilosw camilosw mentioned this issue Feb 9, 2018
Closed
This was referenced Apr 27, 2018
Closed
Closed
Closed
Closed
@MillenniumJim MillenniumJim mentioned this issue Apr 27, 2018
Closed
Closed
@iljamm iljamm mentioned this issue May 1, 2018
Closed
@auludag auludag mentioned this issue Nov 19, 2018
Closed
@JapeNZ JapeNZ mentioned this issue Dec 27, 2018
Closed
@FifthCreations FifthCreations mentioned this issue Mar 8, 2019
Closed
@hatem21 hatem21 mentioned this issue Mar 26, 2019
Closed
@dutchuy72 dutchuy72 mentioned this issue Apr 18, 2019
Closed
@codegeek1001 codegeek1001 mentioned this issue Oct 15, 2019
Closed
@kidpluszmedia kidpluszmedia mentioned this issue Mar 12, 2020
Closed
@anderslundyo anderslundyo mentioned this issue May 20, 2020
Closed
deivamagalhaes pushed a commit that referenced this issue May 21, 2020
@unfulvio
Fix critical issues in facebook-for-woocommerce.php (#2)
ea85079 
Fix critical issues in facebook-for-woocommerce.php
deivamagalhaes pushed a commit that referenced this issue May 21, 2020
@wvega
Merge pull request #2 from skyverge/ch25109/add-to-cart-event-tacking…
71c6e5e 
…-fixes

Improve tracking consistency of AddToCart and ViewContent events
@JossPer0 JossPer0 mentioned this issue Nov 13, 2020
Closed
@rubenp152 rubenp152 mentioned this issue Dec 9, 2020
Closed
@hermosabeach hermosabeach mentioned this issue Dec 10, 2020
Closed
@jnz31 jnz31 mentioned this issue May 10, 2021
Closed
2 tasks
@TristanVelop TristanVelop mentioned this issue Dec 7, 2021
Closed
@HerveKL HerveKL mentioned this issue Dec 18, 2021
Closed
2 tasks
@Klausdks Klausdks mentioned this issue Aug 3, 2022
Closed
2 tasks
@TimMcCauley TimMcCauley mentioned this issue Sep 25, 2022
Closed
2 tasks
@Mikroo32 Mikroo32 mentioned this issue Oct 14, 2022
Closed
2 tasks
@Fidget656 Fidget656 mentioned this issue Oct 14, 2022
Closed
@micolf micolf mentioned this issue Jan 20, 2023
Closed
@rawdreeg rawdreeg mentioned this issue Aug 3, 2023
Open
2 tasks
budzanowski pushed a commit that referenced this issue Oct 17, 2023
@rahulraina7
Merge pull request #2 from woocommerce/develop
88d3480 
Merge fork
@davidhefti davidhefti mentioned this issue Jan 3, 2024
Closed
@kozbilek kozbilek mentioned this issue Jan 19, 2024
Closed
@angelosleebos angelosleebos mentioned this issue Feb 1, 2024
Closed
brezocordero added a commit that referenced this issue Jul 26, 2024
@brezocordero
Fix for WordPress.WP.DeprecatedParameters.Get_termsParam2Found
e3b490b 
The parameter at position #2 of get_terms() has been deprecated since WordPress version 4.5.0.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
help wanted
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@helgatheviking @dmitridr