-
Notifications
You must be signed in to change notification settings - Fork 249
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
Refactor definition-based extraction for multiple resource and create extraction context #670
Labels
effort:xsmall
Extra small effort - 1 day
P1
High priority issue
type:enhancement
New feature or request
Projects
Comments
type of the extraction context could probably be: |
6 tasks
@Tarun-Bhardwaj This should be completed as part of the next release of the DefinitionBased Extraction. We should plan to provide documentation and use cases with how to implement this within Questionnaire |
8 tasks
8 tasks
8 tasks
This is done in #1014 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
effort:xsmall
Extra small effort - 1 day
P1
High priority issue
type:enhancement
New feature or request
Is your feature request related to a problem? Please describe.
The extraction code for multiple resources needs to be refactored
Describe the solution you'd like
Refactor the definition based extraction functions so that they are not extension functions of Base any more. Instead, a extraction context should be passed down during extraction."
We need to create something like
data class ExtractionContext
and pass it down during the definition-based extraction process.It should probably include the current target resource, as well as anything else that has been created to be added to the final bundle.
Describe alternatives you've considered
Current approach does work, but it's difficult to read and understand and lacks the level of abstraction we need for this code.
Additional context
NA
Would you like to work on the issue?
NA
The text was updated successfully, but these errors were encountered: