- Issue created by @Maeglin
- First commit to issue fork.
- 🇦🇺Australia darvanen Sydney, Australia
Thank you so much for reporting this and for the patch!
I've created an MR which also has a couple of maintenance chores in it that I was planning to do soon. Assuming it goes green I'll make a new release with this fix in very soon.
-
darvanen →
committed 8bb0e2c8 on 2.x
Issue #3478851 by darvanen, maeglin: Missing type in SettingsForm...
-
darvanen →
committed 8bb0e2c8 on 2.x
- 🇦🇺Australia darvanen Sydney, Australia
Alright, the only test that is failing now is the webform validation handler test, and only on the previous major test run. The only change we made to the validation handler was to alphabetise the use statements so I'm going to say that failure is out of scope for this issue.
- 🇦🇺Australia darvanen Sydney, Australia
Yes I see that now, will put any work or thoughts on 🐛 Drush error Active
- 🇵🇹Portugal adaragao
Updated. Working fine now.
Thanks you for all your work @darvanen. - 🇮🇳India ankitv18
Hi @darvanen,
$typedConfigManager must be Drupal\\Core\\Config\\TypedConfigManagerInterface introduced in the D10.2.x
It would be better we add a minimum support of D10.2 into the info.yml - 🇦🇺Australia darvanen Sydney, Australia
Thanks @ankitv18, but with the approach taken in 🐛 Drush error Active that avoids touching the constructor that won't be necessary.
Automatically closed - issue fixed for 2 weeks with no activity.