Skip to content

[MKT-897]feat/Open link in new tab and simplify URL#267

Merged
internxt-yu merged 4 commits into
masterfrom
feat/simplify-url
Jul 7, 2026
Merged

[MKT-897]feat/Open link in new tab and simplify URL#267
internxt-yu merged 4 commits into
masterfrom
feat/simplify-url

Conversation

@internxt-yu

Copy link
Copy Markdown
Contributor

Description

Clear the parameters of google analytics and google tags manager in url section, so the user don't see a super long url.

Changes Made

In the App.tsx:

  1. Extracts current URL search parameters once the component mounts.
  2. Delete GTM parameters (_gl, _gcl_au) and any keys starting with _ga (handling dynamic analytics IDs).
  3. Rebuilds the URL, preserving non-Google parameters and the navigation anchor (url.hash).
  4. Updates the browser's address bar seamlessly using window.history.replaceState.

@internxt-yu internxt-yu requested a review from jaaaaavier July 3, 2026 11:05
@internxt-yu internxt-yu self-assigned this Jul 3, 2026
@internxt-yu internxt-yu added the enhancement New feature or request label Jul 3, 2026
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jul 3, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
send-web ad8d214 Commit Preview URL

Branch Preview URL
Jul 06 2026, 01:49 PM

@larryrider

Copy link
Copy Markdown
Contributor

Hey!! in the description it says: - Delete GTM parameters (_gl, _gcl_au) and any keys starting with _ga (handling dynamic analytics IDs).
But that second part about keys starting with _ga is not implemented 🤔

is that expected?

@internxt-yu

Copy link
Copy Markdown
Contributor Author

Yes, you're absolutely right, is not implemented to clear those parameters that starts with '_ga', only parameter exactly named _ga. When i was testing it, it looks like to be working and it must be React has handled the incomplete url and did the cleanup, so didn't found out this was actually wrong. And by rechecking it, i find out that also the parameter '_fplc' needs to be cleared in the path. Thanks a lot for helping me to find this huge mistake!!!

Comment thread src/App.tsx Outdated
Comment thread src/utils/cleanUrl.ts Outdated
@internxt-yu internxt-yu merged commit 10fe9d5 into master Jul 7, 2026
5 checks passed
@internxt-yu internxt-yu deleted the feat/simplify-url branch July 7, 2026 08:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants