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

PRMT-4402 #78

Merged
merged 3 commits into from
Mar 21, 2024
Merged

PRMT-4402 #78

merged 3 commits into from
Mar 21, 2024

Conversation

martin-nhs
Copy link
Contributor

No description provided.

joefong-nhs and others added 3 commits March 14, 2024 11:38
…uelize Dependencies (#73)

* [PRMT-4565] store a working version of dynamodb localstack setup.

* [PRMT-4565] add basic test for dynamodb client

* [PRMT-4565] minor change in config, small change to generate_secure_string to address a mac specific problem

* [PRMT-4565] Read dynamodb table name from SSM, change the table name for local testing

* [PRMDR-4565] Replace localstack with dynamodb-local for running dynamodb integration test

* [PRMT-4565] Change timestamp format to include timezone offset

* [PRMT-4566] add new version of updateFragmentAndCreateItsParts, start separating model and repository layer

* [PRMT-4566] Config test setup and teardown

* [PRMT-4566] Add option to query specific record type

* [PRMT-4566] Add state enums

* [PRMT-4566] separate client and repository layer

* [PRMT-4566] Reorganise new codes

* [PRMT-4566] Add terraform iam policy for accessing the ehr-transfer-tracker table

* [PRMT-4566] Reorganise code, use CreatedAt instead of UpdatedAt to determine latest record

* [PRMT-4566] Migrate all tests in message-repository.integration.test.js to new dynamodb-based implementation

* [PRMT-4566] Rename test files

* [PRMT-4566] Continue migrating database methods and integration tests. Store current progress

* [PRMT-4566] Migrating integrated test for more methods

* PRMT-4566 reorganise code

* [PRMT-4566] Migrated all database-related integration tests. Store current progress

* [PRMT-4566] Rename tests files

* [PRMT-4566] Rename methods to avoid import conflict, add deprecated mark and ticket number to old postgres related codes

* [PRMT-4566] Rename methods

* [PRMT-4566] minor change in wordings

* [PRMT-4566] Correct dynamodb related env vars

* [PRMT-4566] Change DeletedAt field to store ttl time of 8 weeks later

* [PRMT-4567] Migrated store-message-controller.js and its tests

* [PRMT-4567] migrated message-location-controller.js and its tests

* [PRMT-4567] migrated app integration tests for POST /messages endpoint

* [PRMT-4567] rename test files

* [PRMT-4567] Migrate get-fragment-controller.js and integration tests for /fragments/${conversationId}/${fragmentMessageId} endpoint

* [PRMT-4567] migrated endpoint `GET /patients/:nhsNumber` and controller at `src/api/patients/patient-details-controller.js`

* [PRMT-4567] Migrate controller `src/api/patients/health-record-controller.js` and endpoint `GET /patients/:nhsNumber/health-records/:conversationId`

* [PRMT-4567] minor clean up

* [PRMT-4567] migrated controller delete-ehr-controller.js

* [PRMT-4567] Add integration test for endpoint DELETE /patients/:nhsNumber

* [PRMT-4567] Remove postgres db checking from health check endpoint

* [PRMR-4567] Unplug postgres db from application, comment out redundant tests

* [PRMT-4567] mark redundant tests as deprecated, fix unit tests

* [PRMT-4567] Update dtest.yml, remove postgresdb healthcheck from docker test

* [PRMT-4567] comment out postgresdb migration command in ./tasks

* [PRMT-4567] minor bugfix

* [PRMT-4567] Update README for the change related to database and integration tests

* [PRMR-4567] improve configs about running integration test with dojo

* [PRMT-4567] Fix test related to DeletedAt timestamp

* [PRMT-4567] Amend Layer sort key prefix to be all capital case

* [PRMT-4567] use the string "STORED_IN_REPOSITORY" for complete status of CORE and FRAGMENT

* [PRMT-4567] Fixed an issue related to DeletedAt time and DST

* [PRMT-4568] Remove postgres-db related code

* [PRMT-4568] Continue removing postgres-db related code

* [PRMT-4568] Remove redundant comments

* [PRMT-4568] Fix broken import of deprecated code

* [PRMT-4568] Remove postgres related packages from dependencies

* [PRMT-4568] Remove postgresdb from docker config, remove .sequelizerc

* [PRMT-4568] Continue removing postgres-db related stuffs

* [PRMT-4568] Remove postgres db related content from ./tasks (except terraform things)

* [PRMT-4568] Run linter and formatter

* [PRMT-4568] Remove commented out code

* [PRMT-4568] remove sequelize from package.json

* [PRMT-4568] Fix Dockerfile (remove references to deleted file)

* [PRMT-4568] remove db:migrate from package.json

* [PRMT-4568] Use underscore instead of hyphen in terraform code

* [PRMT-4568] Remove trailing comma

* [PRMT-4568] Fix typo

* [PRMT-4568] fix typo (Outbound --> OUTBOUND) at TransferStatus

* [PRMT-4568] Adapt to database schema change: sort key for core change from `CORE#{messageId}` to just `CORE`

* [PRMT-4568] address PR comments
@martin-nhs martin-nhs merged commit 94d271b into main Mar 21, 2024
1 check passed
@martin-nhs martin-nhs deleted the PRMT-4402 branch March 21, 2024 10:15
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.

2 participants