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

%resource and %rootResource in default context #2841

Merged
merged 11 commits into from
Sep 4, 2024

Conversation

Kasdejong
Copy link
Contributor

Description

Added %resource and %rootresource to default evaluation context on FP extensions

Related issues

Closes #2177

@Kasdejong Kasdejong marked this pull request as ready for review August 21, 2024 11:49
@mmsmits mmsmits requested a review from ewoutkramer August 23, 2024 07:10
@ewoutkramer
Copy link
Member

fixed nullability. How did this even work before?

Good question.

@Kasdejong Kasdejong changed the title %resource and %resource in default context %resource and %rootResource in default context Aug 27, 2024
@Kasdejong Kasdejong requested a review from ewoutkramer August 29, 2024 13:26
Copy link
Member

@ewoutkramer ewoutkramer left a comment

Choose a reason for hiding this comment

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

(see my comment)

@Kasdejong Kasdejong self-assigned this Sep 4, 2024
@ewoutkramer ewoutkramer merged commit 4c38d9a into develop Sep 4, 2024
16 checks passed
@ewoutkramer ewoutkramer deleted the feature/FP-extensions-default-context branch September 4, 2024 14:31
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.

The default context for FhirPath evaluation is leaving out %resource
2 participants