search

drupal-graphql / graphql

GraphQL integration for Drupal 9/10
288 stars 202 forks source link

feat(dataproducer): make access check optional in entity_label (#1382) #1383

Open edurenye opened 8 months ago

edurenye commented 8 months ago

Fixes #1382

edurenye commented 8 months ago

I'll try to fix the tests

edurenye commented 8 months ago

BTW this is related to https://www.drupal.org/sa-contrib-2023-050

edurenye commented 8 months ago

The issue with the tests is actually a bug in DataProducerExecutionTrait and has an issue for it: https://github.com/drupal-graphql/graphql/issues/1233

DataProducerExecutionTrait does not care about the dafault value, so it needs to be specified.

I just made a workarround for the tests with a todo that marks that the 'access' default value can be removed after that issue is fixed.