1984201 - Update perplexity trending URL to accept lang param

Status: RESOLVED MOVED

(Firefox :: Search, task, P1)

Description

[Mandy Cheang [:mcheang][she/her]]

We need search config v2 changes for perplexity.
This is to add lang={acceptLanguages} param to the trending URL.

Comment 1

[Mandy Cheang [:mcheang][she/her]]

Attachment: Bug 1984201 - Update perplexity trending URL to accept lang param. r=Standard8!

Comment 2

[Mandy Cheang [:mcheang][she/her]]

Changes are on staging for QA to test

Comment 3

[Mandy Cheang [:mcheang][she/her]]

Test Instructions

Experiment URL: Perplexity Staging Experiment
Minimum Browser Version: Firefox 143.0a1 or newer

Steps to Test

1. Open Remote Settings Dev Tools and switch to Stage (preview).

2. In Firefox, go to about:config and set nimbus.debug to true.

3. Enroll in the Perplexity Staging Experiment using the experiment link.

4. Open about:preferences → Language → click Choose….

   • In the Webpage Language Settings dialog, click Select a language to add…, choose Romanian [ro], and move it to the top of the list.
   • Click OK to save.

5. Open the unified search button and select Perplexity as the search engine.

6. Verify that trending results appear in Romanian.

Additional Checks

• Repeat the test with a few other locales (de, es, ru, ko, it).
• If Perplexity does not recognize all the languages, confirm that it defaults to en-US results.

Comment 4

[Mandy Cheang [:mcheang][she/her]]

Please test this on Nightly and Beta branches. The code changes that needed to ride the train has been uplifted in Bug 1982959 to Beta because we want this to be available on 143 Release.

Comment 5

[Adrian Florinescu [:aflorinescu]]

I've done a bit of testing and there is a question in relationship to the scope of testing.
There are two language settings for Firefox, one not being available with default configs in Nightly, config starts defaulting it with Beta and upwards:

• Webpage Language Settings - the one specified in comment3
• Language pack - Choose the languages used to display menus, messages, and notifications from Firefox. - directly installing a localized version of firefox works as well - this is part of what search-config uses at init aside from region

Is it in the scope of this ticket and what should control the localization of Perplexity trending just the Webpage Language Settings ?

Comment 6

[Adrian Florinescu [:aflorinescu]]

(In reply to Adrian Florinescu [:aflorinescu] from comment #5)

I've done a bit of testing and there is a question in relationship to the scope of testing.
There are two language settings for Firefox, one not being available with default configs in Nightly, config starts defaulting it with Beta and upwards:

• Webpage Language Settings - the one specified in comment3
• Language pack - Choose the languages used to display menus, messages, and notifications from Firefox. - directly installing a localized version of firefox works as well - this is part of what search-config uses at init aside from region

Is it in the scope of this ticket and what should control the localization of Perplexity trending just the Webpage Language Settings ?

We've discussed this subject with :standard8 and figured out that for Perplexity the scope is just the Webpage Language Settings, so we're going to focus on testing this for nightly144 and beta143. While discussing this, we figured that google also uses the language settings header in a way, but for that we'll log a separate bug -> bug TBD.

Comment 7

[Adrian Florinescu [:aflorinescu]]

removing the NI

Comment 8

[Adrian Florinescu [:aflorinescu]]

We executed a round of testing on Win11, Mac 13 and Ubuntu 22 using comment 3 enviroment, with just only a branch from the experiment, since I figure the exeperiment is just to enable Perplexity and not part of the testing scope here. The testing was done on both 144.0a1 and 143.0b4.

The scope of testing covered:

• Perplexity trending with multiple languages for Webpage Language Settings, covering (de, es, ru, ko, it) as a base and exploring the rest of the available languages:
• after initial exploratory and comment 6, we ignored completely any region or Language pack configurations

We think that:

• bug 1985501 and bug 1985504 which are already blocking this request should be reviewed and confirmed before moving forward with this change.
• bug 1985515 is not blocking this request, but I think it should block the general Perplexity experiment

Comment 9

[Mandy Cheang [:mcheang][she/her]]

Thank you Adrian.

We're going to follow-up with their team about:

• bug 1985515
• bug 1985504

As for bug 1985501, I don't think it's a bug. I made the testing expectations more clear. See https://bugzilla.mozilla.org/show_bug.cgi?id=1985501#c1

Comment 10

[Mandy Cheang [:mcheang][she/her]]

The changes are live on prod now.
Just to note - perplexity record is still hidden behind the experiment flag.

We're going to follow up with their team about the bugs.

Comment 11

[Mandy Cheang [:mcheang][she/her]]

bug 1985515 is fixed on Perplexity's end.

bug 1985504 has been followed up with Perplexity team. They're working on a fix for it.