Add in google analytics to webpay for each page

RESOLVED FIXED in 2013-09-24

Status

P2
normal
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: andy+bugzilla, Assigned: scolville)

Tracking

2013-09-24
x86
Mac OS X
Points:
---

Details

(Whiteboard: qa+)

(Reporter)

Description

5 years ago
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']);
(Reporter)

Updated

5 years ago
Priority: -- → P2
(Reporter)

Updated

5 years ago
Version: 1.5 → 1.3
Assignee: nobody → wraithan
Assignee: wraithan → scolville
Depends on: 914051
Status: NEW → ASSIGNED
Target Milestone: --- → 2013-09-17
Target Milestone: 2013-09-17 → 2013-09-24
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.
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED

Comment 3

5 years ago
(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-]?
(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
Whiteboard: qa+
You need to log in before you can comment on or make changes to this bug.