Skip to content

RELEASE elation contribution #4090

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 43 commits into from
Jun 25, 2025
Merged

RELEASE elation contribution #4090

merged 43 commits into from
Jun 25, 2025

Conversation

thomasyopes
Copy link
Contributor

@thomasyopes thomasyopes commented Jun 25, 2025 •
edited by coderabbitai bot
Loading

Issues:

Description

Testing

Check each PR.

Release Plan

  • ⚠️ Points to master
  • Merge this

Summary by CodeRabbit

  • New Features

    • Introduced support for the Elation EHR source, enabling retrieval and conversion of CCDA documents and FHIR resource bundles.
    • Added new FHIR conversion workflows, including direct HTTP and AWS Lambda-based conversion handlers, with automatic environment-based selection.
    • Implemented utilities for partitioning and saving FHIR conversion payloads and results.
    • Enhanced infrastructure to integrate and grant permissions for FHIR converter Lambda and storage resources.

  • Improvements

    • Expanded resource diff bundle creation to support Elation EHR.
    • Improved error messages and context for bundle processing across EHR sources.
    • Added new utilities for generating file paths and S3 keys for conversion artifacts.

  • Bug Fixes

    • Removed a deprecated internal API endpoint for setting resource diff job entry status.

  • Documentation

    • Added schema validation and type definitions for Elation CCDA documents.

Thomas Yopes added 30 commits June 23, 2025 10:49
feat(ehr): elation contribution
849f72b 
Ref: ENG-139

Ref: #1040
Signed-off-by: Thomas Yopes <thomasyopes@Thomass-MBP.attlocal.net>
feat(ehr): adding elation hook
5ddbd0c 
Ref: ENG-139

Ref: #1040
Signed-off-by: Thomas Yopes <thomasyopes@Thomass-MBP.attlocal.net>
feat(ehr): update conversion
3f9b30d 
Ref: ENG-139

Ref: #1040
Signed-off-by: Thomas Yopes <thomasyopes@Thomass-MBP.attlocal.net>
feat(ehr): wrong env var
b674f1e 
Ref: ENG-139

Ref: #1040
Signed-off-by: Thomas Yopes <thomasyopes@Thomass-MBP.attlocal.net>
feat(ehr): wrong env ar name
44ffe3c 
Ref: ENG-139

Ref: #1040
Signed-off-by: Thomas Yopes <thomasyopes@Thomass-MBP.attlocal.net>
feat(ehr): conversion stuff
59ce680 
Ref: ENG-139

Ref: #1040
Signed-off-by: Thomas Yopes <thomasyopes@Thomass-MBP.attlocal.net>
feat(ehr): final fixes
2ca5152 
Ref: ENG-139

Ref: #1040
Signed-off-by: Thomas Yopes <thomasyopes@Thomass-MBP.attlocal.net>
feat(ehr): implementation of factory
85a1552 
Ref: ENG-139

Ref: #1040
Signed-off-by: Thomas Yopes <thomasyopes@Thomass-MBP.attlocal.net>
feat(ehr): small fixes
46a9c96 
Ref: ENG-139

Ref: #1040
Signed-off-by: Thomas Yopes <thomasyopes@Thomass-MBP.attlocal.net>
Merge branch 'eng-72-athena-data-contribution' of https://github.com/…
cd90530 
…metriport/metriport into eng-139-elation-contribution

Signed-off-by: Thomas Yopes <thomasyopes@Thomass-MBP.attlocal.net>
feat(ehr): adding intermediate steps + s3 key piece
77e3df7 
Ref: ENG-139

Ref: #1040
Signed-off-by: Thomas Yopes <thomasyopes@Thomass-MBP.attlocal.net>
feat(ehr): adding dedup to contribution
f1dbf35 
Ref: ENG-139

Ref: #1040
Signed-off-by: Thomas Yopes <thomasyopes@Thomass-MBP.attlocal.net>
feat(ehr): fix
7b7f511 
Ref: ENG-139

Ref: #1040
Signed-off-by: Thomas Yopes <thomasyopes@Thomass-MBP.attlocal.net>
feat(ehr): fixes
e5de0c3 
Ref: ENG-139

Ref: #1040
Signed-off-by: Thomas Yopes <thomasyopes@Thomass-MBP.attlocal.net>
Merge branch 'eng-72-athena-data-contribution' of https://github.com/…
4b0af30 
…metriport/metriport into eng-139-elation-contribution

