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

ORM NK1 Segment Support #1547

Closed
26 tasks
GilmoreA6 opened this issue Nov 4, 2024 · 7 comments
Closed
26 tasks

ORM NK1 Segment Support #1547

GilmoreA6 opened this issue Nov 4, 2024 · 7 comments

Comments

@GilmoreA6
Copy link
Contributor

GilmoreA6 commented Nov 4, 2024

Story

As a LA sender I need to be able to send nonstandard ORM_001 messages that contain NK1 segments

Context

Add ReportStream functionality that allows NK1 segment sent in ORM messages to convert to RelatedPerson FHIR resources

Acceptance Criteria

  • ORM_001 message with NK1 segment is sent to ReportStream and translates to FHIR message with RelatedPerson resource(s).
  • Sample HL7 OML NK1 segments sent out of ReportStream exactly match the incoming ORM NK1 content.

Tasks

Research

  • How does ReportStream want to accomplish this?
  • Should ReportStream have a sender setting that allows specifying which HL7 message structure to use?

Engineering

  • New Sender setting added to RS to allow specifying which HL7 message structure to use
  • New ORM_001_Oschner HL7 message structure added to ReportStream

Definition of Done

  • Documentation tasks completed
    • Documentation and diagrams created or updated
      • ADRs (/adr folder)
      • Main README.md
      • Other READMEs in the repo
      • If applicable, update the ReportStream Setup section in README.md
    • Threat model updated
    • API documentation updated
  • Code quality tasks completed
    • Code refactored for clarity and no design/technical debt
    • Adhere to separation of concerns; code is not tightly coupled, especially to 3rd party dependencies
  • Testing tasks completed
    • Load tests passed
    • Additional e2e tests created
    • Additional RS e2e assertions created in the rs-e2e project for any new transformations. Includes improvements to the assertion code required to make the new assertions
  • Build & Deploy tasks completed
    • Build process updated
    • API(s) are versioned
    • Feature toggles created and/or deleted. Document the feature toggle
    • Source code is merged to the main branch

Note: Please remove any DoD items that are not applicable

Research Questions

  • Optional: Any initial questions for research

Decisions

  • Optional: Any decisions we've made while working on this story

Notes

  • Optional: Any reference material or thoughts we may need for later reference
@brick-green
Copy link

Blocker on research question. Split out to different ticket:

Can LA-Oschner not send an OML_021 message from their EPIC system?

@JohnNKing JohnNKing changed the title Add ReportStream functionality that allows NK1 segment sent in ORM messages to convert to RelatedPerson FHIR resources ORM NK1 Segment Support Nov 13, 2024
@JohnNKing
Copy link
Contributor

This one is likely to be pulled into the next sprint, but the team was hesitant to commit to this being completed in the sprint's timeframe, so I'm leaving it in the product backlog for now.

@brick-green
Copy link

I think we should confirm with Epic about the ability to change MSH.9. If they can change it to OML then we should just be treating the messages as OML which would already include mapping for NK1.

@basiliskus
Copy link
Contributor

We're currently waiting to hear if Ochsner can change MSH-9 message type to be OML. If they can, then the mapping already works in RS

@brick-green
Copy link

Ochsner made the change and sent a new message. See comment here

@basiliskus
Copy link
Contributor

We could close this story if not needed now that we know we'll get NK1 in an OML message instead of ORM

@GilmoreA6
Copy link
Contributor Author

Closing as this ticket is no longer needed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants