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

[0.2.0] Add support for Web Pixel events #57

Merged
merged 18 commits into from
Jan 31, 2024
Merged

[0.2.0] Add support for Web Pixel events #57

merged 18 commits into from
Jan 31, 2024

Conversation

markmur
Copy link
Contributor

@markmur markmur commented Jan 26, 2024

What are you trying to accomplish?

Adds support for Web Pixel events

  • Swift
  • Android
const shopify = useShopifyCheckoutSheet();

useEffect(() => {
  const pixel = shopify.addEventListener('pixel', (event: PixelEvent) => {
      // Dispatch web pixel events to third-party services
      if (hasPermissionToTrack) {
        sendEventToAnalyticsProvider(event);
      }
    },
  );
}, [shopify])

Before you deploy

  • I have added tests to support my implementation
  • I have read and agree with the contributing documentation readme
  • I have read and agree with the code of conduct documentation readme
  • I have updated any documentation related to these changes.
  • I have updated the README (if applicable).

@markmur markmur requested a review from a team as a code owner January 26, 2024 15:18
@markmur markmur self-assigned this Jan 26, 2024
@markmur markmur changed the title Add support for Web Pixel events [0.2.0] Add support for Web Pixel events Jan 26, 2024
@markmur markmur force-pushed the markmur/web-pixels branch from 9d1b082 to b36ea87 Compare January 30, 2024 10:37
@markmur markmur force-pushed the markmur/web-pixels branch from b36ea87 to ade76d8 Compare January 30, 2024 10:38
@markmur markmur force-pushed the markmur/web-pixels branch from b2046a9 to 13873c9 Compare January 30, 2024 18:25
@markmur markmur force-pushed the markmur/web-pixels branch 2 times, most recently from ced3bb1 to 692576a Compare January 31, 2024 09:15
@markmur markmur force-pushed the markmur/web-pixels branch from 692576a to d22debe Compare January 31, 2024 09:17
@markmur markmur merged commit 5cf72ea into main Jan 31, 2024
7 checks passed
@markmur markmur deleted the markmur/web-pixels branch January 31, 2024 09:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants