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

Minor email fixes #130

Merged
merged 3 commits into from
Oct 3, 2023
Merged

Minor email fixes #130

merged 3 commits into from
Oct 3, 2023

Conversation

Amanda2900
Copy link
Collaborator

@Amanda2900 Amanda2900 commented Oct 3, 2023

Description

This PR corrects some errors in the Request email that is sent. These errors include:

  • the spelling of 'Appointment' in Appointment Type
  • the way the Requested Date is displayed
  • Emergency is now 'Yes' or 'No' instead of true or false

Other additions:

  • capitalizing the Requested Time and Appointment Type responses
  • adding 'None' if Description is left blank

Related Issue

closes #110

Acceptance Criteria

The requested date is in a format that is more user readable
The "is this an emergency" status is either Yes or No (not true/false)
Corrected spelling for Appointment

Type of Changes

Type
🐛 Bug fix
✨ New feature
🔨 Refactoring
💯 Add tests
🔗 Update dependencies
📜 Docs

Updates

Before

Screen Shot 2023-10-03 at 11 38 32 AM

After

Screen Shot 2023-10-03 at 11 39 28 AM

Testing Steps / QA Criteria

Submit an appointment request using your email as the patient email. The changes mentioned above should be reflected in that email.

Copy link
Contributor

@mentalcaries mentalcaries left a comment

Choose a reason for hiding this comment

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

Smooth! Thank you

@mentalcaries mentalcaries merged commit c283bb9 into main Oct 3, 2023
1 check passed
@mentalcaries mentalcaries deleted the minor-email-fixes branch October 3, 2023 20:19
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.

Requested date in confirmation email needs to be adjusted
2 participants