@moin cette PR DEV: Add resolve_group_membership functionality to theme settings - Pull Request #41360 - discourse/discourse - GitHub est maintenant fusionnée et j’ajoute de la documentation dans DEV: Docs for theme setting resolve_group_membership - Pull Request #41537 - discourse/discourse - GitHub . J’ai corrigé discourse-copy-post avec DEV: Use resolve_group_membership core theme functionality - Pull Request #37 - discourse/discourse-copy-post - GitHub et maintenant je passe en revue nos autres plugins officiels pour voir si d’autres ont besoin du même traitement (par exemple, vous avez déjà pointé vers GitHub - discourse/discourse-unanswered-filter · GitHub )
Pour votre propre Filter Favorites , vous pouvez voir l’exemple de copy-post que j’ai fait, mais en gros vous devez ajouter resolve_group_membership: true à la fois à default_favorite_filters_groups et à custom_favorite_filters_allowed_groups , alors ils seront disponibles en tant que booléens selon les appartenances aux groupes de l’utilisateur sur settings en tant que settings.user_in_default_favorite_filters_groups et settings.user_in_custom_favorite_filters_allowed_groups respectivement, et vous n’avez plus besoin de vérifier manuellement les identifiants de groupe des utilisateurs sur le frontend.
Je changerais la valeur par défaut de default_favorite_filters_groups en 4|5 dans votre composant de thème, qui correspond aux utilisateurs connectés et anonymes, plutôt qu’à 0 (everyone) qui sera bientôt supprimé.
Je ferai une note pour exécuter une migration de la base de données sur les paramètres de thème lorsque je me débarrasserai du groupe everyone, j’en ai besoin pour les paramètres du site aussi, cela se produira lorsque le changement à venir deviendra permanent dans les prochaines semaines (selon si d’autres problèmes surgissent et à quelle vitesse je parviens à corriger d’autres thèmes/composants qui nécessitent l’ajout de resolve_group_membership).
Non, il n’y a pas de moyen de faire cela pour le moment comme c’est le cas pour les paramètres du site… Je ne pense pas que ce soit nécessaire d’ajouter cela maintenant, vous pouvez simplement mentionner cela dans la description du paramètre et protéger avec currentUser dans le frontend de toute façon. Peut-être que dans le futur, j’ajouterai cela s’il y a plus besoin.