Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

BUG: Patient phone number isn't converting correctly #2730

Closed
1 of 3 tasks
angelathe opened this issue Oct 9, 2024 · 2 comments · Fixed by #2868
Closed
1 of 3 tasks

BUG: Patient phone number isn't converting correctly #2730

angelathe opened this issue Oct 9, 2024 · 2 comments · Fixed by #2868
Assignees
Labels
🐛 bug Something isn't working medium Medium-priority issues Streamline eCR Issues for Streamline eCR team to work on

Comments

@angelathe
Copy link
Collaborator

angelathe commented Oct 9, 2024

Description

In the example eCR below, the patient's phone number is not converting correctly to FHIR format. Investigate the cause of this issue and implement a fix.

Note: The phone number provided is invalid. Please verify that this is not a data quality issue before addressing the FHIR conversion problem.

Steps to reproduce

TN_sample_eCR3: phone number not being converted successfully

Expected Result

If any patient contact information is available, it should be correctly converted to FHIR and displayed in the eCR Viewer.

Actual result

Example from TN_sample_eCR3: Phone number is not displayed
image.png

Priority

Provide an estimate for level of impact the bug has today by placing an X in the applicable bracket

Priority

  • Low priority The bug does not have to be fixed immediately. High and medium priority bugs should be addressed first.
  • Medium priority Typically, medium-priority bugs do not affect customers directly and therefore can be fixed in the normal course of testing and development.
  • High priority High priority bugs must be addressed immediately. They often affect the app’s functionality and impact customers and their user experience. They must take priority.

Notes

FHIR conversion may be required for this ticket.

@angelathe angelathe added FHIR conversion required This ticket will need FHIR conversion work Streamline eCR Issues for Streamline eCR team to work on 🐛 bug Something isn't working labels Oct 9, 2024
@emmastephenson emmastephenson added the medium Medium-priority issues label Oct 24, 2024
@mcmcgrath13 mcmcgrath13 self-assigned this Nov 5, 2024
@mcmcgrath13
Copy link
Collaborator

I don't have access to the tennessee data, but based on the screenshot added this telecom node to another patient's eCR and it does convert as I'd expect

{
      "system": "phone",
      "value": "+1-111-111-111",
      "use": "mobile"
}

I believe the telephone formatting service is rejecting this as invalid. @ashton-skylight @emmastephenson in this case do we want to do something else other than null it out? maybe pass it through, but with some sort of invalid caveat?

@mcmcgrath13 mcmcgrath13 removed the FHIR conversion required This ticket will need FHIR conversion work label Nov 5, 2024
@ashton-skylight
Copy link
Collaborator

Perhaps something like this?
Screenshot 2024-11-05 at 11 45 44 AM

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
🐛 bug Something isn't working medium Medium-priority issues Streamline eCR Issues for Streamline eCR team to work on
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants