Send Source Language Changes to SelectTranslationsPanel Telemetry
Categories
(Firefox :: Translations, task, P3)
Tracking
()
People
(Reporter: nordzilla, Assigned: nordzilla)
References
Details
Attachments
(2 files)
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
phab-bot
:
approval-mozilla-beta+
|
Details | Review |
Description
While our flow-id telemetry can technically tell when the source language was changed from the unsupported-language view based on tracing the events in the flow, it will add extra cost on the data side to do these calculations.
It would be much better to add extra keys to the translate_button
event when a user has decidedly changed the incorrectly detected source language and requested a corrected translation.
Steps to implement
- Add a
from_language
extra key to the SelectTranslationsPaneltranslate_button
event. - Add a
to_language
extra key to the SelectTranslationsPaneltranslate_button
event. - Add a
detected_language
extra key to the SelectTranslationsPaneltranslate_button
event.
Tests to implement
- Ensure that these events have coverage in SelectTranslationsPanel telemetry tests.
Assignee | ||
Comment 1•3 months ago
|
||
Adds the ability for the SelectTranslationsPanel to send
telemetry events when the source language is changed and a
translation is requested from the unsupported-language view.
Depends on D214159
Updated•3 months ago
|
Assignee | ||
Comment 3•3 months ago
|
||
Adds the ability for the SelectTranslationsPanel to send
telemetry events when the source language is changed and a
translation is requested from the unsupported-language view.
Original Revision: https://phabricator.services.mozilla.com/D214160
Updated•3 months ago
|
Updated•3 months ago
|
Updated•3 months ago
|
Updated•3 months ago
|
Comment 5•3 months ago
|
||
bugherder |
Comment 6•3 months ago
|
||
I can confirm that the onTranslateButton event contains the from_language, to_language and detected_language extra keys.
Verified in Windows10, MacOS11 and Ubuntu22 on Nightly v129.0a1 and Beta v128.0b7.
Description
•