Antoine Bluchet
Antoine Bluchet
it looks like it breaks our tests, I'm surprised that this doesn't work as expected though.
should we target main instead for these? I'm not sure we cover the bc on every parameters that get removed in this patch
yes this would be perfect! Also could you point me to why we remove the graphql-playground ?
Thanks a lot @maxhelias !
tests need fixing
``` - Locking doctrine/persistence (4.0.0) ``` the doctrine-common package does fail with `4.0` I added a documentation to run component's tests: https://github.com/api-platform/core/blob/main/CONTRIBUTING.md#components-tests
Thanks this was fixed https://github.com/api-platform/core/pull/7276
I'm not sure how we could not lazy load, do you have a stack trace?
I see but I'm not sure how we can do otherwise, if you set the correct groups the property will not be read: https://github.com/api-platform/core/blob/79edced67ccca1a7b80455dd94203501d9c4fa89/src/Laravel/Eloquent/PropertyAccess/PropertyAccessor.php#L53-L55
By default all the properties of your Eloquent model will be serialized, therefore we try to access their value. To pick a set of values there are a few solutions:...