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

Document danielvladco.protobuf.graphql.required since proto3 fields all "optional" but GraphQL has "required" concept #40

Open
perrydunn opened this issue May 26, 2022 · 0 comments

Comments

@perrydunn
Copy link

In proto3 all fields are optional. In GraphQL fields can be required. There is a protobuf field annotation already available and being used within protoc-gen-gql, danielvladco.protobuf.graphql.required, but it does not appear in any of the examples and it is not otherwise documented.

I think it's worth adding the annotation to an example or otherwise documenting its existence.

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