From cb0aaa5197c6643c60eddfdae5049fbc892bc385 Mon Sep 17 00:00:00 2001 From: Hemant Umre Date: Sun, 3 Sep 2023 08:47:14 +0530 Subject: [PATCH] settings_org: Move video chat provider from update_dependent_subsettings. Since `settings_org.update_dependent_subsettings` handles settings whose value and state depend on other elements and video chat provider settings is an independent setting moving it out of update_dependent_subsettings. --- web/src/settings_org.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/web/src/settings_org.js b/web/src/settings_org.js index 9504518dfa..557054f354 100644 --- a/web/src/settings_org.js +++ b/web/src/settings_org.js @@ -577,9 +577,6 @@ function update_dependent_subsettings(property_name) { } switch (property_name) { - case "realm_video_chat_provider": - set_video_chat_provider_dropdown(); - break; case "realm_allow_message_editing": update_message_edit_sub_settings(page_params.realm_allow_message_editing); break; @@ -712,6 +709,9 @@ export function discard_property_element_changes(elem, for_realm_default_setting case "realm_move_messages_between_streams_limit_seconds": set_msg_move_limit_setting(property_name); break; + case "realm_video_chat_provider": + set_video_chat_provider_dropdown(); + break; case "realm_message_retention_days": case "message_retention_days": set_message_retention_setting_dropdown(sub);