search

shopware5 / SwagGoogle

Shopware integration for Google Analytics, Universal Analytics and Google Adwords services
http://shopware.com
MIT License
12 stars 17 forks source link

Switch für header / footer include #2

Closed tim-bec closed 9 years ago

tim-bec commented 9 years ago

Google empfiehlt den analytics-Code vor dem schließenden /head einzufügen. ( https://developers.google.com/analytics/devguides/collection/analyticsjs/ ) Ich verstehe warum das Javascript standardmäßig im Footer platziert wird. Jedoch macht das einbinden des GA Codes im Footer oft beim exakten Tracking Probleme.

Vielleicht wäre ein guter Kompromiss ein switch der Wahlweise in frontend_index_header oder wie bisher frontend_index_header_javascript extended.

Einen Pull-Request kann ich ggf. beisteuern.

bcremer commented 9 years ago

/cc @klarstil can your team please review this.

klarstil commented 9 years ago

@tim-bec Ich sehe aktuell leider nicht das Problem. Einen Block einzubauen, worin der Code dann eingebaut werden kann ist natürlich kein Problem.

Das Skript macht ja nichts als ein anderes Skript nachzuladen und einen Request an Google zu senden. Dieser Prozess läuft bereits asynchron ab, weshalb ich kein Problem sehe warum das Tracking nicht richtig funktionieren sollte.

tim-bec commented 9 years ago

@klarstil - Danke für eure schnelle Antwort. Mit der Einbindung im Footer gibt es ( vor allem in Kombination mit Adwords) immer mal Abweichungen in den Daten zwischen 5-15%. Mit einer Einbindung in den header erreichen wir hingegen fast 100%

klarstil commented 9 years ago

@tim-bec Danke für die Antwort. Ich habe das Problem intern abgesprochen. Wir werden es so machen, dass wir einen neuen Block mit Shopware 5.1 in den Core einbauen und einen Switch wie vorgeschlagen im Plugin zur Verfügung stellen, damit die Position der Einbindung festgelegt werden kann.

klarstil commented 9 years ago

Ich lasse die Issue so lange auf bis wir den Block eingebaut haben.

tim-bec commented 9 years ago

@klarstil sauber. Besten Dank!

klarstil commented 9 years ago

Hier schon mal der Block: https://github.com/shopware/shopware/commit/f325b3b2c056875f4be7d5288d7d435b09f0e9df

klarstil commented 9 years ago

@tim-bec Der Switch ist jetzt drin :) Daher mache ich das Ticket zu