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

Non-null fields #57

Closed
melinath opened this issue Jan 23, 2018 · 2 comments
Closed

Non-null fields #57

melinath opened this issue Jan 23, 2018 · 2 comments
Assignees
Projects

Comments

@melinath
Copy link
Member

melinath commented Jan 23, 2018

GraphQL has a concept of Non-Null Types. This ticket covers fields, and not arguments.

Requirements

The parts of the Non-Null type documentation related to fields are covered by this ticket.

Non-Null needs to be represented as a "wrapping type" in relation to the API (for example when coercing a field value or introspecting the API), but would probably be better represented as null=False on field definitions to match Django conventions.

Exclusions

As validation does not yet exist in django-graph-api, validation of non-null arguments should not happen as part of this ticket.

@melinath melinath created this issue from a note in v0.3.0 (TODO) Jan 23, 2018
@fugu13
Copy link
Contributor

fugu13 commented Jan 24, 2018

I'm going to take an attempt at this one here at PyCascades

@ariannedee ariannedee moved this from TODO to In Progress in v0.3.0 Jan 24, 2018
@fugu13
Copy link
Contributor

fugu13 commented Jan 24, 2018

Pausing on this, if anyone wants to take it on. This requires either major restructuring or meta class manipulation.

@ariannedee ariannedee moved this from In Progress to TODO in v0.3.0 Jan 25, 2018
@ariannedee ariannedee moved this from TODO to In Progress in v0.3.0 Feb 21, 2018
@ariannedee ariannedee self-assigned this Feb 21, 2018
@ariannedee ariannedee mentioned this issue Feb 26, 2018
5 tasks
@ariannedee ariannedee moved this from In Progress to Awaiting Review in v0.3.0 Feb 26, 2018
v0.3.0 automation moved this from Awaiting Review to Done Mar 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
v0.3.0
  
Done
Development

No branches or pull requests

3 participants