- 🇳🇿New Zealand quietone
The work here is longer suitable for a novice, in my opinion.
- First commit to issue fork.
- 🇺🇸United States dcam
Based on #22 I removed the collapsible behavior entirely.
- 🇺🇸United States smustgrave
Asking about that update hook number (small change if needed)
But we should have a CR for the configuration change. Mentioning why it's been removed.
- 🇺🇸United States dcam
Change record added at https://www.drupal.org/node/3515212 → . Leaving status as Needs Work due to the question of what the update number should be. I checked Slack and no one has responded yet. I'll try to check back in a day or two.
- 🇺🇸United States dcam
The update function number was updated per the discussion in Slack. I also rebased it against 11.x to get the latest changes.
- 🇺🇸United States smustgrave
Apologize for taking so long to get back to this one, even after posting it in slack lol
But feedback appears to be addressed.
- Status changed to RTBC
24 days ago 9:38am 12 May 2025 - 🇬🇧United Kingdom catch
The change record should explicitly reference the config key that's being removed. There will be some cases like distributions with a full set of exported config that could end up with stale config after this change and will need to remove the key manually.
I'm confused by the update number here - left a comment on the MR.
- 🇧🇷Brazil igorgoncalves
Hi guys
As the last feedback review was given by #41, i made another up-to-date check after all last commits and the changes seems to address the requests.
The advanced column "is gone" as the collapsible behavior.Checked with Drupal 11.2.x-dev