forked from mirrors/gecko-dev
Bug 1894517 - Add region-specific parameter to Pocket feed URL for experimentation. r=home-newtab-reviewers,nbarrett
Differential Revision: https://phabricator.services.mozilla.com/D209075
This commit is contained in:
parent
a9b2ecff48
commit
e024c86bb9
3 changed files with 17 additions and 1 deletions
|
|
@ -1771,6 +1771,9 @@ pref("browser.newtabpage.activity-stream.discoverystream.region-spocs-config", "
|
||||||
// List of regions that don't get the 7 row layout.
|
// List of regions that don't get the 7 row layout.
|
||||||
pref("browser.newtabpage.activity-stream.discoverystream.region-basic-config", "");
|
pref("browser.newtabpage.activity-stream.discoverystream.region-basic-config", "");
|
||||||
|
|
||||||
|
// Add parameters to Pocket feed URL.
|
||||||
|
pref("browser.newtabpage.activity-stream.discoverystream.pocket-feed-parameters", "&enableRankingByRegion=1");
|
||||||
|
|
||||||
// Allows Pocket story collections to be dismissed.
|
// Allows Pocket story collections to be dismissed.
|
||||||
pref("browser.newtabpage.activity-stream.discoverystream.isCollectionDismissible", true);
|
pref("browser.newtabpage.activity-stream.discoverystream.isCollectionDismissible", true);
|
||||||
pref("browser.newtabpage.activity-stream.discoverystream.personalization.enabled", true);
|
pref("browser.newtabpage.activity-stream.discoverystream.personalization.enabled", true);
|
||||||
|
|
|
||||||
|
|
@ -564,10 +564,17 @@ export class DiscoveryStreamFeed {
|
||||||
}
|
}
|
||||||
|
|
||||||
generateFeedUrl(isBff) {
|
generateFeedUrl(isBff) {
|
||||||
|
// check for experiment parameters
|
||||||
|
const hasParameters = lazy.NimbusFeatures.pocketNewtab.getVariable(
|
||||||
|
"pocketFeedParameters"
|
||||||
|
);
|
||||||
|
|
||||||
if (isBff) {
|
if (isBff) {
|
||||||
return `https://${Services.prefs.getStringPref(
|
return `https://${Services.prefs.getStringPref(
|
||||||
"extensions.pocket.bffApi"
|
"extensions.pocket.bffApi"
|
||||||
)}/desktop/v1/recommendations?locale=$locale®ion=$region&count=30`;
|
)}/desktop/v1/recommendations?locale=$locale®ion=$region&count=30${
|
||||||
|
hasParameters || ""
|
||||||
|
}`;
|
||||||
}
|
}
|
||||||
return FEED_URL;
|
return FEED_URL;
|
||||||
}
|
}
|
||||||
|
|
|
||||||
|
|
@ -958,6 +958,12 @@ pocketNewtab:
|
||||||
type: boolean
|
type: boolean
|
||||||
fallbackPref: >-
|
fallbackPref: >-
|
||||||
browser.topsites.contile.sov.enabled
|
browser.topsites.contile.sov.enabled
|
||||||
|
pocketFeedParameters:
|
||||||
|
type: string
|
||||||
|
fallbackPref: >-
|
||||||
|
browser.newtabpage.activity-stream.discoverystream.pocket-feed-parameters
|
||||||
|
description: >-
|
||||||
|
Add parameters to Pocket feed URL.
|
||||||
|
|
||||||
saveToPocket:
|
saveToPocket:
|
||||||
description: The save to Pocket feature
|
description: The save to Pocket feature
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue