Commit 2e4dab1
authored
chore: update support.metamask urls to track utm source (#27331)
<!--
Please submit this PR as a draft initially.
Do not mark it as "Ready for review" until the template has been
completely filled out, and PR status checks have passed at least once.
-->
## **Description**
Adds UTM parameters (`utm_source=mobile_app`) to outbound links in the
MetaMask Mobile app, specifically updating the support page link to
`https://support.metamask.io?utm_source=mobile_app`.
Currently all mobile app traffic appears as "Direct" in analytics. This
change enables proper attribution to distinguish mobile users from
extension users and other sources.
The solution updates link URLs in relevant components (help screens,
settings, etc.) while maintaining identical user experience.
## **Changelog**
CHANGELOG entry: Added UTM parameters to mobile app links for improved
analytics tracking
## **Related issues**
## **Manual testing steps**
```gherkin
Feature: UTM parameters on support links
Scenario: Support link contains UTM parameters
Given MetaMask Mobile app is running on test build
And user navigates to Settings → Help
When user taps the "Get help" / support link
Then link URL contains "?utm_source=mobile_app"
Scenario: External browser opens with correct UTM parameters
Given MetaMask Mobile app is running on test build
And user navigates to a screen with support link
When user taps support link
Then external browser opens https://support.metamask.io?utm_source=mobile_app
```
## **Screenshots/Recordings**
<!-- If applicable, add screenshots and/or recordings to visualize the
before and after of your change. -->
### **Before**
<!-- [screenshots/recordings] -->
### **After**
<!-- [screenshots/recordings] -->
## **Pre-merge author checklist**
- [x] I've followed [MetaMask Contributor
Docs](https://github.com/MetaMask/contributor-docs) and [MetaMask Mobile
Coding
Standards](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/CODING_GUIDELINES.md).
- [x] I've completed the PR template to the best of my ability
- [x] I've included tests if applicable
- [x] I've documented my code using [JSDoc](https://jsdoc.app/) format
if applicable
- [x] I've applied the right labels on the PR (see [labeling
guidelines](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/LABELING_GUIDELINES.md)).
Not required for external contributors.
## **Pre-merge reviewer checklist**
- [x] I've manually tested the PR (e.g. pull and build branch, run the
app, test code being changed).
- [x] I confirm that this PR addresses all acceptance criteria described
in the ticket it closes and includes the necessary testing evidence such
as recordings and or screenshots.
<!-- CURSOR_SUMMARY -->
---
> [!NOTE]
> **Medium Risk**
> Touches many user-facing outbound support links and `AppConstants` URL
wiring; low functional complexity, but mistakes could send users to
incorrect help pages or break deep-link constants.
>
> **Overview**
> Adds a `MOBILE_UTM` suffix and expands `app/constants/urls.ts` to
provide canonical, UTM-tagged MetaMask support/article URLs (plus shared
deep-link base constants like `MM_UNIVERSAL_LINK_HOST` and
`MM_ANDROID_BUNDLE_ID`).
>
> Updates a wide set of screens/modals (Support/Help, onboarding,
password reset, staking/earn, perps, ramp region/eligibility, network
verification, error boundary, etc.) to **stop using hardcoded support
URLs** and instead reference these centralized constants, with
corresponding test expectation updates. Also switches `AppConstants` to
import several URL values and bundle/universal-link constants from
`constants/urls` to avoid duplication.
>
> <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit
6ff5d04. Bugbot is set up for automated
code reviews on this repo. Configure
[here](https://www.cursor.com/dashboard/bugbot).</sup>
<!-- /CURSOR_SUMMARY -->1 parent 61a0b3c commit 2e4dab1
36 files changed
Lines changed: 203 additions & 147 deletions
File tree
- app
- components
- UI
- Compliance/contexts
- Earn
- components/Tron/TronStakingLearnMoreModal
- constants
- NetworkVerificationInfo
- OptinMetrics
- Perps
- components/PerpsTutorialCarousel
- constants
- ProtectYourWalletModal
- Ramp
- Aggregator/components
- UnsupportedRegionModal
- components/EligibilityFailedModal
- Rewards/components
- Onboarding
- __tests__
- Settings
- Stake/constants
- Views
- AccountBackupStep1B
- AccountsMenu
- ChoosePassword
- ErrorBoundary
- ImportFromSecretRecoveryPhrase
- ImportPrivateKey
- Quiz/SRPQuiz
- ResetPassword
- Settings
- AppInformation
- SecuritySettings/Sections
- confirmations/constants
- constants
- core
Lines changed: 2 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
| 9 | + | |
9 | 10 | | |
10 | 11 | | |
11 | 12 | | |
| |||
128 | 129 | | |
129 | 130 | | |
130 | 131 | | |
131 | | - | |
| 132 | + | |
132 | 133 | | |
133 | 134 | | |
134 | 135 | | |
| |||
Lines changed: 2 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
9 | 9 | | |
10 | 10 | | |
11 | 11 | | |
| 12 | + | |
12 | 13 | | |
13 | 14 | | |
14 | 15 | | |
| |||
185 | 186 | | |
186 | 187 | | |
187 | 188 | | |
188 | | - | |
| 189 | + | |
189 | 190 | | |
190 | 191 | | |
191 | 192 | | |
| |||
Lines changed: 1 addition & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
19 | 19 | | |
20 | 20 | | |
21 | 21 | | |
22 | | - | |
23 | | - | |
24 | | - | |
| 22 | + | |
25 | 23 | | |
26 | 24 | | |
27 | 25 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
1 | 3 | | |
2 | | - | |
| 4 | + | |
3 | 5 | | |
Lines changed: 2 additions & 4 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
25 | 25 | | |
26 | 26 | | |
27 | 27 | | |
| 28 | + | |
28 | 29 | | |
29 | 30 | | |
30 | 31 | | |
| |||
41 | 42 | | |
42 | 43 | | |
43 | 44 | | |
44 | | - | |
45 | 45 | | |
46 | 46 | | |
47 | 47 | | |
| |||
99 | 99 | | |
100 | 100 | | |
101 | 101 | | |
102 | | - | |
103 | | - | |
104 | | - | |
| 102 | + | |
105 | 103 | | |
106 | 104 | | |
107 | 105 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
51 | 51 | | |
52 | 52 | | |
53 | 53 | | |
| 54 | + | |
54 | 55 | | |
55 | 56 | | |
56 | 57 | | |
| |||
257 | 258 | | |
258 | 259 | | |
259 | 260 | | |
260 | | - | |
| 261 | + | |
261 | 262 | | |
262 | 263 | | |
263 | 264 | | |
| |||
Lines changed: 2 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
32 | 32 | | |
33 | 33 | | |
34 | 34 | | |
| 35 | + | |
35 | 36 | | |
36 | 37 | | |
37 | 38 | | |
| |||
379 | 380 | | |
380 | 381 | | |
381 | 382 | | |
382 | | - | |
| 383 | + | |
383 | 384 | | |
384 | 385 | | |
385 | 386 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
10 | 10 | | |
11 | 11 | | |
12 | 12 | | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
13 | 17 | | |
14 | 18 | | |
15 | 19 | | |
| |||
191 | 195 | | |
192 | 196 | | |
193 | 197 | | |
194 | | - | |
| 198 | + | |
195 | 199 | | |
196 | 200 | | |
197 | 201 | | |
| |||
210 | 214 | | |
211 | 215 | | |
212 | 216 | | |
213 | | - | |
214 | | - | |
| 217 | + | |
215 | 218 | | |
216 | 219 | | |
217 | 220 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
23 | 23 | | |
24 | 24 | | |
25 | 25 | | |
| 26 | + | |
26 | 27 | | |
27 | 28 | | |
28 | 29 | | |
| |||
63 | 64 | | |
64 | 65 | | |
65 | 66 | | |
66 | | - | |
| 67 | + | |
67 | 68 | | |
68 | 69 | | |
69 | 70 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
8 | 8 | | |
9 | 9 | | |
10 | 10 | | |
| 11 | + | |
11 | 12 | | |
12 | 13 | | |
13 | 14 | | |
| |||
120 | 121 | | |
121 | 122 | | |
122 | 123 | | |
123 | | - | |
| 124 | + | |
124 | 125 | | |
125 | 126 | | |
126 | 127 | | |
| |||
0 commit comments