1545838 - Update Fennec's AMO URL (again)

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: Add-on Manager, defect, P1)

Description

[Chris Peterson [:cpeterson]]

Petru, I see you just landed the new AMO URL and title for bug 1544525. Sorry to keep changing the feature requirements, but the AMO team has a new recommended AMO URL:

https://addons.mozilla.org/firefox/collections/4757633/mob/?page=1&collection_sort=-popularity

The URL they had recommended in bug 1544525 sorts the add-ons by "newest first". The new URL above sorts the add-ons by popularity. The "newest first" sort order is not meaningful to users. The popularity sort order will guide users to the most relevant add-ons first.

Comment 1

[Andrei Teslovan [:ateslovan]]

Attachment: Bug 1545838 - Update Fennec's AMO URL; r?petru

Comment 2

[Narcis Beleuzu [:NarcisB]]

Andrei, patch failed to land with reason:

We're sorry, Autoland could not rebase your commits for you automatically. Please manually rebase your commits and try again.

(255, 'applying /tmp/tmpyizcAp\npatching file mobile/android/app/mobile.js\nHunk #1 FAILED at 211\n1 out of 1 hunks FAILED -- saving rejects to file mobile/android/app/mobile.js.rej\nabort: patch failed to apply', '')

Comment 3

[Andrei Teslovan [:ateslovan]]

Rebased on top of the latest central. Please try again.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b6dd2d9ef99d
Update Fennec's AMO URL; r=petru

Comment 5

[Oana Pop-Rus]

https://hg.mozilla.org/mozilla-central/rev/b6dd2d9ef99d

Comment 6

[Chris Peterson [:cpeterson]]

Andrei, we will want to uplift your AMO URL fix to Fennec 67 Beta, but we need to make sure Petru's AMO patches from bug 1544525 landed on Beta first.

Comment 7

[Andrei Teslovan [:ateslovan]]

Comment on attachment 9059813 [details]
Bug 1545838 - Update Fennec's AMO URL; r?petru

Beta/Release Uplift Approval Request

• User impact if declined: outdated experience for the AMO addons
• 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: Open about:addons
  Press "Browse Firefox’s Recommended Extensions"
  Check if the URL opened is "https://addons.mozilla.org/firefox/collections/4757633/mob/?page=1&collection_sort=-popularity"
• List of other uplifts needed: Bug 1544525
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): Not risky - it's just a simple URL change
• String changes made/needed: No

Comment 8

[Sorina Florean [:sflorean]]

Verified as fixed on latest Nightly build (68.0a1 - 04/23). The URL "https://addons.mozilla.org/firefox/collections/4757633/mob/?page=1&collection_sort=-popularity" is opened after "Browse Firefox's Recommended Extensions" is tapped.

Bug 1544525 is verified as well, on Nightly.

Comment 9

[Pascal Chevrel:pascalc (PTO until April 26)]

Comment on attachment 9059813 [details]
Bug 1545838 - Update Fennec's AMO URL; r?petru

Low risk, uplift approved for 67 beta 14, thanks.

Comment 10

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/releases/mozilla-beta/rev/c53e3fd76964

Comment 11

[Laurentiu Apahidean]

Hello,

I have verified the issue on Beta 67.0b14 using OnePlus A3000 (Android 6.0.1) and Samsung Galaxy S8+ (Android 8.0.0).
The correct link is opened after taping the "Browse Firefox's Recommended Extensions".

Bug 1544525 is verified as well on Beta.

Due to my findings, I'll mark this issue as verified in Firefox 67.