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-2605: Remove patient names from email to: fields #1845

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

acrowthe
Copy link
Contributor

  • Adds a configuration option to include the patient's name in the emails' To: field.
    • This configuration is False by default.
  • Remove unused first and last name properties from prems email configurations

To test:

  • Build with -P docker
  • In compose-cluster, run python3 generate_compose_yaml.py --dev_docker_image --oak_filesystem --cards_project cards4prems --smtps --smtps_test_container --smtps_test_mail_path ~/cardsmail --composum
  • Edit the generated docker-compose.yml file, and at the end of the environment section of the cardsinitial service, add:
- NIGHTLY_NOTIFICATIONS_SCHEDULE=0 * * * * ? *
  • Start with docker-compose up
    Create a new Patient Information form with no names and a Visit information form 7 days ago. Remember which clinic is selected.
  • Wait one minute, then check the contents of the ~/cardsmail folder. Make sure there is no name in the emails To: field (eg. To: [email protected])
  • Edit the Patient Information form to add a first and last name
  • Use the web console to edit the configuration for the used clinic
    • http://localhost:8080/system/console/configMgr
    • The desired configuration will be named similar to io.uhndata.cards.patients.emailnotifications.AppointmentEmailNotificationsFactory~PM-AC-InitialInvitationTask
    • Select Patient name in header and save the configuration
  • Wait one minute and check the ~/cardsmail folder. Make sure that the patient name is now in the most recent To: field (eg. To: John Doe <[email protected]>)

- Adds a configuration option to include the patient's name in the emails' To: field.
  - This configuration is False by default.
- Remove unused first and last name properties from prems email configurations
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