-
Notifications
You must be signed in to change notification settings - Fork 22
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
Send productClick events to PixelManager #128
Conversation
ace9227
to
85a48b9
Compare
7b22cda
to
191c5f0
Compare
5c8f985
to
37c39f8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
fe0db1d
to
eae122d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You started to export two ShelfItems, one with Pixel (as default) and the other without pixel, and within your code, you were importing only the version without Pixel.
Remove the named export and always import ShelfItem as default.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add a linked workspace? That way we can test the solution
248acc3
to
751f07d
Compare
@@ -0,0 +1,9 @@ | |||
import React from 'react' | |||
|
|||
export function Pixel(Comp) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why did u add just the mock and not the test?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added the mock because some tests were breaking after my changes.
751f07d
to
f5fa04b
Compare
What problem is this solving?
Sending productClick event to Pixel Manager with the purpose of usage in google-analytics.
Types of changes
Workspace: https://aguaviva--storecomponents.myvtex.com/