Skip to content
This repository was archived by the owner on Jul 22, 2025. It is now read-only.

Commit e0d86e7

Browse files
committed
FEATURE: Show relevant settings in features tab only
This update hides feature related settings from main UI and only shows it in the features tab. This will help with having an overwhelming amount of settings in the settings tab and will make it easier to parse different settings.
1 parent 0963a6a commit e0d86e7

File tree

4 files changed

+22
-7
lines changed

4 files changed

+22
-7
lines changed

app/controllers/discourse_ai/admin/ai_features_controller.rb

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,11 @@ def feature_config
4545
description_key: "discourse_ai.features.summarization.description",
4646
persona_setting_name: "ai_summarization_persona",
4747
enable_setting_name: "ai_summarization_enabled",
48+
additional_settings: %w[
49+
ai_summary_backfill_topic_max_age_days
50+
ai_summary_backfill_maximum_topics_per_hour
51+
ai_summary_backfill_minimum_word_count
52+
],
4853
},
4954
{
5055
id: 2,
@@ -92,6 +97,14 @@ def persona_backed_features
9297
value: SiteSetting.get(feature[:enable_setting_name]),
9398
type: SiteSetting.type_supervisor.get_type(feature[:enable_setting_name]),
9499
},
100+
additional_settings:
101+
(feature[:additional_settings] || []).map do |setting_name|
102+
{
103+
name: setting_name,
104+
value: SiteSetting.get(setting_name),
105+
type: SiteSetting.type_supervisor.get_type(setting_name),
106+
}
107+
end,
95108
}
96109
end
97110
end

assets/javascripts/discourse/admin/models/ai-feature.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,8 @@ export default class AiFeature extends RestModel {
88
"description",
99
"enable_setting",
1010
"persona",
11-
"persona_setting"
11+
"persona_setting",
12+
"additional_settings"
1213
);
1314
}
1415
}

assets/javascripts/initializers/admin-plugin-configuration-nav.js

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -41,12 +41,11 @@ export default {
4141
route: "adminPlugins.show.discourse-ai-spam",
4242
description: "discourse_ai.spam.spam_description",
4343
},
44-
// TODO(@keegan / @roman): Uncomment this when structured output is merged
45-
// {
46-
// label: "discourse_ai.features.short_title",
47-
// route: "adminPlugins.show.discourse-ai-features",
48-
// description: "discourse_ai.features.description",
49-
// },
44+
{
45+
label: "discourse_ai.features.short_title",
46+
route: "adminPlugins.show.discourse-ai-features",
47+
description: "discourse_ai.features.description",
48+
},
5049
]);
5150
});
5251
},

config/settings.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -278,10 +278,12 @@ discourse_ai:
278278
default: 30
279279
min: 1
280280
max: 10000
281+
hidden: true
281282
ai_summary_backfill_maximum_topics_per_hour:
282283
default: 0
283284
min: 0
284285
max: 10000
286+
hidden: true
285287
ai_summary_backfill_minimum_word_count:
286288
default: 200
287289
hidden: true

0 commit comments

Comments
 (0)