feat: track Token Details secondary action buttons (ASSETS-3212) cp-7.80.0 by Prithpal-Sooriya · Pull Request #30379 · MetaMask/metamask-mobile

Prithpal-Sooriya · 2026-05-19T13:34:23Z

Description

Adds Token Details Action Clicked analytics event instrumentation for the Token Details Page (TDP) secondary action buttons, enabling product to track how users interact with actions beyond the primary CTA.

Segment Schema: https://github.com/Consensys/segment-schema/pull/577

What changed:

New event: Token Details Action Clicked registered in MetaMetrics.events.ts
New enum: TokenDetailsAction with values: send, receive, more_opened, remove_token, view_on_explorer, copy_token_address
New tracking hook: useTokenDetailsActionTracking — accepts token params, balance, and severity; returns a stable callback that fires the event
Instrumented components:
- TokenDetailsActions — fires on Send, Receive, and More (menu open) button presses
- MoreTokenActionsMenu — fires on Remove Token and View on Block Explorer
- TokenDetailsList → Copy Token Address button

Event properties:

Property	Type	Description
`action`	enum	`send`, `receive`, `more_opened`, `remove_token`, `view_on_explorer`, `copy_token_address`
`token_symbol`	string	e.g. ETH
`token_address`	string	Token contract address
`chain_id`	string	Chain ID
`has_balance`	boolean	Whether user holds the token
`severity`	string	Security classification: Verified, Benign, Warning, Spam, Malicious
`source`	string	Mirrors `Token Details Opened` source enum

Changelog

CHANGELOG entry: null

Related issues

Fixes: https://consensyssoftware.atlassian.net/browse/ASSETS-3212

Manual testing steps

Feature: Token Details Action Clicked analytics

  Scenario: user taps Send on Token Details Page
    Given user is viewing a token with balance on Token Details Page

    When user taps the Send button
    Then "Token Details Action Clicked" event fires with action="send"

  Scenario: user taps Receive on Token Details Page
    Given user is viewing a token on Token Details Page

    When user taps the Receive button
    Then "Token Details Action Clicked" event fires with action="receive"

  Scenario: user taps More menu on Token Details Page
    Given user is viewing a token on Token Details Page

    When user taps the More (⋯) button
    Then "Token Details Action Clicked" event fires with action="more_opened"

  Scenario: user taps View on Block Explorer in More menu
    Given user has opened the More menu on Token Details Page

    When user taps View on Block Explorer
    Then "Token Details Action Clicked" event fires with action="view_on_explorer"

  Scenario: user taps Remove Token in More menu
    Given user has opened the More menu for a non-native token

    When user taps Remove Token
    Then "Token Details Action Clicked" event fires with action="remove_token"

  Scenario: user copies token contract address
    Given user is viewing token details section with contract address

    When user taps the copy address button
    Then "Token Details Action Clicked" event fires with action="copy_token_address"

Screenshots/Recordings

https://www.loom.com/share/73dce87dc6bc47b48a0d40588213c4e1

Pre-merge author checklist

I've followed MetaMask Contributor Docs and MetaMask Mobile Coding Standards.
I've completed the PR template to the best of my ability
I've included tests if applicable
I've documented my code using JSDoc format if applicable
I've applied the right labels on the PR (see labeling guidelines). Not required for external contributors.

Implements ASSETS-3212 - tracks Send, Receive, More (menu open), Remove Token, View on Explorer, and Copy Token Address actions on the Token Details Page. Event schema: - action: send | receive | more_opened | remove_token | view_on_explorer | copy_token_address - token_symbol, token_address, chain_id, has_balance, severity, source Co-authored-by: Prithpal Sooriya <prithpal.sooriya@gmail.com>

Tests all 6 action types (send, receive, more_opened, remove_token, view_on_explorer, copy_token_address) and source fallback behavior. Co-authored-by: Prithpal Sooriya <prithpal.sooriya@gmail.com>

github-actions · 2026-05-19T13:34:36Z

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

Co-authored-by: Prithpal Sooriya <prithpal.sooriya@gmail.com>

Renamed from 'Token Details Action Tapped' to 'Token Details Action Clicked' per schema alignment. Co-authored-by: Prithpal Sooriya <prithpal.sooriya@gmail.com>

Covers TokenDetailsActions (send/receive/more), MoreTokenActionsMenu (view_on_explorer/remove_token), and TokenDetailsList (copy address) to satisfy SonarQube 80% new-code coverage gate. Co-authored-by: Prithpal Sooriya <prithpal.sooriya@gmail.com>

sonarqubecloud · 2026-05-20T07:49:18Z

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
94.4% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

…r/assets-3212-token-details-action-tapped-cea8

…ed-cea8

juanmigdr

LGTM! 🚀 Just missing link to segment

cursor

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 98b8149. Configure here.}

…ed-cea8

github-actions · 2026-06-01T16:26:58Z

🔍 Smart E2E Test Selection

Selected E2E tags: SmokeWalletPlatform
Selected Performance tags: None (no tests recommended)
Risk Level: low
AI Confidence: 85%

click to see 🤖 AI reasoning details

E2E Test Selection:
The changes are purely analytics/tracking additions to the Token Details screen:

