Add telemetryId's to engines.json
Categories
(Firefox :: Search, task, P2)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox71 | --- | fixed |
People
(Reporter: daleharvey, Assigned: daleharvey)
References
Details
Attachments
(1 file)
As a follow up to https://bugzilla.mozilla.org/show_bug.cgi?id=1542243
|
Assignee | |
Comment 1•5 years ago
|
||
So I will leave this open until we are all sure but I dont think this is necessary, we can still generate the same shortName using the new configuration format
|
||
Comment 2•5 years ago
|
||
(In reply to Dale Harvey (:daleharvey) from comment #1)
So I will leave this open until we are all sure but I dont think this is necessary, we can still generate the same shortName using the new configuration format
I think we should add the telemetryId - when we are able to start re-arranging some of the search add-ons (e.g. merge locales or rename them), I think we're going to need it. Adding it at this stage will give us a simple way to verify that we're reporting the id that we expect.
|
|
Assignee | |
Updated•5 years ago
|
|
||
Updated•5 years ago
|
|
|
Assignee | |
Comment 3•5 years ago
|
||
|
|
Assignee | |
Comment 4•5 years ago
|
||
So the above patch matches the modern configuration telemetryIds with the ones list.json will us. Its a simple field we need to add to every configuration we want to distinguish in telemetry
Mike Connor I only matched the telemetryId's for the 6 engines we test, mostly because if we are that bothered about getting it right we should add tests for it and we only have tests for those engines, this means twitter / wikipedia will report as "twitter" and not "twitter-jp" etc, is that ok?
Pushed by dharvey@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/365f3fcf9ba1 Add telemetryIds to modern config. r=Standard8
|
||
Comment 6•5 years ago
|
||
| bugherder | ||
|
|
||
Comment 7•5 years ago
|
||
(In reply to Dale Harvey (:daleharvey) from comment #4)
So the above patch matches the modern configuration telemetryIds with the ones list.json will us. Its a simple field we need to add to every configuration we want to distinguish in telemetry
Mike Connor I only matched the telemetryId's for the 6 engines we test, mostly because if we are that bothered about getting it right we should add tests for it and we only have tests for those engines, this means twitter / wikipedia will report as "twitter" and not "twitter-jp" etc, is that ok?
Mike answered this elsewhere and commented that this was fine.
Description
•