Skip to content
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

We should support dynamic arguments for reverse relations. #148

Open
jnwng opened this issue Nov 28, 2016 · 0 comments
Open

We should support dynamic arguments for reverse relations. #148

jnwng opened this issue Nov 28, 2016 · 0 comments

Comments

@jnwng
Copy link

jnwng commented Nov 28, 2016

in regards to #147, based on some of the changes to the resources themselves, it looks like arguments passed to the reverse relation are hardcoded; however, we're definitely going to want to be able to manipulate those reverse relations like any other resource / finder.

under the hood, we could potentially take a reference to the designated finder and infer the arguments from there.

here's an example of what i'm talking about:

query {
  courses {
    memberships(someParameter: true, someOtherParameter: false) {
      id 
    }
  }
}

where memberships is designated as a reverse relation on the courses resource.

/cc @bryan-coursera

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant