-
Notifications
You must be signed in to change notification settings - Fork 913
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Explore changing /firefox/all/ to be less process intensive. #14324
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #14324 +/- ##
==========================================
- Coverage 76.85% 76.42% -0.44%
==========================================
Files 156 156
Lines 8149 8198 +49
==========================================
+ Hits 6263 6265 +2
- Misses 1886 1933 +47 ☔ View full report in Codecov by Sentry. |
I've started but I'm not finished and I don't have time to leave notes for someone else to pick it up :/ Fortunately, there's no rush. |
8967869
to
e80f8a5
Compare
e80f8a5
to
435c116
Compare
I pushed my most recent changes.
|
One-line summary
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.Unfinished business
Issue / Bugzilla link
#9845
Testing
This currently is hosted on demo2 at /en-US/firefox/drill/
Mobile:
Localization:
QR codes go to expected App/Play store: