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_languageextra key to the SelectTranslationsPaneltranslate_buttonevent. - Add a
to_languageextra key to the SelectTranslationsPaneltranslate_buttonevent. - Add a
detected_languageextra key to the SelectTranslationsPaneltranslate_buttonevent.
Tests to implement
- Ensure that these events have coverage in SelectTranslationsPanel telemetry tests.
| Assignee | ||
Comment 1•1 year 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•1 year ago
|
| Assignee | ||
Comment 3•1 year 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•1 year ago
|
Updated•1 year ago
|
Updated•1 year ago
|
Updated•1 year ago
|
Comment 5•1 year ago
|
||
| bugherder | ||
Comment 6•1 year 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
•