Skip to content
This repository was archived by the owner on Jul 31, 2023. It is now read-only.

rems endpoint updates#38

Merged
smalho01 merged 2 commits into
masterfrom
databaseUpdates
Aug 25, 2022
Merged

rems endpoint updates#38
smalho01 merged 2 commits into
masterfrom
databaseUpdates

Conversation

@smalho01

@smalho01 smalho01 commented Aug 24, 2022

Copy link
Copy Markdown

you should be able to submit the patient enrollment forms, prescriber enrollment form, and knowledge assessment form and the REMS endpoint should be able to adjust the status of the rems request dynamically instead of having everything stubbed out.

The endpoint changes break the REMS UI in dtr in the case of forms that do not create a new rems request case (basically everything that's not a patient enrollment form). Dtr is supposed to be all white with no data shown in these cases, updating that UI will be done in follow up work (REMS-232)

Best way to test this would be to connect to the h2 console at http://localhost:8090/h2-console (jdbc url: jdbc:h2:mem:myDb;DB_CLOSE_DELAY=-1, user: sa, pass: "") and watch the met requirements table to see how it populates as the forms are submitted. Restarting CRD will reinitialize the database to its starting position and remove any additional met requirements if that helps with testing.

There are also still some aspects that need to be cleaned up with the database (for example setting the drug - rems relationship and setting the completed resources for each MetReq) however these are not critical for the 0.6 demo and will be finished in a follow up ticket (REMS-233)

@zacharyrobin zacharyrobin left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

tested out each iteration I could think of and verified ETASU updates correctly. Looks good on my end.

@smalho01 smalho01 merged commit 65eaa1d into master Aug 25, 2022
@smalho01 smalho01 deleted the databaseUpdates branch November 2, 2022 18:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants