737220 - src attributes for apps stats

Status: RESOLVED FIXED

(Marketplace Graveyard :: Consumer Pages, defect, P3)

Description

[krupa raj[:krupa]]

We need to have src attributes to track:

a) app downloads
b) app purchases(?)

fligtar is okay with using the same format as AMO.

Comment 1

[Christopher Van Wiemeersch [:cvan]]

Homepage: ?src=mkt-home
Category browse: ?src=mkt-cat
Detail page: ?src=mkt-detail
Search results: ?src=mkt-search
Search suggestions: ?src=mkt-ss

Comment 2

[Andy McKay]

This is the same as data pings.

Comment 3

[Andy McKay]

src is being passed to metrics, we should set these up in the code as per @cvan's suggestion.

Comment 4

[Andy McKay]

The src is there, but this passes them through to the record and install method.

Comment 5

[Christopher Van Wiemeersch [:cvan]]

We are not passing `src` when we record the app installation: https://github.com/mozilla/zamboni/blob/master/mkt/detail/views.py#L48

We should do that until we have metrics logging all this.

Comment 6

[Andy McKay]

I've noticed that too. But note you don't need to pass src:

https://github.com/mozilla/zamboni/blob/master/lib/metrics/__init__.py#L35

So would need to look at that in more detail.

Comment 7

[Andy McKay]

[16/May/2012 02:50:50] "GET /en-US/app/marble-run-1/?src=search HTTP/1.0" 200 5400
02:50:54 z.metrics:INFO 7c25d2ae-fd2a-4482-8424-f5a0bf05d461|install|{"app-domain": "dekkostudios.com", "locale": "en-US", "src": "search", "app-id": 26, "user-agent": "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:15.0) Gecko/15.0 Firefox/15.0a1"} :/Users/andy/sandboxes/zamboni/lib/metrics/__init__.py:23
[16/May/2012 02:50:55] "POST /en-US/app/marble-run-1/record?src=search HTTP/1.0" 200 870

As long as its in the page that generates the button, it gets passed. Note its in the button JSON.

{"recordUrl": "/en-US/app/marble-run-1/record?src=search"..

Please re-open when you've got a particular example where it doesn't work and we'll fix that.

Comment 8

[Christopher Van Wiemeersch [:cvan]]

We're not setting ?src= from the Homepage, Category Browse, Detail Page, etc. per comment 1.

Comment 9

[Christopher Van Wiemeersch [:cvan]]

Reopening per comment 8.

Comment 10

[Kevin Ngo [:ngoke]]

https://github.com/mozilla/zamboni/commit/a32b7c3c4d029cb79a29f2dfc8c4cee6efafa4ff