Skip to content

Allow to use custom pageview url script for GoogleAnalytics tracker.#119

Merged
DonSchado merged 2 commits into
railslove:masterfrom
Haerezis:feature/custom-google-analytics-pageview-script
Jun 6, 2018
Merged

Allow to use custom pageview url script for GoogleAnalytics tracker.#119
DonSchado merged 2 commits into
railslove:masterfrom
Haerezis:feature/custom-google-analytics-pageview-script

Conversation

@Haerezis

@Haerezis Haerezis commented Jun 5, 2018

Copy link
Copy Markdown
Contributor

Hi, we use your gem in our project and we need to track (with google analytics) the hash part of the url in the pageview event. We found a solution in this stackoverflow thread : https://stackoverflow.com/a/4813223 .
I have simply added the option to the Google Analytics tracker for a custom pageview url script (which default to the old and normal url script).

I'm at your disposal if you need anything.

@DonSchado

Copy link
Copy Markdown
Collaborator

Hey @Haerezis thanks for your pull request! 💚
Could you add a simple spec around this?
That would be awesome

@DonSchado DonSchado requested a review from bumi June 5, 2018 15:02
@Haerezis Haerezis force-pushed the feature/custom-google-analytics-pageview-script branch from 8b7fb6e to cb4af06 Compare June 5, 2018 21:07
@Haerezis Haerezis force-pushed the feature/custom-google-analytics-pageview-script branch from cb4af06 to 3939ee6 Compare June 5, 2018 21:30
@Haerezis

Haerezis commented Jun 5, 2018

Copy link
Copy Markdown
Contributor Author

I've added the spec as requested. Sorry for not doing it directly, I just forgot :/

@bumi

bumi commented Jun 6, 2018

Copy link
Copy Markdown
Contributor

nice! looks good to me and I think a good addition. thanks @Haerezis

(note: I am not too familiar with the google analytics API)

@DonSchado DonSchado merged commit bcce595 into railslove:master Jun 6, 2018
@DonSchado

Copy link
Copy Markdown
Collaborator

Thanks! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants