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

CARDS-2512: PM Acute Care methodology #1755

Merged
merged 24 commits into from
Sep 17, 2024
Merged

CARDS-2512: PM Acute Care methodology #1755

merged 24 commits into from
Sep 17, 2024

Conversation

sdumitriu
Copy link
Member

@sdumitriu sdumitriu commented Apr 16, 2024

  • Build with mvn clean install -Pdocker

  • In compose-cluster make a docker-compose file with python3 generate_compose_yaml.py --mssql --cards_project cards4prems --oak_filesystem --dev_docker_image --composum --adminer --smtps --smtps_test_container --smtps_test_mail_path ~/cards-mail --server_address localhost:8080 then start it up with docker-compose build && docker-compose up

  • At http://localhost:1435/?mssql=mssql&username=sa&db=master&ns=path&import= import the attached SQL files, sqls.tar.gz (unzip the archive first to get 3 sql files)

  • Log in to cards at http://localhost:8080/

  • Import YE visits: http://localhost:8080/Subjects.importClarity?dayToQuery=-7&config=Your%20Experience%20-%20Inpatient%20and%20ED%20discharge%20events

  • Check that visits for patients 1, 2, and 3 were imported

  • Import YVM visits: http://localhost:8080/Subjects.importClarity?dayToQuery=-2&config=Your%20Voice%20Matters%20-%20PMCC%20outpatient%20visits

  • Check that YVM visits for patients 11, 12, 13, 21 and 29 were imported

  • Import AC visits: http://localhost:8080/Subjects.importClarity?dayToQuery=-7&config=PMH%20Acute%20Care%20visits

  • Check that AC visits for patients 21, 22 and 23 were imported; make sure 21 has TWO visits, one for YVM and one for AC

  • "Import" AC discard visits: http://localhost:8080/Subjects.importClarity?dayToQuery=-2&config=PMH%20AC%20Trumps%20YVM

  • Check that the YVM visit for 29 was deleted, and the YVM visit for 21 was not deleted

  • In the end there should be 10 Visit Information forms, ED for 1&2, IP for 3, YVM for 11, 12, 13, 21, and AC for 21, 22, 23

  • Stop docker-compose and clean up with docker-compose rm -vf && ./cleanup.sh

@marta- marta- changed the base branch from dev to CARDS-2544-multivalued-props-in-emails July 5, 2024 11:25
@marta-
Copy link
Contributor

marta- commented Jul 5, 2024

Rebased on #1789 - emails can now be tested @sashaandjic . The first paragraph of the emails should include the unit visited by the patient, as specified in the provider answer of Visit information.

Email schedule: initial - 7 days after visit, reminders - 14 and 21 days after visit.

@sashaandjic

This comment was marked as outdated.

Base automatically changed from CARDS-2544-multivalued-props-in-emails to dev July 9, 2024 15:08
@marta- marta- added the urgent Needs to be completed and merged asap label Aug 20, 2024
Copy link
Contributor

@sashaandjic sashaandjic left a comment

Choose a reason for hiding this comment

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

tested and EDITED
I was wrong about and extraNot applicable option for question pmac_7_ad_5. It is supposed to be there.

@sashaandjic sashaandjic added question Further information is requested tested Passed manual testing, needs code review and removed urgent Needs to be completed and merged asap question Further information is requested labels Aug 21, 2024
sdumitriu and others added 13 commits September 1, 2024 21:07
Add a placeholder Acute Care questionnaire, currently just a copy of OED
Add questionnaire sets for AC and AC+IP
Add the two new clinics
Configure the Clarity import
Added CopyAnswers and Statistics (placeholders)
* Renamed the questionnaire
* Corrected the "Overall" question id based on the updated questionnaire structure
* Updated the Statistic config
* Added CopyAnswers config
Minor reordering of side bar entries
* Fixed the pagination mode
* Minor indentation and wording fixes
* Added hidden reference question for department
* Reorganized the questions
* Naming cleanup
Renamed PMH to PM in Acute Care configs
marta- and others added 6 commits September 1, 2024 21:07
Added email templates and schedules
Added export config for PM Acute Care
…ation available to be displayed in the survey invitation email
Updated email templates according to new specs
Updated according to the latest data table and specs
Discard YVM surveys when an Acute Care survey would be sent for the same day
@sashaandjic sashaandjic added testing... Testing in progress and removed tested Passed manual testing, needs code review labels Sep 13, 2024
Revert no longer needed changes to the import rules
Copy link
Member Author

@sdumitriu sdumitriu left a comment

Choose a reason for hiding this comment

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

@marta- One thing missing: we import the first visit for the day, not the last one.

Copy link
Contributor

@sashaandjic sashaandjic left a comment

Choose a reason for hiding this comment

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

tested

image

image

image

image

image

@sashaandjic sashaandjic added tested Passed manual testing, needs code review and removed testing... Testing in progress labels Sep 13, 2024
@sashaandjic
Copy link
Contributor

AC - second email ( 14 days)
image

Import the newest event in a day for Acute Care visits
Don't include the PE logo in emails as attachment, since it is not used in the email body
Copy link
Contributor

@sashaandjic sashaandjic left a comment

Choose a reason for hiding this comment

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

tested the latest change. only the latest AC visit gets imported.
image
image

@sdumitriu sdumitriu merged commit fe70597 into dev Sep 17, 2024
@sdumitriu sdumitriu deleted the CARDS-2512 branch September 17, 2024 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tested Passed manual testing, needs code review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants