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

GTMv4 and TrackCustomEvent #15

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

GTMv4 and TrackCustomEvent #15

wants to merge 7 commits into from

Conversation

vargash
Copy link

@vargash vargash commented Aug 27, 2015

  • A "trackGenericEvent" method has been implemented to enable a more generic event tracking. This currently accept only a single label-value pair of parameters, but will be updated in the future for an even more generic use.
  • iOS SDK Libraries have been updated to latest versions.
  • Android SDK Libraries and code have been updated from old GTMv3 to newest GTMv4.

@kraihn
Copy link
Owner

kraihn commented Aug 28, 2015

Hey @vargash, thanks for submitting this PR.

Can you make a case for using the GTM v4 library? I'm trying to avoid requiring Google Play Services as a dependency.

@joey91133
Copy link

I change this code to GTM v4 same day(27 Aug).
Because I need to use Google AdMob same time.(App is release now on play & app store)
Now found almost one in this PR.
but I use this

and it's work too.
Do I need a new PR for this??

luizcva pushed a commit to luizcva/cordova-plugin-tag-manager that referenced this pull request Nov 2, 2015
…pelo autor do plugin, porem esse PR parece ser uma atualizacao importante para a versao mais recente do SDK Android do Google Tag Manager
@macolo
Copy link

macolo commented Mar 23, 2017

@kraihn well the case is that v3 doesnt appear to be working anymore. According to this issue: #34

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.

5 participants