Because:
- We cannot add parameters to job triggered through circleci's API
This Commit:
- Removes parameters from the `test-content-server-remote` job
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
* Organizing l10n strings in per-component files aligns payments-server with the setup used for auth-server and settings. Splitting the FTL files also improves maintability by more tightling coupling the strings with the components where the strings are used.
This commit
* Create a temporary branding file in fxa-payments-server/src
* Create a gruntfile with tasks for FTL concatenation and watching
* Rename the destination file from main.ftl to payments.ftl
* Switch payments-server's default locale from en-US to en and update all references of en-US to en
* Create individual FTL files per component and move messages to their respective component FTL file
* Add a l10n entry in the package readme
* Update clone-l10n.sh to copy 'payments' ftl files instead of 'main'
* Update AppLocalizationProvider to use 'payments' bundles
* Replace setupFluentLocalizationTest with getFtlBundle/getFtlFromPackage from fxa-react
* Remove setupFluentLocalizationTest function (no longer used in fxa-settings)
* Add merge-ftl:test task to package.json start and test:frontend scripts
* Remove legacy strings from FTL files
* Update currency and date formats to use 'en' as default locale
* Remove legacy paths from .gitignore files in payments-server
* Move remaining paths to global .gitignore
* Delete .gitignore files in payments-server
Closes #FXA-5996 and #FXA-6255
Because:
- When starting tracing with open telemetry the network name would sometimes collide and result in error.
This Commit:
- Sets an 'fxa' network name
- Applies this network to services
- Allows services to communicate by their names
Because:
- We need to pull the latest pdfs from legal-docs into the fxa
assets folder.
This commit:
- Create a script to copy the latest pdfs into assets/legal.
- Create a Github Action to automatically pull in the pdfs from
legal-docs, and open a pull request to move pdfs into assets/legal.
Closes #fxa-5100
Because:
* Test all didn't run in parallel to take advantage of more cores and
a few older tests were flakey.
* Tests would hang as packages had a default test that was in watch
mode.
This commit:
* Updates several flakey tests and runs the test all in parallel for a
shorter completion time.
* Updates settings and payment-server packages to have new watch command
and deafult test command that runs the tests.
Closes #FXA-6096
Because:
* We updated the local mysql to a version that doesn't pass auth-server
tests.
This commit:
* Drops the local mysql to a version that the test passes.
Closes #FXA-6068.
Because:
- We want to start trace capture at client side
- We can't report trace data directly to cloud trace
This Commit:
- Adds support for client side tracing in tracing/browser-tracing.ts
- Some refactors to fxa-shared/tracing to ensure code is reused
- Cleans up tests
- Introduces traceparent header and tracestate html headers
- Opens up OTLP ports on jaeger
- Introduces a clientName to trace config
- Adds an open telemetry collector service (otel-collector)
- Uses this services for all trace capture
- Adds flow id to trace id headers so traces can be validated
Because:
- We want to start trace capture at client side
This Commit:
- Adds support for client side tracing in tracing/browser-tracing.ts
- Some refactors to fxa-shared/tracing to ensure code is reused
- Cleans up tests
- Introduces traceparent header and tracestate html headers
- Opens up OTLP ports on jaeger
- Introduces a clientName to trace config
Because:
- Subscription Services is prepareing for initial release
This commit:
- Adds initial set of Subscription Services legal documents.
Closes #FXA-6036
Because:
- When not enabled Jaeger was exiting with an 'error' status which was confusing.
This commit:
- Adjust infrastructure.config.js so that Jaeger exists cleanly with a 'stopped' status.
- Reports a better log message so that it's clear Jaeger is not required to run FxA.
Because:
- We want to be able to see trace information
- We want to be able to export trace information to google cloud
This Commit:
- Adds utility class to fxa-shared for configuring and initializing open telemetry
- Initializes open telemetry in the auth server
- Forces resolution of google-gax.
- Forces resolution of @grpc/grpc-js.
- Adds the jaeger docker container for viewing traces locally. This is accessible on localhost:16686.
Note, adding @google-cloud/opentelemetry-cloud-trace-exporter resulted
in a version conflicts for google-gax and @grpc/grpc-js, which caused
runtime failures when interacting with @google-cloud/firestore. After
many experiments, this seems to do the trick.
Because:
- Some affected packages were not being compiled
This pull request
- Includes the first line output by 'yarn workspaces list', so that the package is included in subsequent checks.
[skip ci]
Because:
* The fxa-content-server-l10n is still using master as default branch, and we want to update it to main.
This commit:
* Updates references from master to main in automation
* Fixes or removes master references in docs
* Fixes a few typos
Because:
- we want to upgrade to mysql 8
This commit:
- upgrade mysql for local dev
- start mysqld with default auth set to native password for backwards
compat with the 'mysql' module
Because:
* We should be informed about TS errors before a patch merges into main.
This commit:
* Adds a new script to run as the first task in the test-many job that compiles any modified back-end Typescript packages and any packages that depend on them.
Closes#12823
Co-authored-by: Barry Chen <bchen@mozilla.com>
Because:
* `gh-pages.sh` and `upload-assets-to-cdn.sh` are no longer used
This commit:
* removes them. Once this lands I will update the documentation as
well.
On M1 we currently sometimes need to 'yarn restart services' after a 'yarn start' before things start working. Maybe if we wait for the containers to be responsive before starting services we can avoid this. The important one seems to be mysql.
Because:
- We want to be able to generate a single FTL file for fxa-content-server-l10n to pick up
- We want to be able to pull in existing auth locales from fxa-content-server-l10n
This commit:
- Adds a new grunt task for generating a single FTL file, placing it in a location fxa-content-server-l10n will find it
- Updates the clone-l10n script to check for auth server locales in the fxa-content-server-l10n repo
In order to decomission fxa-auth-db-mysql we need a new place for its migrations. Since we've also got other dbs and migrations scattered around, now is a good opportunity to bring them all together since they don't live in separate repos anymore.
In local dev the patcher now runs as part of the mysql docker process startup.
This refactoring fixes checkout of the official L10N Git repository by using the
existing environment variable FXA_L10N_SHA. By default, the commit history depth
for checkout into a specific commit is set to 1000 but can be increased with
environment variable FXA_L10N_DEPTH.
New features include:
* checkout a different repository (e.g. a fork) with variable FXA_L10N_REPO
* checkout a branch other than "master" with variable FXA_L10N_BRANCH
* symlink to an existing local checkout instead with variable FXA_L10N_COPY_FROM
* clone to/link from a different in-package directory with variable FXA_L10N_DOWNLOAD_PATH
* skipping the clone/update with variable FXA_L10N_SKIP
Current behavior is retained, the default values for the above variables are:
FXA_L10N_REPO = https://github.com/mozilla/fxa-content-server-l10n.git
FXA_L10N_BRANCH = master
FXA_L10N_COPY_FROM = <empty>
FXA_L10N_DOWNLOAD_PATH = fxa-content-server-l10n
FXA_L10N_SKIP = <empty>
Closes: #7767
Because:
- we have multiple similar scripts that pull translations from the
fxa-content-server-l10n repo
- we need that functionality for new settings
This commit:
- combine the scripts into a single one
dschom
Valerie Pomerleau
Dan Schomburg
Barry Chen
Reino Muhl
Ben Bangert
Julian Poyourow
Bianca Danforth
Francesco Lodolo
郭佳慧
Vijay Budhram
Florian Dieminger
Wil Clouser
Danny Coates
Jody Heavener
Michael Kane Juncker
lesley
lesleyjanenorton
Dave Justice