MetaMetrics.events.ts: Adds a new TOKEN_DETAILS_ACTION_CLICKED event (additive, no breaking changes)
constants/constants.ts: Adds a new TokenDetailsAction enum (additive)
useTokenDetailsActionTracking.ts: New hook for tracking token detail actions (new file)
AssetOverviewContent.tsx: Integrates the tracking hook and passes optional callbacks down the component tree
TokenDetailsActions.tsx: Accepts optional onActionTapped callback for Send, Receive, MoreOpened actions
MoreTokenActionsMenu.tsx: Accepts optional onActionTapped callback for ViewOnExplorer, RemoveToken actions
TokenDetails.tsx / TokenDetailsList.tsx: Accepts optional onCopyAddress callback

All changes are additive with optional props - no existing behavior is modified. The token details screen is part of the wallet platform (asset overview/token list). SmokeWalletPlatform covers transaction history and wallet features which include token detail views. No confirmations, swaps, navigation infrastructure, or core controllers are affected. No dedicated E2E tests exist for the Token Details screen specifically.

Performance Test Selection:
The changes are purely analytics tracking additions (new event, new hook, new optional callbacks). No UI rendering changes, no data loading changes, no state management changes, and no component structure changes that would affect performance. The new tracking hook uses useCallback with proper memoization, so no performance regression is expected.

View GitHub Actions results

Prithpal-Sooriya and others added 2 commits May 19, 2026 13:15

test: add unit tests for useTokenDetailsActionTracking hook

1aab37d

Tests all 6 action types (send, receive, more_opened, remove_token, view_on_explorer, copy_token_address) and source fallback behavior. Co-authored-by: Prithpal Sooriya <prithpal.sooriya@gmail.com>

github-actions Bot added the pr-not-ready-for-e2e Skip E2E and block merging. Remove this label once the PR is ready to run the E2E tests. label May 19, 2026

metamaskbotv2 Bot added team-assets INVALID-PR-TEMPLATE PR's body doesn't match template labels May 19, 2026

github-actions Bot added the size-M label May 19, 2026

Prithpal-Sooriya and others added 3 commits May 19, 2026 13:41

style: fix prettier formatting

d010c33

Co-authored-by: Prithpal Sooriya <prithpal.sooriya@gmail.com>

refactor: rename event to Token Details Action Clicked

f7673b7

Renamed from 'Token Details Action Tapped' to 'Token Details Action Clicked' per schema alignment. Co-authored-by: Prithpal Sooriya <prithpal.sooriya@gmail.com>

Merge branch 'main' of github.com:MetaMask/metamask-mobile into curso…

312f3bc

…r/assets-3212-token-details-action-tapped-cea8

Prithpal-Sooriya marked this pull request as ready for review June 1, 2026 14:01

Prithpal-Sooriya requested a review from a team as a code owner June 1, 2026 14:01

Merge branch 'main' into cursor/assets-3212-token-details-action-tapp…

98b8149

…ed-cea8

github-actions Bot added the risk:medium AI analysis: medium risk label Jun 1, 2026

juanmigdr approved these changes Jun 1, 2026

View reviewed changes

cursor Bot reviewed Jun 1, 2026

View reviewed changes

Comment thread app/components/UI/TokenDetails/components/TokenDetailsActions.tsx

Prithpal-Sooriya changed the title ~~feat: track Token Details secondary action buttons (ASSETS-3212)~~ feat: track Token Details secondary action buttons (ASSETS-3212) cp-7.80.0 Jun 1, 2026

Prithpal-Sooriya enabled auto-merge June 1, 2026 14:45

Merge branch 'main' into cursor/assets-3212-token-details-action-tapp…

f406fab

…ed-cea8

Prithpal-Sooriya added no-changelog no-changelog Indicates no external facing user changes, therefore no changelog documentation needed and removed pr-not-ready-for-e2e Skip E2E and block merging. Remove this label once the PR is ready to run the E2E tests. labels Jun 1, 2026

Merge branch 'main' into cursor/assets-3212-token-details-action-tapp…

679ed6a

…ed-cea8

Prithpal-Sooriya added this pull request to the merge queue Jun 1, 2026

Merged via the queue into main with commit 1bdcec5 Jun 1, 2026
408 of 418 checks passed

Prithpal-Sooriya deleted the cursor/assets-3212-token-details-action-tapped-cea8 branch June 1, 2026 19:20

github-actions Bot locked and limited conversation to collaborators Jun 1, 2026

metamaskbotv2 Bot added the release-7.81.0 Issue or pull request that will be included in release 7.81.0 label Jun 1, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat: track Token Details secondary action buttons (ASSETS-3212) cp-7.80.0#30379

feat: track Token Details secondary action buttons (ASSETS-3212) cp-7.80.0#30379
Prithpal-Sooriya merged 9 commits into
mainfrom
cursor/assets-3212-token-details-action-tapped-cea8

Prithpal-Sooriya commented May 19, 2026 •

edited

Loading

Uh oh!

github-actions Bot commented May 19, 2026

Uh oh!

sonarqubecloud Bot commented May 20, 2026

Uh oh!

juanmigdr left a comment

Uh oh!

cursor Bot left a comment

Uh oh!

Uh oh!

github-actions Bot commented Jun 1, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

Prithpal-Sooriya commented May 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Changelog

Related issues

Manual testing steps

Screenshots/Recordings

Pre-merge author checklist

Uh oh!

github-actions Bot commented May 19, 2026

Uh oh!

sonarqubecloud Bot commented May 20, 2026

Quality Gate passed

Uh oh!

juanmigdr left a comment

Choose a reason for hiding this comment

Uh oh!

cursor Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

github-actions Bot commented Jun 1, 2026

🔍 Smart E2E Test Selection

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Prithpal-Sooriya commented May 19, 2026 •

edited

Loading