GraphQL: Nested Collections #6038
Conversation
jotwea
force-pushed
the
graphql-nested
branch
from
506cb81 to
e5dc5b0
Compare
jotwea
force-pushed
the
graphql-nested
branch
2 times, most recently
from
4a56dce to
5adc973
Compare
Updated the PR. Still some jobs fail because of deprecations, but this is not related to the changes in this PR. Maybe it can be merged anyway |
|
In my opinion, since it's a collection it should follow the Relay specification and have |
jotwea
force-pushed
the
graphql-nested
branch
from
5adc973 to
9539827
Compare
Works now. Can be controlled via |
|
Nice, maybe could you add one case with the pagination enabled too? |
jotwea
force-pushed
the
graphql-nested
branch
from
9539827 to
c1eb026
Compare
Updated the tests. We now have nested collection with and without pagination enabled. |
jotwea
force-pushed
the
graphql-nested
branch
2 times, most recently
from
abab343 to
899ac0a
Compare
jotwea
force-pushed
the
graphql-nested
branch
from
899ac0a to
c3a35cf
Compare
jotwea
force-pushed
the
graphql-nested
branch
from
c3a35cf to
da5a126
Compare
jotwea
force-pushed
the
graphql-nested
branch
from
da5a126 to
8bdd7bc
Compare
|
I'll solve phpstan in another patch thanks @jotwea @alanpoulain ! It'd be nice to document this behavior now. |
* feat(graphql): support nested collections * null safe operator --------- Co-authored-by: josef.wagner <[email protected]> Co-authored-by: Antoine Bluchet <[email protected]>
We sometimes use JSON arrays to persist a collection of Value Objects which are not an entity. So this collection should be treated like a property of the parent entity. On the GraphQL queries fetching this collection was not an easy task. We had to create some special resolvers which try to fetch the ID of the parent entity out of the
$context, load it and determine the collection from there.So I decided to spend this PR that utilizes the
nestedproperty of theApiPropertyand uses the collection already loaded with the entity when no own resolver is given. I added a Behat test that covers that new scenario.