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

Jump to bottom

Execute phenotype FHIR Bundle (#46) #52

Merged
merged 1 commit into from
Mar 8, 2021
Merged

Execute phenotype FHIR Bundle (#46) #52

merged 1 commit into from
Mar 8, 2021

Conversation

lrasmus
Copy link
Member

@lrasmus lrasmus commented Mar 5, 2021

Initial support for JSON file-based Bundle only. Will expand to URL-based later.

@lrasmus
Execute phenotype FHIR Bundle (#46)
c6e83b9 
Initial support for JSON file-based Bundle only.  Will expand to URL-based later.
@lrasmus lrasmus requested a review from psbrandt March 5, 2021 04:56
psbrandt
psbrandt approved these changes Mar 5, 2021
View reviewed changes
Copy link
Member

@psbrandt psbrandt left a comment

Choose a reason for hiding this comment

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

Love it!

@psbrandt
Copy link
Member

psbrandt commented Mar 6, 2021

Looks like we're having the issue where it's trying to load a dependency over http instead of https :/

@lrasmus
Copy link
Member Author

lrasmus commented Mar 8, 2021

Looks like we're having the issue where it's trying to load a dependency over http instead of https :/

Tracking in #53 now so it stays in my face

@lrasmus lrasmus merged commit f3f1c70 into master Mar 8, 2021
@lrasmus lrasmus deleted the execute-bundles branch March 8, 2021 15:55
@lrasmus lrasmus mentioned this pull request Mar 11, 2021
Closed
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@psbrandt psbrandt psbrandt approved these changes

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
@lrasmus @psbrandt