Closed
Bug 1213903
Opened 9 years ago
Closed 9 years ago
Add custom telemetry to contacts to measure the times the import options are used
Categories
(Firefox OS Graveyard :: Gaia::Contacts, defect)
Firefox OS Graveyard
Gaia::Contacts
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: arcturus, Assigned: arcturus)
Details
Attachments
(1 file)
In order to get more insights of the usage of the application from users, we would like to measure how many times the import features are being used (importing from sim, sd, gmail or hotmail)
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → francisco
Comment 1•9 years ago
|
||
Assignee | ||
Updated•9 years ago
|
Attachment #8672673 -
Flags: feedback?(thills)
Comment 2•9 years ago
|
||
Comment on attachment 8672673 [details] [review] [gaia] arcturus:bug-1213903 > mozilla-b2g:master Hi Francisco, It looks good. I did check that the gecko telemetry accepted the '.' in the name. It works fine and calculates the histogram properly. Thanks, -tamara
Attachment #8672673 -
Flags: feedback?(thills) → feedback+
Assignee | ||
Comment 3•9 years ago
|
||
Comment on attachment 8672673 [details] [review] [gaia] arcturus:bug-1213903 > mozilla-b2g:master Fernando could you take a look to the implementation of custom metrics in contacts. Here we are counting how many times the different importers are being used.
Attachment #8672673 -
Flags: review?(ferjmoreno)
Comment 4•9 years ago
|
||
Comment on attachment 8672673 [details] [review] [gaia] arcturus:bug-1213903 > mozilla-b2g:master LGTM. Thanks! I left only a few minor comments in the PR.
Attachment #8672673 -
Flags: review?(ferjmoreno) → review+
Assignee | ||
Comment 5•9 years ago
|
||
Landed in master: https://github.com/mozilla-b2g/gaia/commit/e458fb503d82222a4afae458a75ea1f633064857 and nga branch: https://github.com/mozilla-b2g/gaia/commit/435f356b38f22321c195c5d40eae5d4b987b3661
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•