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

Remove loadSegment related logic (leveraged GTM) #1217

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

jasonbryant84
Copy link
Contributor

@jasonbryant84 jasonbryant84 commented Oct 3, 2024

Project Ticket

Introducing...

  • Segment Analytics-Next
  • Data dictionary
  • form_submit event
  • segment_consent_preference_updated event

Copy link

netlify bot commented Oct 3, 2024

Deploy Preview for condescending-goldwasser-91acf0 ready!

Name Link
🔨 Latest commit 2803fc1
🔍 Latest deploy log https://app.netlify.com/sites/condescending-goldwasser-91acf0/deploys/671b8160694a9c0008ded936
😎 Deploy Preview https://deploy-preview-1217--condescending-goldwasser-91acf0.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@trean trean requested a review from generall October 23, 2024 12:31
@trean
Copy link
Collaborator

trean commented Oct 23, 2024

Hey @jasonbryant84 should we review this already or is this WIP?

@trean trean removed the request for review from generall October 23, 2024 12:35
@jasonbryant84
Copy link
Contributor Author

Hey @jasonbryant84 should we review this already or is this WIP?

Thanks for asking/checking. We're waiting for an upgrade to our Segment subscription. So didn't want to bother anyone with reviewing until that's sorted as we'll need to change the writekey as well anyway. Can I ping you when we're ready?

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