903050 - Add in google analytics to webpay for each page

Status: RESOLVED FIXED

(Marketplace Graveyard :: Payments/Refunds, defect, P2)

Description

[Andy McKay]

We'd like to have analytics data sent throughout webpay so we can find out what's going on. Fireplace has analytics so feel free to crib data out of that, including the account id etc.

This bug is to add in analytics onto each page view. So as each page is rendered call:

_gaq.push(['_trackPageview', '/url/to/that/page']);

Comment 1

[[e-mail]]

r? https://github.com/mozilla/webpay/pull/280/files

Comment 2

[[e-mail]]

https://github.com/mozilla/webpay/commit/7f6c1136a75c81c8ad4e5d9eb654b8bfd7519f79

QA: Notes - This adds tracking to all the basic pages testing payment flows to ensure no JS regressions.
I'll verify separately in GA.

Comment 3

[Victor Carciu]

(In reply to Stuart Colville [:scolville] from comment #2)
> https://github.com/mozilla/webpay/commit/
> 7f6c1136a75c81c8ad4e5d9eb654b8bfd7519f79
> 
> QA: Notes - This adds tracking to all the basic pages testing payment flows
> to ensure no JS regressions.
> I'll verify separately in GA.

Can you please mark it as [qa-]?

Comment 4

[[e-mail]]

(In reply to Victor Carciu from comment #3)
> (In reply to Stuart Colville [:scolville] from comment #2)
> > https://github.com/mozilla/webpay/commit/
> > 7f6c1136a75c81c8ad4e5d9eb654b8bfd7519f79
> > 
> > QA: Notes - This adds tracking to all the basic pages testing payment flows
> > to ensure no JS regressions.
> > I'll verify separately in GA.
> 
> Can you please mark it as [qa-]?

Sorry my previous comment was unclear. This needs checking across all the webpay flow to ensure no JS regressions. So it's probably more qa+ than qa- Same goes for bug: 903056