- Issue created by @RichardGaunt
- Merge request !81Updated ExternalEntityTypeInterface to extend ConfigurableExternalEntityTypeInterface. → (Open) created by RichardGaunt
- 🇦🇺Australia RichardGaunt Melbourne
Created a MR that fixes this issue:
https://git.drupalcode.org/project/external_entities/-/merge_requests/81Not sure whether inheritance should be this way as ConfigurableExternalEntityTypeInterface feels like an extension on ExternalEntityTypeInterface but its a starting point.
- 🇦🇺Australia RichardGaunt Melbourne
https://git.drupalcode.org/issue/external_entities-3516546/-/jobs/4830977
Tests are failing with this change but won't fix until I know the direction to take.
- 🇫🇷France guignonv Montpellier
OK. I think I understand the problem but I need to think if your approach is the best. There might be a deeper design issue (-probably my bad-). I'm working other projects in parallel so I'll do my best to think about it as soon as I can and come back to you here. No hurry?
- 🇦🇺Australia RichardGaunt Melbourne
I am all good I made a local patch to fix for my purposes.