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

Review Data Filter Conditions #1760

Closed
JohnPTobe opened this issue Aug 30, 2019 · 0 comments
Closed

Review Data Filter Conditions #1760

JohnPTobe opened this issue Aug 30, 2019 · 0 comments
Assignees
Milestone

Comments

@JohnPTobe
Copy link

Pain Point? Please describe.
Review the data filter conditions and see if they make sense. Also update documentation to make them clearer. For example, when using contains you are checking if the value being inspected contains one of the values in the list from the condition. Currently the 'in' condition checks if the output is in the list of values or is a subset of one of the values in the list. We probably want to change this so it just checks the former.

Desired Solution
A clear and concise description of what you want to happen.

Alternative / Workaround
A clear and concise description of any alternative solutions or features you've considered.

Additional Context
Add any other context or screenshots about the feature request here.

@JohnPTobe JohnPTobe self-assigned this Aug 30, 2019
@emimaesmith emimaesmith added this to the Sprint 6 milestone Sep 3, 2019
gisjedi pushed a commit that referenced this issue Sep 11, 2019
* 📚 Updating data filter description to clarify filter types, conditions, etc

* 🔨 Changing 'in' conditions to only check overall list and not each item in the list; changing type variables to filter_type so we don't conflict with the keyword; adding test for 'array' type

* ✅ Fixing tests

* 🔨 Fixing transient batch testing bug and 504 error with empty results for creating dataset members

* 👌 Clarifying condition input params

* 👌 Adding examples for each condition

Closes #1760
Fixes #1772; Fixes #1773
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

No branches or pull requests

2 participants