Signed-off-by: Thomas Yopes <thomasyopes@Thomass-MBP.attlocal.net>
feat(ehr): fix shadowing
cf0ea6b 
Ref: ENG-139

Ref: #1040
Signed-off-by: Thomas Yopes <thomasyopes@Thomass-MBP.attlocal.net>
Merge branch 'eng-72-athena-data-contribution' of https://github.com/…
85b9e1a 
…metriport/metriport into eng-139-elation-contribution

Signed-off-by: Thomas Yopes <thomasyopes@Thomass-MBP.attlocal.net>
feat(ehr): remove unused route
4cb1efc 
Ref: ENG-139

Ref: #1040
Signed-off-by: Thomas Yopes <thomasyopes@Thomass-MBP.attlocal.net>
Merge branch 'eng-72-athena-data-contribution' of https://github.com/…
8adfe85 
…metriport/metriport into eng-139-elation-contribution

Signed-off-by: Thomas Yopes <thomasyopes@Thomass-MBP.attlocal.net>
feat(ehr): no arrow function
2a9defe 
Ref: ENG-139

Ref: #1040
Signed-off-by: Thomas Yopes <thomasyopes@Thomass-MBP.attlocal.net>
feat(ehr): fix elation thing
ecc8514 
Ref: ENG-139

Ref: #1040
Signed-off-by: Thomas Yopes <thomasyopes@Thomass-MBP.attlocal.net>
feat(ehr): fixes
6ed01ef 
Ref: ENG-139

Ref: #1040
Signed-off-by: Thomas Yopes <thomasyopes@Thomass-MBP.attlocal.net>
feat(ehr): align config with upstream
0b6f7e6 
Ref: ENG-139

Ref: #1040
Signed-off-by: Thomas Yopes <thomasyopes@Thomass-MBP.attlocal.net>
feat(ehr): remove fhirresource from lambda return
58288ce 
Ref: ENG-139

Ref: #1040
Signed-off-by: Thomas Yopes <thomasyopes@Thomass-MBP.attlocal.net>
Merge branch '1040-fhir-converter-in-lambda' of https://github.com/me…
974ae48 
…triport/metriport into eng-139-elation-contribution

Signed-off-by: Thomas Yopes <thomasyopes@Thomass-MBP.attlocal.net>
feat(ehr): wiring converter lambda into refresh lambda
02f7a8f 
Ref: ENG-139

Ref: #1040
Signed-off-by: Thomas Yopes <thomasyopes@Thomass-MBP.attlocal.net>
Merge branch '1040-fhir-converter-in-lambda' of https://github.com/me…
eae442a 
…triport/metriport into eng-139-elation-contribution

Signed-off-by: Thomas Yopes <thomasyopes@Thomass-MBP.attlocal.net>
feat(ehr): implement class
86db2cc 
Ref: ENG-139

Ref: #1040
Signed-off-by: Thomas Yopes <thomasyopes@Thomass-MBP.attlocal.net>
feat(ehr): remove config from factory
dc108f9 
Ref: ENG-139

Ref: #1040
Signed-off-by: Thomas Yopes <thomasyopes@Thomass-MBP.attlocal.net>
feat(ehr): better naming
018811e 
Ref: ENG-139

Ref: #1040
Signed-off-by: Thomas Yopes <thomasyopes@Thomass-MBP.attlocal.net>
@coderabbitai coderabbitai bot mentioned this pull request Jul 3, 2025
Merged
5 tasks
This was referenced Jul 18, 2025
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
This was referenced Jul 25, 2025
Merged
Merged
Closed
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
@coderabbitai coderabbitai bot mentioned this pull request Aug 20, 2025
Merged
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@coderabbitai coderabbitai[bot] coderabbitai[bot] left review comments

@Orta21 Orta21 Orta21 approved these changes

@Goncharo Goncharo Awaiting requested review from Goncharo Goncharo is a code owner

@leite08 leite08 Awaiting requested review from leite08 leite08 is a code owner

@RamilGaripov RamilGaripov Awaiting requested review from RamilGaripov RamilGaripov is a code owner

@lucasdellabella lucasdellabella Awaiting requested review from lucasdellabella lucasdellabella is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@thomasyopes @Orta21