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.

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

[FhirPath] more customization in fhirpathcompilercache extension methods #2765

Open
Kasdejong opened this issue Apr 9, 2024 · 0 comments
Open

[FhirPath] more customization in fhirpathcompilercache extension methods #2765

Kasdejong opened this issue Apr 9, 2024 · 0 comments
Labels
breaking change This issue/commit causes a breaking change, and requires a major version upgrade enhancement to be triaged

Comments

@Kasdejong
Copy link
Contributor

Kasdejong commented Apr 9, 2024 •
edited

The fhirpath predicate/istrue/select/etc extension methods take an evaluationcontext as a parameter, but:

  • It is not easy to quickly set environment variables like this
  • they do not set %resource, %rootresource and %context

We should revise this, since currently this is only possible in our lower level API. This might be breaking.
@Kasdejong Kasdejong added enhancement breaking change This issue/commit causes a breaking change, and requires a major version upgrade labels Apr 9, 2024
@Kasdejong Kasdejong changed the title [FhirPath] [FhirPath] more customization in fhirpathcache extension methods Apr 9, 2024
@Kasdejong Kasdejong changed the title [FhirPath] more customization in fhirpathcache extension methods [FhirPath] more customization in fhirpathcompilercache extension methods Apr 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
breaking change This issue/commit causes a breaking change, and requires a major version upgrade enhancement to be triaged
Projects
Firely .NET SDK
Status: Todo
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mmsmits @Kasdejong