Benedikt Franke
Benedikt Franke
See https://github.com/webonyx/graphql-php/issues/552 and https://github.com/graphql/graphql-js/issues/2020 for discussion.
Can you provide a pull request with a failing test case? The issue might be with the lexer, but I think a test for this would fit better with https://github.com/webonyx/graphql-php/blob/e8098fae49e2a007f389aa838796c6c154138744/tests/Language/SchemaParserTest.php#L166.
What exactly would you expect? Should the message change in some way?
`GraphQL\Type\Definition\ResolveInfo` might be a suitable inclusion, it holds a lot of information about the execution state at the time the error was thrown. It would have to be optional, since...
We have solved debugging issues like this through the use of [Sentry](https://sentry.io). I recommend generally improving the logging and traceability of what happens in your application.
@mremi do you mind taking a look at this PR?
The `@globalId` variant should work, I am looking into this.
Please try and add a failing test case to https://github.com/nuwave/lighthouse/blob/master/tests/Integration/Schema/Directives/BelongsToDirectiveTest.php
I agree with the general sentiment and your proposal. The name `failOnNull` is a bit ambiguous. How about we just forego finding another term to describe the concept and just...
Please add a PR with a failing test case in https://github.com/nuwave/lighthouse/blob/master/tests/Integration/WhereConditions/WhereHasConditionsDirectiveTest.php in order to investigate this further. By the way (not invalidating your issue), but have you considered using the...