Closed Bug 1665914 Opened 1 year ago Closed 1 year ago

Account for scenarios of Topsites experiment and Topsites Remote Settings integration

Categories

(Firefox :: Top Sites, task, P2)

task
Points:
1

Tracking

()

RESOLVED FIXED
83 Branch
Iteration:
83.1 - Sept 21 - Oct 4
Tracking Status
firefox82 --- disabled
firefox83 --- fixed

People

(Reporter: aflorinescu, Assigned: dao)

References

(Blocks 2 open bugs)

Details

Attachments

(1 file)

Taking in account the weekly search sync with Mike de Boer and additional brain storming with :cmuresan, it is very likely that the Topsites Experiment (https://experimenter.services.mozilla.com/experiments/topsites-url-redirect/ + https://experimenter.services.mozilla.com/experiments/topsites-url-redirect-germany/) and the turn on of the Topsites powered by Remote Settings will intersect (experiment rampdown and RS Topsites ramp up) and the scenarios combining the two are not covered anywhere at this moment.

Logging this to keep track that the above should be considered in order to ensure the transition betwen the two goes smoothly.

Worth verifying, but in theory this will "just work". The experiment turns search tiles into direct links, but we don't plan on using search tiles from remote settings, so at that point the experiment code wouldn't do anything.

Assignee: nobody → dao+bmo
Severity: -- → S3
Status: NEW → ASSIGNED
Iteration: --- → 83.1 - Sept 21 - Oct 4
Points: --- → 1
Priority: -- → P2
Pushed by dgottwald@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f8784af2caa8
Ignore search tile override prefs when using top site defaults from remote settings. r=mikedeboer
Flags: qe-verify+
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 83 Branch

Comment on attachment 9177353 [details]
Bug 1665914 - Ignore search tile override prefs when using top site defaults from remote settings. r=mikedeboer

Beta/Release Uplift Approval Request

  • User impact if declined: I'm told we might in fact use search tiles from remote settings in some regions, at which point this will become relevant.
  • Is this code covered by automated tests?: No
  • Has the fix been verified in Nightly?: No
  • Needs manual test from QE?: Yes
  • If yes, steps to reproduce:
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): trivial fix
  • String changes made/needed:
Attachment #9177353 - Flags: approval-mozilla-beta?
QA Whiteboard: [qa-triaged]
Attachment #9177353 - Flags: approval-mozilla-beta?
You need to log in before you can comment on or make changes to this bug.