1814195 - Add telemetry to about:translations

Status: NEW

(Firefox :: Translations, enhancement, P3)

Description

[Greg Tatum [:gregtatum]]

The telemetry from the addon was not ported over. We should do that and request a data review.

Comment 1

[Sylvestre Ledru [:Sylvestre]]

Greg, it could be a good first bug?

Comment 2

[Greg Tatum [:gregtatum]]

I'm not sure we're wanting to work on about:translations right now, since it's only developer-focused at this moment. I believe Erik is going to take a pass through our backlog and mark good first bugs.

Comment 3

[Greg Tatum [:gregtatum]]

I'm marking this as a good first bug since it's a fairly tractable problem. At the time of this writing about:translations is somewhat hidden and is developer focused, so it's a fairly safe place to contribute some code.

The work here would be to add telemetry to about:translations.

The non-privileged UI code is in: toolkit/components/translations/content/translations.mjs

The telemetry code needs to be hooked into a global function in the AboutTranslationsChild.

Examples of existing telemetry can be found via this searchfox search. I would recommend using the blame information in the gutter to find a good single commit to see how we have done this in the past.

Comment 4

[Girish Jangam]

Hey, I would like to work on this bug as i'm absolute beginner open source . This is a good opportunity for me to learn more. would you please highlight me prerequisites.

Comment 5

[Greg Tatum [:gregtatum]]

Girish: Are you part of the outreachy program? If so I would suggest reaching out to: https://chat.mozilla.org/#/room/#firefoxtranslations:mozilla.org

There are a bunch of folks getting started and you all can help each other out.

Comment 6

[stanzin]

Hello Greg,

I hope this message finds you well. I am absolutely thrilled about the possibility of joining the Outreachy program. Could you please assign the issue to me?

Thank you!

Comment 7

[Greg Tatum [:gregtatum]]

Hi stanzin, feel free to start working on it, and if you attach a patch I'll make sure it's assigned to. Feel free to ask questions here: https://chat.mozilla.org/#/room/#firefoxtranslations:mozilla.org

Comment 8

[stanzin]

Thank you so much Greg, I will work on the issue tomorrow. have a nice day ahead )

Comment 9

[anayocrescent2]

please Greg Tatum [:gregtatum] please can you explain telementary is and what features it adds, it would be good if i can understand what i am working on

Comment 10

[anayocrescent2]

Attachment: Bug 1814195 - Add telemetry to about translations r?gregtatum

Comment 11

[stanzin]

Hello : anayocrescent2 , just wondering are you a outreachy applicant ?

Comment 12

[anayocrescent2]

yes

Comment 13

[anayocrescent2]

(In reply to stanzin from comment #11)

Hello : anayocrescent2 , just wondering are you a outreachy applicant ?

yes i am, i really need help here please

Comment 14

[Greg Tatum [:gregtatum]]

More information about telemetry can be found here:

https://firefox-source-docs.mozilla.org/toolkit/components/telemetry/

I left some comments on how to go about this work here: https://bugzilla.mozilla.org/show_bug.cgi?id=1814195#c3

Finally, the code can be run through building Firefox and going to about:translations.

The tests for about:translations should pass for an initial fix:

./mach mochitest toolkit/components/translations/tests/browser/

And once telemetry is added a test assertion will need to be added as well. For outreachy, I would suggest adding a single telemetry call for when about:translations is opened.

Comment 15

[BugBot [:suhaib / :marco/ :calixte]]

This good-first-bug hasn't had any activity for 2 months, it is automatically unassigned.
For more information, please visit BugBot documentation.