overblog / GraphQLBundle

This bundle provides tools to build a complete GraphQL API server in your Symfony App.
MIT License
783 stars 221 forks source link

Fix profiler & update phpcs #1155

Closed Vincz closed 7 months ago

Vincz commented 8 months ago
Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Documented? no
Fixed tickets #...
License MIT

This fix the problem with the profiler if the schema doesn't have a subscription. It's related to https://github.com/overblog/GraphQLBundle/issues/1112 and the fact that Webonyx expects null from not found type with the TypeResolver while we raise an exception.

Vincz commented 8 months ago

@mcg-web Hey mate! It would be great if you could check this (just a bunch of fixes and CS). Also, we should release a new version as the latests commits are fixing a bunch of problematics issues (and the SF 7 compat).