- 🇦🇺Australia mstrelan
As per #3 this doesn't seem to make sense to have in core. Perhaps fieldgroup and other modules can alter this to add support. Attaching a patch to remove the parent column / fields and see what breaks.
- Status changed to Needs work
about 2 years ago 9:44pm 19 February 2023 - 🇺🇸United States smustgrave
This issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request → as a guide.
Can the issue summary be updated? Makes reference for Drupal 8.0.5, just want to confirm Drupal 10.1.x is still valid. Also some TBD were in there.
As a bug will need a test case as well.
Thanks!
- 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10
Would this not break fieldgroup module pretty hard? Can we just prevent it from erroring as it stands?
- Status changed to Needs review
about 2 years ago 6:01am 28 February 2023 - 🇦🇺Australia mstrelan
Uploading another patch that should fix the test failures.
Note that this completely removes the "parent" concept as I haven't be able to find where it's used in core, and I suspect passing tests will reveal that it's not, or it's at least not covered. Fieldgroup would likely need its own customisations to allow for choosing a parent.
The last submitted patch, 15: 2699907-15.patch, failed testing. View results →
