616216 - Add tracking tag to Student Reps email subscription

Status: VERIFIED FIXED

(www.mozilla.org :: General, defect)

Description

[Laura Forrest]

Please add this tracking code to the "Subscribe" button on this page. 


onMouseDown="dcsMultiTrack('DCS.dcsuri',
'/button/studentreps/newsletter_subscribe.html', 'WT.ti',
'Studentreps%20Newsletter%20Subscribe');">

See this for more info: https://intranet.mozilla.org/Webanalytics#Intra-page_Event_Tracking

Comment 1

[James Long (:jlongster)]

Attachment: Fix for Bug 616216

Here's a patch to implement this.  As with bug 616212, I'm not sure who needs to review this patch.

Comment 2

[James Long (:jlongster)]

Attachment: Fix for Bug 616216

Forgot to specify that last attachment as a patch.

Comment 3

[James Long (:jlongster)]

On the page with the spinner wheel (http://mozilla.james.local/en-US/firefox/fx/), the attachment says that the 3 sections on the wheel (mobile, personas, addons) are each supposed to be tagged with tracking code.  Is it supposed to track when the user clicks the wheel to spin?  All that happens when the user clicks any part of the wheel is it spins.

Comment 4

[James Long (:jlongster)]

Sorry, in the previous comment I gave a local url, here is the real url to that page:

http://www.mozilla.com/en-US/firefox/fx/

Comment 5

[Laura Forrest]

(In reply to comment #3)
> On the page with the spinner wheel
> (http://mozilla.james.local/en-US/firefox/fx/), the attachment says that the 3
> sections on the wheel (mobile, personas, addons) are each supposed to be tagged
> with tracking code.  Is it supposed to track when the user clicks the wheel to
> spin?  All that happens when the user clicks any part of the wheel is it spins.

Hey James - is this comment meant for bug 620705? 
https://bugzilla.mozilla.org/show_bug.cgi?id=620705

The scope of this bug is limited to the /newsletter page.

Comment 6

[James Long (:jlongster)]

Yes, sorry about that, I will move the comment.

Comment 7

[James Long (:jlongster)]

Same thing needs to happen here as bug 616212.  I've added Wes; if you wouldn't mind, could you approve this patch?

Comment 8

[Wes Contreras]

I don't have access to test directly, but reviewing the revised HTML code, it looks good to me.

Comment 9

[James Long (:jlongster)]

r80221 | [e-mail] | 2011-01-05 15:53:10 -0500 (Wed, 05 Jan 2011) | 2 lines

Comment 10

[James Long (:jlongster)]

Pushed to staging

--

r80663 | [e-mail] | 2011-01-12 16:27:13 -0500 (Wed, 12 Jan 2011) | 2 lines

r80220 from trunk for bug 616212
r80221 from trunk for bug 616216 (Committing all changes) [Kubla]

Comment 11

[James Long (:jlongster)]

Pushed to production

--

r80664 | [e-mail] | 2011-01-12 16:35:47 -0500 (Wed, 12 Jan 2011) | 6 lines

Comment 12

[raymond [:retornam] (inactive)]

verified fixed http://www.mozilla.com/en-US/newsletter/studentreps/