-
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
Boolean expressions for httpQL queries #4
Comments
I've noodled on this in the past and the biggest issue I have is there's no guaranteed order on query params. While there's no semantic support for including a request body in a GET there is some industry support for it (ElasticSearch for example) This feels a little safer and also, certainly more human readable. Thoughts @boulter? If we like this approach, I can try putting together a sample JSON spec that we can play with. The plumbing shouldn't be too difficult. |
Yeah, it may not be possible to do using an unencoded query string. We could either encode the entire query as one parameter or allow some expressions to be more complex. For example:
where expr1 decoded is |
re: get with body, https://git.hubteam.com/HubSpot/HubSpotConnect/issues/383 |
Currently, all httpQL conditions are ANDed together. It would be super useful to expand this the rest of the boolean expression family - OR and NOT.
We'll first need to figure out how to group and format these new expressions as a query string.
@hs-lsong
The text was updated successfully, but these errors were encountered: