Merge pull request #6140 from sbwalker/dev

add comment related to ISettingsControl usage in Module Settings
This commit is contained in:
Shaun Walker
2026-03-26 20:19:46 -07:00
committed by GitHub

View File

@@ -298,7 +298,8 @@
} }
else else
{ {
// legacy support - module settings updated by convention ( ie. by calling a public method named "UpdateSettings" in settings component ) // legacy approach - module settings updated by convention (ie. by calling a public method named "UpdateSettings" in settings component)
// this method should be removed however the ISettingsControl declaration was not added to the default module template until version 10.1.2
_moduleSettings?.GetType().GetMethod("UpdateSettings")?.Invoke(_moduleSettings, null); _moduleSettings?.GetType().GetMethod("UpdateSettings")?.Invoke(_moduleSettings, null);
} }
} }