Skip to content

feat: adds asset picker opened metric event#7985

Open
ghgoodreau wants to merge 3 commits intomainfrom
SWAPS-2559-asset-picker-opened-event
Open

feat: adds asset picker opened metric event#7985
ghgoodreau wants to merge 3 commits intomainfrom
SWAPS-2559-asset-picker-opened-event

Conversation

@ghgoodreau
Copy link
Contributor

@ghgoodreau ghgoodreau commented Feb 18, 2026

Explanation

Adds an AssetPickerOpened metric event.

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

Low Risk
Metrics-only/type updates with minimal runtime behavior change, limited to event-name plumbing and property composition.

Overview
Adds a new UnifiedSwapBridgeEventName.AssetPickerOpened metrics event and types its required payload to include asset_location: 'source' | 'destination'.

Updates BridgeController.#getEventProperties to treat this event as controller-state-independent (returns only base/client properties) and documents the addition in the bridge-controller changelog.

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

@ghgoodreau ghgoodreau marked this pull request as ready for review February 18, 2026 23:25
@ghgoodreau ghgoodreau requested review from a team as code owners February 18, 2026 23:25
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

Comments