-
Notifications
You must be signed in to change notification settings - Fork 14
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
Handle survey_response cells when searching in view #1199
Conversation
You are not adding a |
That's not correct. The The old logic of searching just understood text cells, but this is a cell which contains an array. The new code joins the array entries within a single cell into a single string of text, and searches that. |
Ah okay now it makes sense! It was the array that I was confused about. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great! Works and code looks good!
Cool! I think what you've found (adding a column without finishing all the options) is a bug though. I will log it separately! |
Logged the bug you found as #1200. |
This PR handles cells in
survey_response
columns properly when searching in a view. These cells include an array of objects, each object representing a response to the given survey question. The logic merges the array into a single string and searches it the same way that regular search in text-based cells work.Fixes #1142