@moin Dieser PR DEV: Add resolve_group_membership functionality to theme settings - Pull Request #41360 - discourse/discourse - GitHub wurde jetzt gemerged und ich füge die Dokumentation in DEV: Docs for theme setting resolve_group_membership - Pull Request #41537 - discourse/discourse - GitHub hinzu. Ich habe discourse-copy-post mit DEV: Use resolve_group_membership core theme functionality - Pull Request #37 - discourse/discourse-copy-post - GitHub behoben und gehe jetzt unsere anderen offiziellen Plugins durch, um zu sehen, ob andere dieselbe Behandlung benötigen (z. B. hast du bereits auf GitHub - discourse/discourse-unanswered-filter · GitHub hingewiesen).
Für dein eigenes Filter Favorites kannst du das copy-post-Beispiel sehen, das ich gemacht habe, aber im Grunde musst du resolve_group_membership: true sowohl zu default_favorite_filters_groups als auch zu custom_favorite_filters_allowed_groups hinzufügen, dann werden sie als Boolesche Werte basierend auf den Gruppenmitgliedschaften des Benutzers auf settings als settings.user_in_default_favorite_filters_groups und settings.user_in_custom_favorite_filters_allowed_groups verfügbar sein, und du musst nicht mehr manuell die Gruppen-IDs der Benutzer im Frontend überprüfen.
Ich würde den Standardwert von default_favorite_filters_groups in deinem Theme-Component auf 4|5 ändern, was angemeldete und anonyme Benutzer bedeutet, anstatt 0 (everyone), der bald entfernt wird.
Ich werde einen Hinweis machen, eine Datenbankmigration für Theme-Einstellungen auszuführen, wenn ich die everyone-Gruppe entferne. Ich muss auch eine für Site-Einstellungen machen. Dies wird passieren, wenn die kommende Änderung in den nächsten Wochen endgültig wird (abhängig davon, ob andere Probleme auftreten und wie schnell ich andere Themes/Components durchgehe, die resolve_group_membership hinzugefügt bekommen müssen).
Nein, das ist im Moment nicht möglich, wie es für Site-Einstellungen der Fall ist… Ich denke, es ist nicht notwendig, das jetzt hinzuzufügen. Du kannst dies einfach in der Beschreibung der Einstellung erwähnen und im Frontend mit currentUser schützen. Vielleicht werde ich das in Zukunft hinzufügen, wenn der Bedarf größer ist.