From f8ca688b2d706eaca65a35e9093c6284f5c3284c Mon Sep 17 00:00:00 2001 From: sbwalker Date: Thu, 26 Mar 2026 20:19:30 -0700 Subject: [PATCH] add comment related to ISettingsControl usage in Module Settings --- Oqtane.Client/Modules/Admin/Modules/Settings.razor | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/Oqtane.Client/Modules/Admin/Modules/Settings.razor b/Oqtane.Client/Modules/Admin/Modules/Settings.razor index bde62f80..cf84e971 100644 --- a/Oqtane.Client/Modules/Admin/Modules/Settings.razor +++ b/Oqtane.Client/Modules/Admin/Modules/Settings.razor @@ -298,7 +298,8 @@ } 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); } }