graphql.md: add "MediaObject Normalization" chapter#2109
graphql.md: add "MediaObject Normalization" chapter#2109vinceAmstoutz merged 3 commits intoapi-platform:4.1from timble-one:graphql-media-object-normalization
Conversation
vinceAmstoutz
left a comment
There was a problem hiding this comment.
@71m024 Tanks for your contribution!
Can you please add more explanation on the “Why”?
Also, can you add an injection example for Laravel? As you can see here with an example of injection, we now document both Symfony and Laravel (since 4.0).
|
@timble-one Do you plan to keep going? |
|
@vinceAmstoutz I made it i little bit clearer, that the injected normalizer must be replaced with the one from the graphql namespace. Otherwise it won't work if you just do it like it is done here: https://api-platform.com/docs/symfony/file-upload/ I have never used Laravel before. If this must be ported to the Laravel Documentation it should be done be someone who has experience. |
|
I think the linting Problem is not caused by this PR. |
@timble-one Yes you're right, could you please rebase it from the |
graphql-normalizer needs to be injected
There was a problem hiding this comment.
Should be great for me after theses changes @timble-one
optimization from @vinceAmstoutz Co-authored-by: Vincent Amstoutz <vincent.amstoutz@outlook.com>
optimization from @vinceAmstoutz Co-authored-by: Vincent Amstoutz <vincent.amstoutz@outlook.com>
|
Thanks @timble-one! |
* graphql.md: add "MediaObject Normalization" chapter graphql-normalizer needs to be injected * graphql.md: add "MediaObject Normalization" chapter optimization from @vinceAmstoutz Co-authored-by: Vincent Amstoutz <vincent.amstoutz@outlook.com> * graphql.md: add "MediaObject Normalization" chapter optimization from @vinceAmstoutz Co-authored-by: Vincent Amstoutz <vincent.amstoutz@outlook.com> --------- Co-authored-by: Vincent Amstoutz <vincent.amstoutz@outlook.com>
graphql-normalizer needs to be injected