Skip to content

[MKT-856]:feat/use partnerId instead of utmMedium in checkout#1977

Draft
jaaaaavier wants to merge 1 commit into
masterfrom
feat/add-resolve-partner-id-for-impact
Draft

[MKT-856]:feat/use partnerId instead of utmMedium in checkout#1977
jaaaaavier wants to merge 1 commit into
masterfrom
feat/add-resolve-partner-id-for-impact

Conversation

@jaaaaavier
Copy link
Copy Markdown
Contributor

Description

Context:
Affiliate links from Impact were appending the partner ID to utm_medium (e.g. utm_medium=312695), which broke GA4's channel grouping. The Impact Partner Tracking Template is being updated to use utm_medium=affiliate and utm_content={irpid} instead.

Changes in impact.service.ts

  • Renamed the utmMedium param of handleImpactDTCCheckout to partnerId to reflect what it actually represents. The payload sent to Impact (partner_id) is unchanged.
  • Added resolvePartnerIdFromUrl() helper that reads the partner ID from the URL with backward compatibility:

Related Issues

Related Pull Requests

Checklist

  • Changes have been tested locally.
  • Unit tests have been written or updated as necessary.
  • The code adheres to the repository's coding standards.
  • Relevant documentation has been added or updated.
  • No new warnings or errors have been introduced.
  • SonarCloud issues have been reviewed and addressed.
  • QA Passed

Testing Process

Additional Notes

@jaaaaavier jaaaaavier self-assigned this May 29, 2026
@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying drive-web with  Cloudflare Pages  Cloudflare Pages

Latest commit: c5c2172
Status: ✅  Deploy successful!
Preview URL: https://44ff7781.drive-web.pages.dev
Branch Preview URL: https://feat-add-resolve-partner-id.drive-web.pages.dev

View logs

@sonarqubecloud
Copy link
Copy Markdown

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