Closed
Bug 1036581
Opened 10 years ago
Closed 10 years ago
Telemetry for SocialAPI
Categories
(Firefox Graveyard :: SocialAPI, defect)
Tracking
(firefox32 fixed, firefox33 fixed)
RESOLVED
FIXED
Firefox 33
People
(Reporter: mixedpuppy, Assigned: mixedpuppy)
Details
Attachments
(2 files, 2 obsolete files)
11.21 KB,
patch
|
Dolske
:
review+
|
Details | Diff | Splinter Review |
11.54 KB,
patch
|
Dolske
:
review+
Sylvestre
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
While FHR gives us install rates for social providers, we have no actual usage data. We need to add hooks for: - opening share/status/bookmark toolbar buttons - sidebar opened/closed at session start - sidebar opened/closed during session - sidebar changed
Assignee | ||
Comment 1•10 years ago
|
||
Assignee: nobody → mixedpuppy
Attachment #8453357 -
Flags: review?(benjamin)
Assignee | ||
Comment 2•10 years ago
|
||
Patch update after further conversations with dolske, jjensen, and BD.
Attachment #8453357 -
Attachment is obsolete: true
Attachment #8453357 -
Flags: review?(benjamin)
Attachment #8455441 -
Flags: review?(dolske)
Assignee | ||
Comment 3•10 years ago
|
||
remove unused code line missed in my last pass.
Attachment #8455441 -
Attachment is obsolete: true
Attachment #8455441 -
Flags: review?(dolske)
Attachment #8455443 -
Flags: review?(dolske)
Assignee | ||
Comment 4•10 years ago
|
||
some slight differences for aurora due to some refactoring that has happened only on nightly.
Attachment #8455484 -
Flags: review?(dolske)
Assignee | ||
Comment 5•10 years ago
|
||
https://tbpl.mozilla.org/?tree=Try&rev=b9498cc9e332 (33) https://tbpl.mozilla.org/?tree=Try&rev=be946e254617 (32)
Updated•10 years ago
|
Attachment #8455443 -
Flags: review?(dolske) → review+
Updated•10 years ago
|
Attachment #8455484 -
Flags: review?(dolske) → review+
Assignee | ||
Comment 6•10 years ago
|
||
new try for fx33 since the first one had issues (apparently unrelated to this patch) https://tbpl.mozilla.org/?tree=Try&rev=8433dd1ef260
Assignee | ||
Comment 7•10 years ago
|
||
fx33: https://hg.mozilla.org/integration/fx-team/rev/6c0b6745b7dc
Assignee | ||
Comment 8•10 years ago
|
||
Comment on attachment 8455484 [details] [diff] [review] add telemetry to social sidebar and buttons (aurora) Approval Request Comment [Feature/regressing bug #]: new telemetry hooks for socialapi [User impact if declined]: none [Describe test coverage new/current, TBPL]: existing tests pass, try's on tbpl [Risks and why]: low [String/UUID change made/needed]: none
Attachment #8455484 -
Flags: approval-mozilla-aurora?
Flags: needinfo?(lmandel)
Updated•10 years ago
|
status-firefox32:
--- → affected
status-firefox33:
--- → affected
Comment 9•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/6c0b6745b7dc
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 33
Updated•10 years ago
|
Updated•10 years ago
|
Attachment #8455484 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment 10•10 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/ea0e9e117349
Flags: needinfo?(lmandel)
Updated•5 years ago
|
Product: Firefox → Firefox Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•