darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-02-06 03:13:42 -05:00
Code Issues Packages Projects Releases Wiki Activity

Remove E2E environment variables from the manifest

Browse Source 
This changeset removes the E2E environment variables from the admin manifest file.  While these might be needed on the API side to ensure that they are exposed and able to be used in the staging environment, we do not need them to be enabled for the admin app; they should only be present in the GitHub Actions that are run in a PR, and the .github/workflows/checks.yml file takes care of that and injects them directly into the container environment that the end-to-end tests run in.

Signed-off-by: Carlo Costino <carlo.costino@gsa.gov>
This commit is contained in:
Carlo Costino
2024-04-23 19:43:56 -04:00
parent 27117d8983
commit f5f5d0359c
1 changed files with 0 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
manifest.yml
View File

@@ -37,11 +37,6 @@ applications:
# Credentials variables
NOTIFY_E2E_AUTH_STATE_PATH: (( NOTIFY_E2E_AUTH_STATE_PATH ))
NOTIFY_E2E_TEST_EMAIL: (( NOTIFY_E2E_TEST_EMAIL ))
NOTIFY_E2E_TEST_PASSWORD: (( NOTIFY_E2E_TEST_PASSWORD ))
NOTIFY_E2E_TEST_URI: (( NOTIFY_E2E_TEST_URI ))
ADMIN_CLIENT_SECRET: ((ADMIN_CLIENT_SECRET))
ADMIN_CLIENT_USERNAME: ((ADMIN_CLIENT_USERNAME))
DANGEROUS_SALT: ((DANGEROUS_SALT))