Skip to content

Commit

Permalink
Explore changing /firefox/all/ to be less process intensive. (#14324)
Browse files Browse the repository at this point in the history
This branch is an effort to eventually replace /firefox/all/ to avoid a very heavy paylaod to the browser by breaking the choices into separate pages rather than a single page.

* UI Changes
* Add windows store to flow
* Update URLs
* Update `firefox_url` helper
* add ios-beta page
* remove macros used by old /all
* show ESR next download button when available
* noindex child pages after talking to Adria
* Switch brand term for Firefox
* Add checks for 404s

---------

Co-authored-by: Stephanie Hobson <[email protected]>
Co-authored-by: Jan Brasna <[email protected]>
  • Loading branch information
3 people authored Aug 29, 2024
1 parent 0f00a74 commit 7fa85e0
Show file tree
Hide file tree
Showing 34 changed files with 1,547 additions and 2,822 deletions.
148 changes: 0 additions & 148 deletions bedrock/base/templates/product-all-unified-macros.html

This file was deleted.

23 changes: 12 additions & 11 deletions bedrock/firefox/redirects.py
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ def mobile_app(request, *args, **kwargs):
"firefox-welcome-17-fr",
"firefox-browsers-mobile-get-app",
"firefox-browsers-mobile-focus",
"firefox-all",
]

for p in product_options:
Expand All @@ -49,14 +50,14 @@ def mobile_app(request, *args, **kwargs):
redirectpatterns = (
# overrides
# issue 8096
redirect(r"^firefox/beta/all/?$", "firefox.all", anchor="product-desktop-beta"),
redirect(r"^firefox/developer/all/?$", "firefox.all", anchor="product-desktop-developer"),
redirect(r"^firefox/aurora/all/?$", "firefox.all", anchor="product-desktop-developer"),
redirect(r"^firefox/nightly/all/?$", "firefox.all", anchor="product-desktop-nightly"),
redirect(r"^firefox/organizations/all/?$", "firefox.all", anchor="product-desktop-esr"),
redirect(r"^firefox/android/all/?$", "firefox.all", anchor="product-android-release"),
redirect(r"^firefox/android/beta/all/?$", "firefox.all", anchor="product-android-beta"),
redirect(r"^firefox/android/nightly/all/?$", "firefox.all", anchor="product-android-nightly"),
redirect(r"^firefox/beta/all/?$", "firefox.all.platforms", to_args=["desktop-beta"]),
redirect(r"^firefox/developer/all/?$", "firefox.all.platforms", to_args=["desktop-developer"]),
redirect(r"^firefox/aurora/all/?$", "firefox.all.platforms", to_args=["desktop-developer"]),
redirect(r"^firefox/nightly/all/?$", "firefox.all.platforms", to_args=["desktop-nightly"]),
redirect(r"^firefox/organizations/all/?$", "firefox.all.platforms", to_args=["desktop-esr"]),
redirect(r"^firefox/android/all/?$", "firefox.all.platforms", to_args=["android-release"]),
redirect(r"^firefox/android/beta/all/?$", "firefox.all.platforms", to_args=["android-beta"]),
redirect(r"^firefox/android/nightly/all/?$", "firefox.all.platforms", to_args=["android-nightly"]),
# bug 831810 & 1142583 & 1239960, 1329931
redirect(r"^mwc/?$", "https://support.mozilla.org/products/firefox-os", re_flags="i"),
# bug 748503
Expand Down Expand Up @@ -107,10 +108,10 @@ def mobile_app(request, *args, **kwargs):
# bug 727561
redirect(r"^firefox/search(?:\.html)?$", "firefox.new"),
# bug 860865, 1101220, issue 8096
redirect(r"^firefox/all-(?:beta|rc)(?:/|\.html)?$", "firefox.all", anchor="product-desktop-beta"),
redirect(r"^firefox/all-aurora(?:/|\.html)?$", "firefox.all", anchor="product-desktop-developer"),
redirect(r"^firefox/all-(?:beta|rc)(?:/|\.html)?$", "firefox.all.platforms", to_args=["desktop-beta"]),
redirect(r"^firefox/all-aurora(?:/|\.html)?$", "firefox.all.platforms", to_args=["desktop-developer"]),
redirect(r"^firefox/aurora/(?P<page>all|notes|system-requirements)/?$", "/firefox/developer/{page}/"),
redirect(r"^firefox/organizations/all\.html$", "firefox.all", anchor="product-desktop-esr"),
redirect(r"^firefox/organizations/all\.html$", "firefox.all.platforms", to_args=["desktop-esr"]),
# bug 729329
redirect(r"^mobile/sync", "firefox.features.sync"),
# bug 882845
Expand Down
Loading

0 comments on commit 7fa85e0

Please sign in to comment.