Skip to content

PRMT-4402#78

Merged
martin-nhs merged 3 commits intomainfrom
PRMT-4402
Mar 21, 2024
Merged

PRMT-4402#78
martin-nhs merged 3 commits intomainfrom
PRMT-4402

Conversation

@martin-nhs
Copy link
Copy Markdown
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
@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