Skip to content

Conversation

@ghgoodreau
Copy link
Contributor

@ghgoodreau ghgoodreau commented Jan 7, 2026

Explanation

References

Checklist

  • I've updated the test suite for new or updated code as appropriate
  • I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
  • I've communicated my changes to consumers by updating changelogs for packages I've changed
  • I've introduced breaking changes in this PR and have prepared draft pull requests for clients and consumer packages to resolve them

Note

Adds tracking for asset picker interactions in cross-chain swaps.

  • Introduces Asset Picker Opened and Asset Selected events in utils/metrics/constants.ts
  • Extends RequiredEventContextFromClient with payloads for these events (location, token/chain identifiers) and updates EventPropertiesFromControllerState to null for controller-derived props
  • Wires bridge-controller to emit these events with base properties via #getEventProperties

Written by Cursor Bugbot for commit 94ae82f. This will update automatically on new commits. Configure here.

@ghgoodreau ghgoodreau requested a review from a team as a code owner January 7, 2026 16:49
@ghgoodreau ghgoodreau marked this pull request as draft January 7, 2026 16:49
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