Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add ref, source and utm_source parameters as source attribute #27

Merged
merged 2 commits into from
Nov 1, 2020

Conversation

BetaHuhn
Copy link

This PR adds support for ref, source and utm_source parameters introduced by electerious/Ackee#173

@BetaHuhn
Copy link
Author

This implementation uses the searchParams API which is not supported by all browsers (95%). Let me know if I should use another way of getting the URL parameters

@electerious
Copy link
Owner

This implementation uses the searchParams API which is not supported by all browsers (95%). Let me know if I should use another way of getting the URL parameters

Please extract the parameters manually using location.search. ackee-tracker should be as backwards compatible as possible (I hope the rest of the script is compatible; I haven't tested in a while).

@BetaHuhn
Copy link
Author

Okay, the parameters are now being extracted from location.search

@electerious electerious changed the base branch from develop to feature/source November 1, 2020 17:58
@electerious electerious merged commit 82bb7db into electerious:feature/source Nov 1, 2020
electerious added a commit that referenced this pull request Dec 31, 2020
@electerious
Copy link
Owner

Added this PR to the v5 branch. Now looking into the Ackee part.

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