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

Add existing filter containers to Extraction via context menu should offer templates/AggregateConfiguration ones [Zsolt] #1336

Open
2 tasks
tznind opened this issue Aug 5, 2022 · 2 comments

Comments

@tznind
Copy link
Contributor

tznind commented Aug 5, 2022

An AggregateConfiguration is a query that builds a bit of a cohort. These can be marked as 'templates' making them reusable within Cohort Builder.

This ticket looks at how awesome that behaviour is in CohortBuilder and seeks to have the same functionality available in the Extraction side of things.

This is primarily about collections of filters. Not collections of datasets.

  • The ability to mark a SelectedDataSet in an ExtractionConfiguration as a 'template'
  • Adding from context menus etc should behave similar to the Cohort Builder experience of templates

Is your feature request related to a problem? Please describe.

When user does this (for extraction):
image

Message

There are no extractions or cohort builder configurations of this dataset that use filters

Describe the solution you'd like
Ask user if they want to pick from template (yes/no - like in cohort builder) . Allow search and pick to inmport the filter tree. Currently it shows the above error message and only lets you pick from existing extractions.

@tznind
Copy link
Contributor Author

tznind commented Aug 5, 2022

Relates to #1327 this is an incremental improvement to make it work with context menus as well as drag and drop.

@tznind
Copy link
Contributor Author

tznind commented Oct 31, 2022

You can already drag a 'template' (or non template) AggregateConfiguration onto a SelectedDataset to import the filter container tree so this is a low priority ticket and mostly about usability.

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

No branches or pull requests

1 participant