[about:addons] Create in-tree discopane to replace the remotely hosted discopane

VERIFIED FIXED in Firefox 68



3 months ago
Last month


(Reporter: robwu, Assigned: robwu)


Dependency tree / graph

Firefox Tracking Flags

(firefox68 verified, firefox69 verified)


(Whiteboard: [feature-scope])


(9 attachments)

First step of bug 1540173 is to create the discopane in the HTML-based about:addons page in m-c.

Type: defect → enhancement
Assignee: nobody → rob
Whiteboard: [feature-scope]
Blocks: 1546980
See Also: → 1546984
Attachment #9061113 - Flags: data-review?(chutten)

Require extensions.htmlaboutaddons.discover.enabled to be enabled
before the HTML-based discopane is shown. This allows the feature
to be turned on and/or off independent of the other HTML views.

Comment on attachment 9061113 [details]
data collection review request


    Is there or will there be documentation that describes the schema for the ultimate data set available publicly, complete and accurate?

Yes. This collection is Telemetry so is documented in its definitions file [Events.yaml](https://hg.mozilla.org/mozilla-central/file/tip/toolkit/components/telemetry/Events.yaml) and the [Probe Dictionary](https://telemetry.mozilla.org/probe-dictionary/).

    Is there a control mechanism that allows the user to turn the data collection on and off?

Yes. This collection is Telemetry so can be controlled through Firefox's Preferences.

    If the request is for permanent data collection, is there someone who will monitor the data over time?

Yes, :ddurst is responsible.

    Using the category system of data types on the Mozilla wiki, what collection type of data do the requested measurements fall under?

Category 2, Interaction.

    Is the data collection request for default-on or default-off?

Default on for all channels.

    Does the instrumentation include the addition of any new identifiers?


    Is the data collection covered by the existing Firefox privacy notice?


    Does there need to be a check-in in the future to determine whether to renew the data?

No. This collection is permanent.

Result: datareview+
Attachment #9061113 - Flags: data-review?(chutten) → data-review+

The client_id part of browser_html_discover_view_clientid.js was failing
on TV because of a pre-existing, test-specific issue in Telemetry.
Fixing this is not trivial, so just check that the ID was set instead of
checking its exact value.

See the comment for more details; the fix will be part of bug 1537933

Pushed by rob@robwu.nl:
Add discopane to about:addons HTML view r=mstriemer,flod
Change "Get Add-ons" to "Recommendations" r=mstriemer,flod
Add notice to discopane when TAAR is enabled r=mstriemer,flod
Put HTML discopane behind separate pref r=mstriemer
Add attribution parameters to AMO links r=mstriemer
Loosen Client-ID check in unit test r=rpl

backed out for not having a proper review peer.

Push that got backed out : https://treeherder.mozilla.org/#/jobs?repo=autoland&revision=b3f31aa87d0d11690a169368e39c8f07edf4ea88

Backout: https://hg.mozilla.org/integration/autoland/rev/2f3479842f9101fe2466775eaf777c728c289839

@Rob: since you landed multiple bugs in the same push, i backed out all of them.

Flags: needinfo?(rob)

Question was answered in #sheriffs . Landing those patches should have been fine.

Flags: needinfo?(apavel)

Is there something I need to do here?

Hi Andreaa, thanks for keeping an eye on the bug.
Since soft freeze has been delayed, I will add another patch (telemetry) before landing them all. Likely within a day.

Duplicate of this bug: 1549521

Try is green: https://treeherder.mozilla.org/#/jobs?repo=try&revision=c873d2cb4d2f321c833dd2441f300b9ead2246e8

I'll already land the existing patches so Mark can already rebase his patches on top of mine, and I'll add the telemetry patch later today.

Keywords: leave-open
Pushed by rob@robwu.nl:
Add discopane to about:addons HTML view r=mstriemer,flod,aswan
Change "Get Add-ons" to "Recommendations" r=mstriemer,flod
Add notice to discopane when TAAR is enabled r=mstriemer,flod
Put HTML discopane behind separate pref r=mstriemer
Add attribution parameters to AMO links r=mstriemer
Loosen Client-ID check in unit test r=rpl

This adds telemetry with new actions and link values to uniquely
identify interactions with the discovery pane and recommended add-on

The names are unique, because recommended add-on cards may also appear
in other views (bug 1523406).

Unrelated to this change, the documentation of "homepage" at Events.yaml
has been updated to clarify that it can also be a URL outside of AMO,
because that is the reality.

Keywords: leave-open
Pushed by rob@robwu.nl:
Add telemetry to discopane r=rpl,chutten
Closed: 2 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla68
Regressions: 1549521
Posted image Bug1546248.gif

This issue is verified as fixed on Firefox 69.0a1 (20190523044159) and Firefox 68.0b3 (20190521110747) under Win 7 64-bit and Mac OS X 10.14.1.

  • The preference extensions.htmlaboutaddons.discover.enabled is set to false by default.
    When the preference is set to false, the HTML-based discovery panel from about:addons is not enabled.

  • "Get Add-ons" has been changed to "Recommendations"

  • The links from the cards in the old and the new discopane work as expected.

  • The notice from discopane is visible only when TAAR is enabled.

  • In about:telemetry the events are:

    • Install from "Recommendations"
      {"action": "installFromRecommendation", "view": "discover", "addonId": "jid1-93WyvpgvxzGATw@jetpack", "type": "extension"}

    • Click on the “Manage” button
      {"action": "manage", "view": "discover", "addonId": "treestyletab@piro.sakura.ne.jp", "type": "extension"}

    • discohome (goes to AMO via a recommended add-on card)
      addonsManager link aboutAddons discohome {"view": "discover"}

    • discomore (goes to AMO via discover)
      addonsManager link aboutAddons discomore {"view": "discover"}

    • disconotice (goes to SUMO via discover)
      addonsManager link aboutAddons disconotice {"view": "discover"}

Please see the attached video.

Duplicate of this bug: 1307386
Duplicate of this bug: 1495560
Blocks: 1555012
Regressions: 1558183
You need to log in before you can comment on or make changes to this bug.