Skip to content
This repository has been archived by the owner. It is now read-only.

/ screenshots Archived

Send Telemetry directly from the webextension #3713

Closed
jaredhirsch opened this issue Nov 2, 2017 · 3 comments
Closed

Send Telemetry directly from the webextension #3713

jaredhirsch opened this issue Nov 2, 2017 · 3 comments
Assignees
@jaredhirsch
Labels
analytics firefox
Milestone
Sprint 20 (64-1)

Comments

@jaredhirsch
Copy link
Member

@jaredhirsch jaredhirsch commented Nov 2, 2017

Tracked on Bugzilla as https://bugzilla.mozilla.org/show_bug.cgi?id=1280234

#2218 added Telemetry measurements for Screenshots usage. Currently, when shots are downloaded or uploaded, a message is sent to bootstrap, then bootstrap calls the Telemetry service to increment the session usage counts.

Instead, it would be great to call Telemetry directly from the webextension. This bug tracks implementation of a pass-through subset of the Telemetry service API.

This seems particularly useful to the shield team, and it would give us one less reason to have to stay as an embedded webextension.
@wresuolc wresuolc added this to the Sprint 5 (60-0) 🔘 milestone Nov 2, 2017
@jaredhirsch
Copy link
Member Author

@jaredhirsch jaredhirsch commented Mar 27, 2018

Bumping out to the next milestone
@wresuolc wresuolc modified the milestones: Sprint 10 (61-2) ⛅, Sprint 13 (62-0) 👁 Apr 2, 2018
@marniepw marniepw removed their assignment Aug 2, 2018
@wresuolc wresuolc removed this from the Sprint 18 (63-2) milestone Aug 6, 2018
@wresuolc wresuolc added this to the Sprint 19 (63-3) milestone Aug 6, 2018
@jaredhirsch
Copy link
Member Author

@jaredhirsch jaredhirsch commented Aug 11, 2018

Just landed the webext telemetry API 🎉
@jaredhirsch
Copy link
Member Author

@jaredhirsch jaredhirsch commented Aug 14, 2018

The upstream API has bounced twice now. Leaving this open to track implementation in Screenshots.
@jaredhirsch jaredhirsch reopened this Aug 14, 2018
@jaredhirsch jaredhirsch mentioned this issue Aug 15, 2018
Closed
12 of 17 tasks complete
jaredhirsch added a commit that referenced this issue Aug 15, 2018
@jaredhirsch
Fix #3713, Send Telemetry pings from the WebExtension
Loading status checks…
c0ba50a
jaredhirsch added a commit that referenced this issue Aug 21, 2018
@jaredhirsch
Fix #3713, Send Telemetry pings from the WebExtension
Loading status checks…
d08961b
@jaredhirsch jaredhirsch mentioned this issue Aug 21, 2018
Merged
@chenba chenba closed this in #4798 Aug 21, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@jaredhirsch jaredhirsch

Labels
analytics firefox
Projects
None yet
Milestone
Sprint 20 (64-1)
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
6 participants
@ianb @jaredhirsch @chenba @johngruen @wresuolc @marniepw