-
Notifications
You must be signed in to change notification settings - Fork 2.8k
add a json path argument to query values inside json columns - hgiasac (close #1598) #1744
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
|
Review app for commit d9ea58b deployed to Heroku: https://hge-ci-pull-1744.herokuapp.com |
|
@dsandip I'm not sure if the docs are clear. It's also within the first docs section of simple queries, which makes it more confusing. As a new user looking for how to query stuff, this would be very surprising, it seems like this ought to be more "advanced postgres user" documentation? Perhaps we should have a separate page for this, or maybe we should include it in the reference documentation? Maybe we should have a page for the different GraphQL types we generate. Let's also update the JSON blogpost? https://blog.hasura.io/postgres-json-and-jsonb-type-support-on-graphql-41f586e47536/ |
|
Review app for commit 4ce2c2a deployed to Heroku: https://hge-ci-pull-1744.herokuapp.com |
|
@dsandip The commits in this PR are out of sync with master - i.e. some console changes are included. Need to figure out a way to fix that. |
|
Also, why is this a new PR? What happened to the old one? |
|
closing in favour of #1661 |
|
Review app https://hge-ci-pull-1744.herokuapp.com is deleted |
Description
Affected components
Related Issues
Solution and Design
Steps to test and verify
Limitations, known bugs & workarounds