Closed Bug 1646142 Opened 4 years ago Closed 4 years ago

Use Region.current engine in fetchEngineConfiguration

Categories

(Firefox :: Search, task, P3)

task
Points:
2

Tracking

()

RESOLVED INVALID

People

(Reporter: daleharvey, Assigned: daleharvey)

References

(Blocks 1 open bug)

Details

Attachments

(1 obsolete file)

+++ This bug was initially created as a clone of Bug #1634580 +++

When we start fetching the region more often we can use the up to date region to send the correct region specific code to engines without reloading the engine list

This means we would probably need to keep track of an additional region, current + currently applied to the engine list. Can possibly add a new field to the configuration "regionOverrides" that is stored in the engine object

Points: 5 → 2
Summary: Use US google region codes while in US → Update search configuration to use regionParams for US google codes

Are you thinking of using different codes on a temporary basis, or a permanent if the region changes?

If it is permanent (until next change), I think after we've done the work on maybeReloadEngines, reloading the engine list wouldn't be as invasive as it was previously.

It'd also mean the user would get US versions of websites rather than regions.

I am not sure what you mean by temporary / permanent, I think in some cases, particularly google, this can be the way in which we specify the codes instead of having seperate params matched in the appliesTo section.

Even when we refactor maybeReloadEngines, we still arent going to immediately change the users engine list every time the region changes and its not for certain we would ever want to do that for when the user goes on holiday etc. The cache busting patch in https://bugzilla.mozilla.org/show_bug.cgi?id=1627555 will update the users home region with a minimum 2 week delay at the moment.

I think with the region changes and maybeReloadEngines refactor we may want to make some product changes to what engines we show to the user and how we update them, but I think if we decouple that from what codes we are sending we dont need to make any tradeoffs between the accuracy of the codes and what makes the most sense for the user.

Assignee: nobody → dharvey

Taking a look at this and hit against some problems with complexity and telemetryId, I have another approach in mind and will reuse this bug to do it

Summary: Update search configuration to use regionParams for US google codes → Use Region.current engine in fetchEngineConfiguration
Iteration: 79.2 - June 15 - June 28 → 81.1 - July 27 - Aug 09
Iteration: 81.1 - July 27 - Aug 09 → 81.2 - Aug 10 - Aug 23
Iteration: 81.2 - Aug 10 - Aug 23 → 82.2 - Sep 7 - Sep 20
Attachment #9171893 - Attachment description: Bug 1646142 - Use paramters from current region → Bug 1646142 - Use engines parameters from current region
Iteration: 82.2 - Sep 7 - Sep 20 → 83.1 - Sept 21 - Oct 4
Iteration: 83.1 - Sept 21 - Oct 4 → ---

I think given the increases we have already made with revenue efficiency and the increased complexity this is going to bring, going to shelve it until it becomes a product requirement.

Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → INVALID
Attachment #9171893 - Attachment is obsolete: true
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: