search

hasura / ndc-spec

NDC Specification and Reference Implementation
http://hasura.github.io/ndc-spec
20 stars 5 forks source link

RFC: explain for mutations #85

Closed soupi closed 9 months ago

soupi commented 9 months ago

Rendered

This would require a change to ndc-sdk as well.

soupi commented 9 months ago

Happy to change the PR, but I think that if we want two separate endpoint, renaming /explain to /explain/query would be preferable, but I thought a single endpoint with a new request was nicer. Since we are at a point where there are several breaking changes that need to be addressed, it might be a good time to change this as well.

paf31 commented 9 months ago

Yes, let's change both now while we have the chance to make more breaking changes.

I wonder if query/explain is preferable or explain/query. Either way, the capabilities should mirror the URLs, I think.

soupi commented 9 months ago

@paf31 I asked a few people and it seems like the majority prefer /query/explain. Updated the PR.