Schwierigkeiten beim Verständnis der Einstellung für das Willkommensbanner

martin · 12. August 2025 um 04:19

Ich habe die Korrektur jetzt zusammengeführt. Es hat etwas länger gedauert, weil ich herausfinden musste, wie ich sie testen kann:

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

Ich werde deine Seite bald bereitstellen, Moin.

Bearbeiten: Die Korrektur wurde bereitgestellt.

Thema		Antworten	Aufrufe
Welcome Link Banner Theme component official , welcome-link-banner	42	15087	9. Juli 2025
Advanced Search Banner Theme component official , search-banner , advanced-search-banner	66	22658	19. November 2025
Versatile Banner Theme component official , versatile-banner	99	43406	27. Oktober 2025
Notification Banners Theme component	50	1702	19. November 2025
Themeable site settings are now available Announcements	6	403	1. Oktober 2025

Schwierigkeiten beim Verständnis der Einstellung für das Willkommensbanner

Verwandte Themen