- Issue created by @maxilein
- Status changed to Needs review
almost 2 years ago 1:26pm 20 March 2023 - First commit to issue fork.
- @justcaldwell opened merge request.
- 🇺🇸United States justcaldwell Austin, Texas
Confirmed. Looks like the use statement didn't make it into the MR for 📌 Remove epp default dependency in all fields config sync files Fixed . That's on me!
This MR is slightly different from the patch in that uses
Drupal\Core\Field\FieldConfigInterface;
rather thanDrupal\field\FieldConfigInterface;
.Elevating as this is at least Major.
- Status changed to RTBC
almost 2 years ago 11:40pm 20 March 2023 - 🇩🇪Germany geek-merlin Freiburg, Germany
> This MR is slightly different from the patch in that it uses Drupal\Core\Field\FieldConfigInterface; rather than Drupal\field\FieldConfigInterface;.
Yes this looks right. That's really a disturbing part of the API. But BaseFieldOverride implementing this but not the other speaks for this.
Let's commit the MR.
-
geek-merlin →
committed 0a41eb52 on 8.x-1.x authored by
justcaldwell →
Issue #3349027 by justcaldwell, geek-merlin, Adil_Siddiqui, maxilein:...
-
geek-merlin →
committed 0a41eb52 on 8.x-1.x authored by
justcaldwell →
- Status changed to Fixed
almost 2 years ago 11:48pm 20 March 2023 Automatically closed - issue fixed for 2 weeks with no activity.