Skip to content

ECTO-2581 - Add tracking for extension store checkout - #144

Merged
Jannik (qysp) merged 1 commit into
epic/ECTO-2267from
task/ECTO-2581
Apr 27, 2026
Merged

Jannik (qysp) merged 1 commit into
epic/ECTO-2267from
task/ECTO-2581

Conversation

@qysp

Copy link
Copy Markdown
Collaborator

No description provided.

@codecov-commenter

Codecov Comments Bot (codecov-commenter) commented Apr 17, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 19.29825% with 46 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (epic/ECTO-2267@10d57a2). Learn more about missing BASE report.

Files with missing lines Patch % Lines
...n-store/service/extension-store-channel.service.ts 0.00% 22 Missing ⚠️
...ore/extension-store-purchase-confirmation.store.ts 0.00% 11 Missing ⚠️
...rc/Resources/app/administration/src/vite.config.ts 0.00% 9 Missing ⚠️
...nsion-store/store/extension-store-context.store.ts 33.33% 2 Missing ⚠️
...Resources/app/administration/src/util/telemetry.ts 80.00% 2 Missing ⚠️
Additional details and impacted files
@@                Coverage Diff                @@
##             epic/ECTO-2267     #144   +/-   ##
=================================================
  Coverage                  ?   51.81%           
=================================================
  Files                     ?       37           
  Lines                     ?      911           
  Branches                  ?      190           
=================================================
  Hits                      ?      472           
  Misses                    ?      423           
  Partials                  ?       16           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds telemetry/tracking hooks around the Extension Store iframe lifecycle, routing updates, and checkout flow, including passing version metadata through to analytics.

Changes:

  • Injects the extension version into the admin build via Vite define and a new global constant.
  • Introduces a shared telemetry helper and an Extension Store context store to enrich events (e.g., SkyBridge store version).
  • Tracks key events in the channel service, purchase confirmation flow, and entry/exit of the Extension Store page; adds data-analytics-source to relevant links.

Reviewed changes

Copilot reviewed 11 out of 12 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
src/Resources/app/administration/src/vite.config.ts Reads root composer.json version and defines __SWAG_EXTENSION_STORE_VERSION__ for runtime telemetry enrichment.
src/Resources/app/administration/src/util/telemetry.ts Adds a telemetry wrapper that attaches default Extension Store metadata to all events.
src/Resources/app/administration/src/module/sw-extension-store/store/extension-store-purchase-confirmation.store.ts Tracks purchase lifecycle events (initiated/confirmed/success/failed/cancelled).
src/Resources/app/administration/src/module/sw-extension-store/store/extension-store-context.store.ts Stores SkyBridge store version for inclusion in telemetry.
src/Resources/app/administration/src/module/sw-extension-store/service/extension-store-channel.service.ts Adds tracking-related channel actions and emits connection + page view telemetry.
src/Resources/app/administration/src/module/sw-extension-store/page/sw-extension-store-index/index.js Tracks entering/leaving the Extension Store page.
src/Resources/app/administration/src/module/sw-extension-store/component/...checkout.html.twig Adds analytics source attribute and adjusts permissions link element type.
src/Resources/app/administration/src/module/sw-extension-store/component/...checkout-overview.html.twig Adds analytics source attribute to external links.
src/Resources/app/administration/src/global.types.ts Declares the global version constant for TS type-checking.
src/Resources/app/administration/package.json Bumps @shopware-ag/meteor-admin-sdk to ^6.8.0.
src/Resources/app/administration/package-lock.json Updates lockfile for the SDK bump and new transitive deps.
src/Resources/app/administration/jest.config.js Adds a Jest global for __SWAG_EXTENSION_STORE_VERSION__.
Files not reviewed (1)
  • src/Resources/app/administration/package-lock.json: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@qysp
Jannik (qysp) merged commit 9857ef3 into epic/ECTO-2267 Apr 27, 2026
3 checks passed
@qysp
Jannik (qysp) deleted the task/ECTO-2581 branch April 27, 2026 08:10
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.

5 participants