-
Notifications
You must be signed in to change notification settings - Fork 9
Update search conditions properties to new preferred values #237
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
Conversation
| - `search_attribute` _(required)_ - the attribute you wish to search, can be any attribute | ||
| - `search_type` _(required)_ - the type of search to perform - `equals`, `contains`, `starts_with`, `ends_with`, `greater_than`, `greater_than_equal`, `less_than`, `less_than_equal`, `between` | ||
| - `search_value` _(required)_ - case-sensitive value you wish to search. If the `search_type` is `between` then use an array of two values to search between | ||
| - `comparator` _(required)_ - the type of search to perform - `equals`, `contains`, `starts_with`, `ends_with`, `greater_than`, `greater_than_equal`, `less_than`, `less_than_equal`, `between` | ||
| - `search_value` _(required)_ - case-sensitive value you wish to search. If the `comparator` is `between` then use an array of two values to search between |
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.
Should these be updated too?
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.
Yes, thanks!
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.
Are these fields no longer supported or will they still work on 4.6? Same question for 4.7 and beyond? I assume we are remaining backwards compatible. It may be valuable to include a note somewhere what these fields were previously called in case someone goes searching for them referencing existing code.
search_attribute, search_type, and search_value have been deprecated in favor of attribute, comparator, and value
53b5bae to
2055ca6
Compare
|
@Ethan-Arrowood All should be addressed |
search_attribute,search_type, andsearch_valuehave been deprecated in favor ofattribute,comparator, andvalue(which have the added bonus of being consistent with the resource API)