Update webtrends code for preview tracking

RESOLVED WONTFIX

Status

Websites Graveyard
getpersonas.com
RESOLVED WONTFIX
7 years ago
5 years ago

People

(Reporter: rdoherty, Unassigned)

Tracking

Details

(URL)

Attachments

(1 attachment)

(Reporter)

Description

7 years ago
Our mouseover tracking for persona previews is not being tracked correctly. It currently counts as a pageview and is not being sampled at 50%.

We need the following to be fixed:

* Add WT.dl=50 to the http tracking request
* Update our webtrends code with some extra code in dcsMultiTrack:

WebTrends.prototype.dcsMultiTrack = function() {
    var args = dcsMultiTrack.arguments ? dcsMultiTrack.arguments: arguments;
    if (args.length % 2 == 0) {
        this.dcsSetProps(args);
        var dCurrent = new Date();
        this.DCS.dcsdat = dCurrent.getTime();
        this.dcsFPC();
         //new code below
        if (this.dcsChk()) {
                this.dcsTag();
       }
    }
}
(Reporter)

Comment 1

7 years ago
r74472

For QA:

To see the new WT.dl parameter, open up Firebug when you're on stage, mouseover a few designs (50% sampling means some won't trigger the event) and look at the URL pinged. 'WT.dl=50' should be in the url.

The new sampling % should sample at 50%, so roughly half of the previews should trigger a beacon being fired.
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
(Reporter)

Comment 2

7 years ago
Forgot to mention we need to push this Thursday as our stats are b0rked with the current code:
http://grab.by/6ohH
(Reporter)

Comment 3

7 years ago
According to webtrends the sampling not working is a stage problem and should work on prod:

--
I think just a cookie problem here.  The cookie attaches itself to the domain as defined by this line:

this.fpcdom="getpersonas.com";

Since your staging environment is different, the cookie isn't setting correctly.

I think this should be fine when it moves to prod.

If you want to test in staging, simply set this.fpcdom=""; which tells the Webtrends code to set the cookie to whatever domain it is currently on.

--
Created attachment 476045 [details]
Post-fix screenshot

On mouse-over of Personas (preview), |WT.dl 50| is present.
Verified FIXED per comment 3 and comment 4; Ryan and I took a lot at this together.
Status: RESOLVED → VERIFIED
This works on stage but not on prod. The webtrends events are not getting fired
when you mouse over the personas.
Reopening...
Status: VERIFIED → REOPENED
Resolution: FIXED → ---
(Reporter)

Comment 7

7 years ago
I've emailed webtrends, awaiting a response.
I don't see events on stage on mouseover.  Vishal, how are you testing?
I think preview tracking got backed out by morgamic in r79490 but I'm not 100% sure.

"Removing tracking for Personas Previews in order to fix tracking statistics, per Blake."
getpersonas.com has been retired.  More information at https://blog.mozilla.org/addons/2013/04/11/background-themes-have-moved-to-amo/
Status: REOPENED → RESOLVED
Last Resolved: 7 years ago5 years ago
Resolution: --- → WONTFIX
(Assignee)

Updated

5 years ago
Component: getpersonas.com → getpersonas.com
Product: Websites → Websites Graveyard
You need to log in before you can comment on or make changes to this bug.