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

Add support for Relay-based graphene schemas #1

Open
sjdemartini opened this issue May 10, 2022 · 0 comments
Open

Add support for Relay-based graphene schemas #1

sjdemartini opened this issue May 10, 2022 · 0 comments
Labels
bug Something isn't working enhancement New feature or request help wanted Extra attention is needed

Comments

@sjdemartini
Copy link
Owner

Right now, this library only works properly for Graphene/GraphQL schemas that do not use Relay nodes (see https://docs.graphene-python.org/projects/django/en/latest/tutorial-relay/), but should work whether the schema/queries use Relay nodes or not.

Although it seemingly will not expose any data that isn't permissible (as we'd hope), it seems Relay-based schemas do not behave properly when filtering non-permitted items in lists, since the list ends up containing items like {'node': None} rather than omitting the item entirely.

See the WIP branch support-relay (diff here) which includes unit-tests, a few of which fail per the above. That should help for investigating a solution to add proper support for Relay.

@sjdemartini sjdemartini added bug Something isn't working enhancement New feature or request help wanted Extra attention is needed labels May 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant