Skip to content

feat(core): simplify analytics provider#2662

Merged
jorgemoya merged 1 commit intocanaryfrom
fix-analytics-provider
Oct 31, 2025
Merged

feat(core): simplify analytics provider#2662
jorgemoya merged 1 commit intocanaryfrom
fix-analytics-provider

Conversation

@jorgemoya
Copy link
Copy Markdown
Contributor

What/Why?

Replace StreamableAnalyticsProvider with simplified AnalyticsProvider component.

  • Removed StreamableAnalyticsProvider that used Streamable.from() for async data loading
  • Added new AnalyticsProvider component that accepts channelId and settings as direct props
  • Simplifies analytics initialization by removing unnecessary streaming complexity
  • Maintains same functionality with cleaner, more straightforward implementation
  • Fixes issue of events not triggering by properly wrapping children inside the provider.

Testing

Tested by adding a product to cart.

Before:
Screenshot 2025-10-31 at 12 53 19 PM

After:
Screenshot 2025-10-31 at 12 52 40 PM

Migration

  • Use new AnalyticsProvider component in core/app/[locale]/layout.tsx, instead of StreamableAnalyticsProvider.

@jorgemoya jorgemoya requested a review from a team October 31, 2025 18:10
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Oct 31, 2025

🦋 Changeset detected

Latest commit: 05255a4

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@bigcommerce/catalyst-core Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel
Copy link
Copy Markdown

vercel Bot commented Oct 31, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
catalyst-b2b Ready Ready Preview Comment Oct 31, 2025 6:11pm
catalyst-canary Ready Ready Preview Comment Oct 31, 2025 6:11pm
1 Skipped Deployment
Project Deployment Preview Comments Updated (UTC)
catalyst Ignored Ignored Oct 31, 2025 6:11pm

@jorgemoya jorgemoya added this pull request to the merge queue Oct 31, 2025
Merged via the queue into canary with commit 8c6626e Oct 31, 2025
12 checks passed
@jorgemoya jorgemoya deleted the fix-analytics-provider branch October 31, 2025 19:23
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