启用欢迎横幅设置的难度

martin · 2025 年8 月 12 日 04:19

Merged the fix now, it took a while longer because I needed to figure out how to test it:

github.com/discourse/discourse

FIX: Theme site settings not reloading across processes

main ← issue/fix-multi-process-tss-cache

opened 02:27AM - 12 Aug 25 UTC

martin-brennan

+86 -4

This commit fixes an issue in multi-process setups where the following would ha…ppen: 1. Admin changes a theme site setting. The new value is stored in memory in the process that made the change. For this example, changing `enable_welcome_banner` from `true` to `false` 2. Another user visits the site and hits another process, which has the old value of `enable_welcome_banner` still set to `true` in memory 3. The other user's value for `enable_welcome_banner` is cached in the cache for theme site settings, which is shared across all processes The issue is fixed by doing the same thing as site settings: when the new value is saved, we send a MessageBus message to all processes via `SiteSettingExtension` which calls `refresh!` and reloads the in-memory cache of theme site settings from the DB. c.f. https://meta.discourse.org/t/difficulty-understanding-the-enable-welcome-banner-setting/377321

I will deploy your site soon Moin.

Edit: Fix has been deployed

话题		回复	浏览量
Welcome Link Banner Theme component official , welcome-link-banner	42	14833	2025 年7 月 9 日
Versatile Banner Theme component official , versatile-banner	97	42839	2025 年8 月 28 日
Advanced Search Banner Theme component official , advanced-search-banner	61	22133	2025 年8 月 26 日
Themeable site settings are now available Announcements	5	308	2025 年7 月 23 日
Alien Night Theme - A free Dark Theme for Discourse Theme	78	27254	2025 年2 月 4 日

启用欢迎横幅设置的难度

相关话题