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

feat: implement wix analytics integration #248

Draft
wants to merge 9 commits into
base: development
Choose a base branch
from
Draft

Conversation

barak007
Copy link

@barak007 barak007 commented Nov 17, 2024

This pull request introduces a new analytics context to the application using WixBiAnalytics and integrates it into the main application component. The changes include creating a new context provider for Wix BI, updating the root component to use this provider, and adding a new dependency for the analytics library.

Open questions

  1. node - because we don't have persistent visitorId in the server yet I currently disabled events in node.
  2. dev - currently we null transport on localhost but we might want to be able to enable them?
  3. what should happen if a site does not have metasiteId - should we place MS_ID in the template env.
  4. browserSessionId - we want this value too. it's based on some server session logic we want to implement as part of Wix remix integration? or part of our netlify deploy? or with wix headless login session. (I'm not 100% sure about this one)
  5. cookie banner - we need to check what are the categories of policies we support in the BI event and align the UI.
  6. BO integration (open the analytics tab). check that traffic is visible.
  • maybe we want to disable events if there is not metasiteId (in the analytics lib)

@barak007 barak007 changed the title Barak/bi feat: implement wix analytics integration Nov 17, 2024
@@ -0,0 +1,51 @@
import React, { useEffect, useMemo } from 'react';
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should this be a part of the template? or we want to extract it?

@barak007 barak007 marked this pull request as draft November 17, 2024 11:53
@barak007 barak007 requested a review from alisey November 17, 2024 12:29
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.

1 participant