-
Notifications
You must be signed in to change notification settings - Fork 15
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
feat: introduce config environment variable (#3006)
* ignore .next in jest * mute console error from jest results * rename app_env to nbs_auth docker compose --env-file .env.local --profile "*" up to test with different configs * Remove app env check around s3 client * add collection variables for env groups * fix typos in .env files * create single azure-storage image that contains both azurite and azure cli * add additional chaining to run specific collections * fix local-dev command to use collection * update create-seed-data to use different configs depending on metadata * rename schemas * update .env.sample to include collection * remove azure-storage-init from orchestration * update orchestration docker compose to add collection * add metadata db schema in the env variable * remove undefined values * remove nbs_auth value * Remove redundant variables * change to aws non integrated and add timeout * add docker ignore * add more to docker ignore * Comment all from .env * fix tests that were relying on .env file * make metadata variables optional * pull out saveFhirMetadata into it's own function * delete .env * Use PersistenceResponse instead of Nextjs response * console log error messages * Do not close pool when done with mssql https://www.npmjs.com/package/mssql#advanced-pool-management:~:text=Also%20notice%20that,of%20the%20script. * increase sqlserver timeout to 30 s * don't return anything from save ecr data query * be more specific with env variable names * remove return for ecr data * update .env.sample variables * remove .env * Remove app_env from dockerfile * make metadata env variable optional * fix tests without .env * move getDb from top level to be inside methods * add dockerignore * Remove NEXT_PUBLIC_BASEPATH since it is determined at build time anyways * fix first batch of seed script conflicts * use rewrite instead of redirect to keep url * set pool min to 1 * fix middleware test after changes to middleware rewrite to error page * replace dotenvx with next runtime env and instrumentation * remove console logs * update collection in orchestration * change docker compose collection to default to sql server non integrated * Rename PersistenceResponse to SaveResponse * Remove build from convert-seed-data * remove useless console.error logs * update instrumentation tests * fix middleware * permanently set base path in ecr-viewer * Rename middleware to be a .ts file * replace a tags with Next's Link tag * remove title from Header link * retype base_path to be a string * fix metrics test * update header snapshot test * address middleware tests * add back button to retrieval failed * remove unused rewrite in next config * update matcher * Prevent access to /api/fhir-data * [pre-commit.ci] auto fixes from pre-commit hooks * fix url in readme * remove accident commited next-runtime-env * [pre-commit.ci] auto fixes from pre-commit hooks * use next-runtime-env to access NEXT_PUBLIC_NON_INTEGRATED * [pre-commit.ci] auto fixes from pre-commit hooks * use proper variable name for middleware test * simplify clear-local command * rename collection to CONFIG_NAME * update design script to handle config variable * fix package.json * update lighthouse startup * add dummy keys to .env.sample * Filter by conditions in eCR Library (frontend) (#2981) * First pass, filter conditions functionality * add select/deselect all functionality * update some styling, maintain checkbox state when toggling filter button * styling updates, wip * checkbox color, add icon, add uswds sprite.svg to assets * adjust padding to fix checkbox focus ring cut off * fix icon not displaying by adding static file route * fix unintentional scrolling bug * update filter row top border * wip, add comments, decompose conditions filter to separate const * fix scrolling bug by adding position-relative * add snapshot and unit tests * add JSDocs * remove css classes and use utilities instead * update snapshot test * update select all/deselect all functionality s.t. default is all conditions checked, update tests * update so that filters reset if clicking off filter before clicking the Apply button, add tests * update basepath so it works in prod * update tests * update styles in diff button states, update icon size, make capitalization consistent * Remove log Co-authored-by: Mary McGrath <[email protected]> * use as form/fieldset, update sync state bug, update tests * remove manual checkboxing for select all, lets react handle the render * rework state management, update tests * code review changes, minor * query should persist over a reload * update backend so default (all conditions) would leave out condition param from URL query, add/update tests * use import for icon * Update base_path env var name Co-authored-by: Boban <[email protected]> * update snapshot test * re-use resetFilterConditions * one more nit * update ecr library height to accommodate fiter bar * update env var name for base path --------- Co-authored-by: Mary McGrath <[email protected]> Co-authored-by: Boban <[email protected]> * fix base path for filters * update config name in create-seed-data * move start transaction in the if statement for saveMetadataToSqlServer * remove random transaction from conditons api sql server * move start transaction in the if statement for saveMetadataToSqlServer * remove random transaction from conditons api sql server * [pre-commit.ci] auto fixes from pre-commit hooks * refactor service for condition to simplify logic * [pre-commit.ci] auto fixes from pre-commit hooks * update filter test * update jsdocs * add nbs pub key to environment variables * remove NBS_AUTH variable from local environment setup * fix broken merge * fix where clause fpor date statement sql server * Update .env.sample to replace mssql server_server with server_host --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Angela The <[email protected]> Co-authored-by: Mary McGrath <[email protected]>
- Loading branch information
1 parent
9112d57
commit 61ac6d5
Showing
46 changed files
with
390 additions
and
112 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,3 +1,2 @@ | ||
DATABASE_URL=postgres://postgres:pw@postgres:5432/ecr_viewer_db | ||
APP_ENV=test | ||
AZURE_CONTAINER_NAME=ecr-viewer-files |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.