Skip to content

Fix Share button referral link to use dl.edge.app#6068

Merged
j0ntz merged 1 commit into
developfrom
jon/share-url-dl-edge-app
Jul 11, 2026
Merged

Fix Share button referral link to use dl.edge.app#6068
j0ntz merged 1 commit into
developfrom
jon/share-url-dl-edge-app

Conversation

@j0ntz

@j0ntz j0ntz commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

CHANGELOG

Does this branch warrant an entry to the CHANGELOG?

  • Yes
  • No

Dependencies

none

Requirements

If you have made any visual changes to the GUI. Make sure you have:

  • Tested on iOS device
  • Tested on Android device
  • Tested on small-screen device (iPod Touch)
  • Tested on large-screen device (tablet)

Description

The side-menu "Share Edge" action built its referral URL from the bare website
domain (config.website = https://edge.app), producing
https://edge.app?af=appreferred_<refId>. AppsFlyer refDeviceInstallerId
attribution works for other af= links but not for appreferred shares.

This routes the share link through the AppsFlyer deep-link subdomain so the
appreferred attribution is tracked, matching how other af= attributions
already resolve:

https://dl.edge.app?af=appreferred_<refId>

Only SideMenu.handleShareApp changes. config.website is untouched, so the
Help modal official-site link still points at edge.app.

Tested on the iOS simulator: opened the side menu, tapped "Share Edge", and the
native share sheet link preview resolves to dl.edge.app (screenshot below).

Asana: https://app.asana.com/0/1215088146871429/1206924232175184


Note

Low Risk
Single share-handler URL change with config fallback; no auth, payments, or wallet logic touched.

Overview
Side-menu Share Edge no longer builds referral URLs from config.website (https://edge.app?af=appreferred_…). It now uses a dedicated AppsFlyer deep-link base so appreferred shares get refDeviceInstallerId attribution like other af= links.

Adds optional referralAppShareUrl on app config (Edge sets https://dl.edge.app). Share links become https://dl.edge.app/?af=appreferred_<refId> with the /? path shape expected by DeepLinkParser and AppsFlyer. White-label builds without the field still fall back to website. config.website is unchanged elsewhere (e.g. Help official site).

Reviewed by Cursor Bugbot for commit 6950fe4. Bugbot is set up for automated code reviews on this repo. Configure here.

@j0ntz

j0ntz commented Jul 10, 2026

Copy link
Copy Markdown
Contributor Author

📸 Test evidence

agent proof 1206924232175184 01 share sheet dl edge app

agent proof 1206924232175184 01 share sheet dl edge app

Captured by the agent's in-app test run (build-and-test).

@cursor cursor Bot 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.

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

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 9b2fc21. Configure here.

Comment thread src/components/themed/SideMenu.tsx Outdated
@j0ntz j0ntz force-pushed the jon/share-url-dl-edge-app branch from 9b2fc21 to 7246dac Compare July 10, 2026 19:16
@j0ntz

j0ntz commented Jul 10, 2026

Copy link
Copy Markdown
Contributor Author

📸 Test evidence (after white-label config fix)

agent proof 1206924232175184 01 share sheet dl edge app

agent proof 1206924232175184 01 share sheet dl edge app

Captured by the agent's in-app test run (build-and-test).

@j0ntz j0ntz force-pushed the jon/share-url-dl-edge-app branch from 7246dac to e708ce5 Compare July 10, 2026 23:57
@j0ntz j0ntz enabled auto-merge July 10, 2026 23:58
@j0ntz j0ntz force-pushed the jon/share-url-dl-edge-app branch 2 times, most recently from 0842b3f to 309d792 Compare July 11, 2026 01:12
The side-menu Share action built its referral URL from the bare website
domain (edge.app), which broke AppsFlyer appreferred attribution
(refDeviceInstallerId). Route the share link through the dl.edge.app
deep-link subdomain so the appreferred parameter is tracked, matching how
other af= attributions already work.
@j0ntz j0ntz force-pushed the jon/share-url-dl-edge-app branch from 309d792 to 6950fe4 Compare July 11, 2026 02:23
@j0ntz j0ntz merged commit 1443df3 into develop Jul 11, 2026
7 checks passed
@j0ntz j0ntz deleted the jon/share-url-dl-edge-app branch July 11, 2026 03:50
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