mxabierto / madison

(sin mantenimiento) Madison 2.0 - nos fuimos a mxabierto/participa
GNU General Public License v3.0
2 stars 1 forks source link

Add Google Analytics #241

Closed defvol closed 9 years ago

defvol commented 9 years ago

We had chartbeat https://github.com/mxabierto/madison/pull/132

Now we need google analytics as well

PoC:

screen shot 2015-05-27 at 2 30 13 pm

codemoe commented 9 years ago

Todo se ve en orden....

El único tema es que chartbeat recomienda que el código esté justo antes de y Analytics recomienda que el código esté justo al abrir

En teoría debe funcionar igual pero es importante entrar al panel de google analytics al panel de Tiempo real y ver que efectivamente esté funcionando....

defvol commented 9 years ago

gracias @codemoe no sabía esto

te refieres a antes de body?

moveré entonces chartbeat para que esté en el head y analytics al inicio de body; a eso te referías?

defvol commented 9 years ago

after opening the head (chartbeat) http://support.chartbeat.com/docs/

codemoe commented 9 years ago

No.... no antes de body...

Chartbeat en sus docs menciona que el código debe estar justo antes de cerrar body.... (dónde está ahorita)

Sobre Analytics se recomienda que el código sea el primer elemento dentro del body...

Lo que decía es que solo hay que probar que en donde están ahorita los códigos si estén funcionando, si así funcionan los 2, ya no hace falta ningún cambio....

codemoe commented 9 years ago

El paso 2 de chartbeat es el que incluye la mayor parte de código y menciona que sea:

Part 2: Insert this script right before the closing tag:

defvol commented 9 years ago

ahora dejé separado chartbeat y google analytics

defvol commented 9 years ago

ya mandé google analytics a justo después de abrir body

defvol commented 9 years ago

https://github.com/mxabierto/madison/pull/241/files#diff-c8a5898091cfc69f6f545ea9d710ed27R35

codemoe commented 9 years ago

Excelente! :+1: