Because:
* we want to hide the Taxes & Fees for countries with inclusive tax.
This commit:
* updates the PlanDetails component to hide Taxes & Fees when tax is
inclusive.
* adds basic tests for Taxes & Fees.
Closes #FXA-6391
Because:
- We want to convert content-server views to React and are starting by building out the front-end in Storybook. This PR converts the reset_password view. Additional complexity will be added in (a) later ticket(s).
This commit:
- Create ResetPassword UI in React, including l10n, storybook, initial metrics, initial tests.
- Does not include units tests for success/failure methods, error handling.
- Create additional reusable components (including l10n, storybook, initial tests): CardHeader, LinkRememberPassword, WarningMessage
- Add placeholder for ResetPassword function in models/Accounts
- Update InputText and Tooltip with new properties and updated styling for reuse in ResetPassword
- Move 'cta-xl' TW class from content-server to fxa-react
- Fix spelling of "focussed" to "focused" in InputText and Checkbox
Closes #FXA-6338, FXA-6447
Because:
* For exclusive tax, display the tax amount in subscription management
This commit:
* Adds a new component, PriceDetails, that can be used to display the
price with/out tax, and price intervals with/out tax.
* Adds PriceDetails component to Subscription Management, and updates
the copy used.
* Preview invoice for each subscription to get the current tax amount
* Update preview invoice and subsequent invoices endpoints to return
total and subtotal *_excluding_tax amounts.
Closes #FXA-6187
Because:
- the mysql module's connection pool never close any of its connections
This commit:
- close a connection after it idles for a configurable amount of time
Because:
- We want to speed up pipeline startup.
This Commit:
- Optimizes docker base image builds for the smallest images possible.
- Enables hard links for yarn cache
- Enables global yarn cache
- Avoids needlessly installing playwright browsers.
- Avoids needlessly running yarn install.
- Uses mozilla/fxa-circleci:ci-base-latest image for running test pipelines
- Uses mozilla/fxa-circleci:ci-base-browsers image for running functional tests pipelines
- Creates configurable executors that can be reused across pipelines
- Upgrades to yarn 3.3.0
- Enabled direct check out of PR code to test, which is faster than circle ci’s checkout command.
- Upgrades functional test to X-Large. This was already the case for playwright tests, but is now extended to content server tests too. This decision was made due to running lots of pipelines and realizing flakiness was largely due to CPU or memory hitting 100% for long periods of time.
- Turns off tracing, since it saves a bit of runtime.
This commit:
* Reverts a recent change that updated a bg-image to use an img src and uses TW to load the image. We may have a bug in content-server around updating img src image hashes in at the build step unless it's intentional to always use bg-images, and this commit inlines the SVGs instead as they are tiny and keeps a11y wins
* Adjusts bottom-padding slightly for more consistency with prod
Because:
* The close SVGs are not loading on the build step due to 'img src' path hashes not being updated in the outputted HTML
Closes: FXA-6453
Because:
- we delete from the pushbox db directly when deleting an account now
This commit:
- add the pushbox module to the account routes when used from the
delete account script
Because:
- Storybook and email tests started failing once the l10n started shipping a separate branding file.
This Commit:
- Loads the branding.ftl file as well as the auth.ftl file for when translating emails.
- Changes the l10n postAddAccountRecovery-title-2 to postAddAccountRecovery-title2, which appears to be the current state in the ftl file.
- Changes spelling from cancelled to cancelled for certain tests... This is a bit puzzling, and could use a second look.
Because:
- Concatenating shared FTL strings into multiple packages before sending for localization created duplication of localization work, and we want each string to only be translated once.
This commit:
- Remove concatenation of shared files (branding.ftl, fxa-react ftl files) from grunttasks
- Add grunttasks in fxa-react to concatenate fxa-react FTL files into one react.ftl file
- Add grunt to fxa-react dev dependencies
- Update fxa-react scripts to run l10n and ftl tasks
- Update clone-l10n script to distribute react.ftl to packages
- Bundle branding and shared into AppLocalizationProvider
Closes #FXA-6388
Because:
- we delete from the pushbox db directly when deleting an account now
This commit:
- add the pushbox module to the account routes when used from the
delete account script
Because:
- We want to separate unit tests and integrations tests
This Commit:
- Tags all unit tests, i.e. tests that can run without infrastructure and in a fairly short amount of time, with #unit.
- Adds a test:unit npm command to each package
- Adds a test:integration npm command to each package, and adds '#integration' tags to jest tests since jest has no --invert option like mocha.
Because:
- Storybook and email tests started failing once the l10n started shipping a separate branding file.
This Commit:
- Loads the branding.ftl file as well as the auth.ftl file for when translating emails.
- Changes the l10n postAddAccountRecovery-title-2 to postAddAccountRecovery-title2, which appears to be the current state in the ftl file.
- Changes spelling from cancelled to cancelled for certain tests... This is a bit puzzling, and could use a second look.
Ankita Shrivastava
Reino Muhl
Dan Schomburg
dschom
Lauren Zugai
Valerie Pomerleau
Ankita Shrivastava
Barry Chen
Vijay Budhram
Vijay Budhram
Lisa Chan
dependabot[bot]