616212 - Add tracking tag to mobile newsletter 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/about_mobile/newsletter_subscribe.html', 'WT.ti',
'Mobile%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 616212

Here is the patch.  Mike - who do I know who to assign as the reviewer?  Or do small changes like this need to be reviewed?

Comment 2

[James Long (:jlongster)]

What is the next step to push this along?  Should I go ahead and commit the change or does it need to be reviewed?

Comment 3

[Laura Forrest]

Adding Wes to this bug so he can check the implementation before it goes live. 

Wes, please see the attached patch. Thanks!

Laura

Comment 4

[Wes Contreras]

I'm unable to test directly, but reviewing the revised HTML code, it looks good to me.

Comment 5

[James Long (:jlongster)]

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

Comment 6

[James Long (:jlongster)]

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 7

[James Long (:jlongster)]

Pushed to production

--

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

Comment 8

[raymond [:retornam] (inactive)]

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