1932196 - [Sections Personalization] scheduled corpus item id change

Status: RESOLVED FIXED

(Firefox :: New Tab Page, enhancement)

Description

[Scott [:thecount] Downe]

We need to make a change to one of the ids in the items, this probably involves a new prop with a new item id, and somehow manage the old id.

Comment 1

[Jonathan Petto [:jpetto]]

Background: for the Section experiments, we are moving away from a daily schedule. This means that not every content recommendation will have a scheduledItemId. This will have an impact on both the client (new tab) and our analytics (big query, downstream reports).

We propose using corpusItemId as a more stable identifier, as every scheduledItemId already has a corpusItemId backing it. We will begin sending corpusItemId in all responses from Merino. (Jira ticket TBD.) corpusItemId will be of the same data type as scheduledItemId.

On the client, we should use corpusItemId instead of scheduledItemId to handle dismiss functionality. If there are other places where scheduledItemId is used on the client, we should discuss moving to corpusItemId.

For analytics and reporting, we'd like to continue to pass scheduledItemId through as we do currently, though as noted above it will not be a guaranteed field. We should also pass corpusItemId through to analytics.

It may make sense to make this change even outside of the Section experiments, as it would likely be beneficial to begin sending corpusItemId to analytics for all recommendations delivered from Merino.

Comment 2

[Matt Miermans]

We would also like to make tileId nullable. Functionally, it's like an integer version of scheduled_corpus_item_id.

Comment 3

[Scott [:thecount] Downe]

Attachment: Bug 1932196 - Newtab story id work.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6ed3f305c58f
Newtab story id work. r=home-newtab-reviewers,mconley

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c92619d46c4f
Fix lint failure on DiscoveryStreamFeed.sys.mjs . CLOSED TREE

Comment 6

[Cristina Horotan [:chorotan]]

https://hg.mozilla.org/mozilla-central/rev/6ed3f305c58f
https://hg.mozilla.org/mozilla-central/rev/c92619d46c4f