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 "array" type #36

Open
justinwinslow opened this issue Sep 1, 2015 · 4 comments
Open

Add "array" type #36

justinwinslow opened this issue Sep 1, 2015 · 4 comments

Comments

@justinwinslow
Copy link
Contributor

we ran into an issue where a user was presented with a description that made it unclear the field was an array. The confusion was compounded by the fact they were shown all of the available operators.

@justinwinslow
Copy link
Contributor Author

Ran into another issue where the Suppression List Found In field is an array about the UI didn't pare back the operators and there was no indication the value would be an array and the user selected is equal to resulting in none of the rules with the Found In field validating as expected.

@alexkwolfe
Copy link
Contributor

Isn't the SL response variable an array type?

@justinwinslow
Copy link
Contributor Author

Yes, but there's nothing to inform the UI what operators are applicable.

@justinwinslow
Copy link
Contributor Author

We could add something manually if you don't think an array type belongs in the types api.

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

No branches or pull requests

2 participants