Repository navigation
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
While adding UTM parameters to the destination URL is a common practice, there are two technical limitations in this setup that will prevent you from tracking engagement as intended:
next.config.mjsare executed at the server/edge level (returning a307 Temporary Redirectresponse directly). Because of this, the browser never loads or executes any client-side analytics scripts (e.g., Google Analytics, Plausible, Fathom) on your domain (agentrelay.com) for the/qr-cardpath.utm_source,utm_medium, orutm_campaign.Suggested Alternatives:
app/qr-card/page.tsx(orpages/qr-card.tsx) that triggers a client-side analytics event (e.g.,track('QR Card Scan')) and then redirects the user to GitHub usingwindow.location.replace('https://github.com/agentworkforce/relay').middleware.ts), you can intercept requests to/qr-card, send a server-side tracking event to your analytics provider's HTTP API (e.g., Plausible Events API or GA4 Measurement Protocol), and then return the redirect response.