Skip to content
This repository was archived by the owner on May 10, 2018. It is now read-only.

Implement exists for Graphcool bindings #11

Closed
kbrandwijk opened this issue Dec 29, 2017 · 4 comments
Closed

Implement exists for Graphcool bindings #11

kbrandwijk opened this issue Dec 29, 2017 · 4 comments

Comments

@kbrandwijk
Copy link
Contributor

No description provided.

@schickling
Copy link
Collaborator

Would be great to have this soon! 🎉

@kbrandwijk
Copy link
Contributor Author

Do we have final consensus on the API?

  1. context.db.Post.exists(...)
  2. context.db.posts.exists(...)
  3. context.exists.Post(...)
  4. context.exists.posts(...)

(1) is my personal favorite (by Type name, not query root field).

@schickling
Copy link
Collaborator

For now it will be (4) until we've found a new Binding API design (see prisma-labs/prisma-binding#11)

kbrandwijk added a commit that referenced this issue Jan 2, 2018
kbrandwijk added a commit that referenced this issue Jan 2, 2018
WIP - Needs `graphcool-binding` PR

Closes #11
kbrandwijk added a commit that referenced this issue Jan 2, 2018
@kbrandwijk
Copy link
Contributor Author

After some consideration, the current PR is for option (3), with a corresponding PR on graphcool-binding to implement identical behavior for dynamic bindings. See prisma-labs/prisma-binding#34.

kbrandwijk added a commit that referenced this issue Jan 2, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants