Skip to content
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

Version 1.31.0 #1136

Merged
merged 5 commits into from
May 9, 2023
Merged

Version 1.31.0 #1136

merged 5 commits into from
May 9, 2023

Conversation

cea2aj
Copy link
Member

@cea2aj cea2aj commented May 9, 2023

Changes

A new cloudRegion config option was introduced which supports both "us" and "eu". This option determines which cloud region the search assets are loaded from. If sdkVersion 1.16 or greater is used, then the EU search endpoints will be used when the cloudRegion is set to "eu".

nmanu1 and others added 5 commits December 12, 2022 16:19
Merge master (v1.29.5) into develop
Merge master (v1.30.0) into develop
upgrade express (minor), serve (major), puppeteer (major), and loader-utils (patch) to resolve various non-critical sub-dependency vulnerabilities in snyk. Ran npm audit fix to to resolve various other vulnerabilities called out in npm.

TEST=manual

Served test-site without errors to confirm express and serve versions are compatible. Successfully ran tests to ensure new puppeteer version is compatible
Add a cloud region config option to the global_config to allow either 'us' (default), or 'eu' to be set as the cloud region.

The cloud region determines where the search assets and ytag are sourced from.

J=BACK-2277
TEST=manual, auto

Using the test site, set the cloudRegion to "eu" and confirmed the assets are loaded from the EU CDN and executed searches against the EU search endpoints
Bump the version numbers for version 1.31.0

J=BACK-2277
TEST=none
@coveralls
Copy link

Coverage Status

Coverage: 8.737% (-0.006%) from 8.743% when pulling 0b43256 on develop into f87864a on master.

@cea2aj cea2aj merged commit 64b4730 into master May 9, 2023
22 of 24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants