Closed Bug 1968325 Opened 1 year ago Closed 11 months ago

Remove Pocket suggestions from the `suggest` component

Categories

(Application Services :: Suggest, task, P3)

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: adw, Assigned: kpatenio)

References

(Blocks 1 open bug)

Details

Attachments

(1 obsolete file)

We should remove Pocket suggestions from the suggest component in application-services. There's not a single place or module where they live but this code search should cover everything.

Bug 1959497 is the corresponding desktop bug.

Once that's merged into app-services, we'll need to vendor into m-c, which should have a separate bug filed for it. Vendoring steps:

  1. Update Cargo.toml so the application-services components use the new revision

  2. In a full (non-artifact) build, run:

./mach vendor rust
./mach uniffi generate

Vendoring docs here: 1, 2

  1. Upload the resulting patch to phabricator like usual

WIP pull request for application-services can be found here: https://github.com/mozilla/application-services/pull/6770

Assignee: nobody → rking
Attachment #9492784 - Attachment is obsolete: true

Thanks for your effort to vendor here Becca, sorry for the trouble. As we found out, vendoring is broken on Windows (bug 1938341), so I'll go ahead and take that on. We typically file a new bug for each vendoring, so I'll do that and link it here. This bug tracks the Rust changes that kpatenio made, so I'll set this bug's assignee appropriately and close it since kpatenio's PR has been merged.

Assignee: rking → kpatenio
Status: NEW → RESOLVED
Closed: 11 months ago
Resolution: --- → FIXED
Blocks: 1970949
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: