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

WSTEAMA-1458 - Setup static assets for 2nd preview environment #12215

Conversation

louisearchibald
Copy link
Contributor

@louisearchibald louisearchibald commented Nov 28, 2024

Resolves JIRA [1458]

Overall changes

Sets up static assets in Simorgh for a secondary preview environment

Code changes

  • Renames existing preview.env file to preview1.env.
  • Amends SIMORGH_PUBLIC_STATIC_ASSETS_PATH to reflect this name change.
  • Updates existing command build:preview to be build:preview1 & updates the path to .env file in the script body.
  • Creation of a preview2.env with the SIMORGH_PUBLIC_STATIC_ASSETS_PATH value pointing to this new file.
  • Creation of a build:preview2 command with a path in the script body pointing to the file created above.

Testing

Branch was deployed to the preview environment and then tested. The environment was not broken by any of the changes, including the change made in this infrastructure PR. The network tab was inspected for any unexpected 404s (none present), and the new static asset for preview1 is now shown in the header.

Screenshot 2024-12-06 at 11 27 24

@louisearchibald louisearchibald self-assigned this Nov 28, 2024
@louisearchibald louisearchibald marked this pull request as ready for review November 28, 2024 14:54
envConfig/preview1.env Outdated Show resolved Hide resolved
envConfig/preview2.env Outdated Show resolved Hide resolved
Copy link
Contributor

@pvaliani pvaliani left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@karinathomasbbc karinathomasbbc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for making the changes!

louisearchibald and others added 5 commits November 29, 2024 12:58
…nment' of github.com:bbc/simorgh into WSTEAMA-1458-setup-static-assets-for-2nd-preview-environment
This reverts commit 21355bd.
…nment' of github.com:bbc/simorgh into WSTEAMA-1458-setup-static-assets-for-2nd-preview-environment
package.json Show resolved Hide resolved
@louisearchibald louisearchibald merged commit ae0ab66 into latest Dec 10, 2024
11 checks passed
@louisearchibald louisearchibald deleted the WSTEAMA-1458-setup-static-assets-for-2nd-preview-environment branch December 10, 2024 12:36
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.

4 participants