Impossible de changer member_visibility_level des groupes automatiques

Il n’est pas possible de modifier la visibilité des membres des groupes automatiques via l’interface utilisateur de gestion des groupes. Plus précisément : l’interface utilisateur permet de sélectionner une nouvelle valeur et signale joyeusement « Enregistré ! » lorsque vous appuyez sur Enregistrer les modifications, mais la requête est silencieusement ignorée par le backend.

Cela ressemble à une régression qui a été introduite dans la méthode group_params() de GroupsController dans FIX: Add `errors` field if group update confirmation by udan11 · Pull Request #16260 · discourse/discourse · GitHub

2 « J'aime »

J’ai également rencontré ce problème et je trouve déroutant de pouvoir modifier le paramètre de visibilité, puis qu’il soit ignoré silencieusement.

N’y a-t-il aucun moyen d’afficher les administrateurs et les niveaux de confiance à tout le monde sur la page des groupes ?

Il semble qu’il existe une version de cette demande avec une #demande de fonctionnalité et plus de discussion ici :

2 « J'aime »

Je viens de tester ceci et je ne parviens pas à reproduire le problème exactement, mais nous testons peut-être des choses différentes.

Ici sur meta, j’ai changé la visibilité du groupe trust_level_4 de Utilisateurs connectés à Tout le monde. Avant le changement, visiter https://meta.discourse.org/g/trust_level_4 renvoyait un 404, après le changement, il renvoie la liste des membres du groupe. J’ai depuis annulé cette modification ici afin que cette route redevienne un 404 pour les anonymes, mais changer la visibilité fonctionne. J’ai également testé la même chose localement.

Le problème est-il qu’après avoir changé la visibilité du groupe, le groupe n’est pas listé dans /groups pour les utilisateurs réguliers ? C’est pour des raisons de désencombrement (ce n’est pas une régression récente, c’est ainsi que cela a été conçu en 2017)… les utilisateurs réguliers ne peuvent pas rejoindre les groupes automatiques, donc il n’y a pas beaucoup d’utilité à les lister dans le répertoire, même lorsque la visibilité le permet.

1 « J'aime »

Dans cette optique, je pense que nous pouvons clore la version Bug de ce fil et concentrer la conversation dans la version Feature de Respect the visibility settings of all automatic groups. :+1:

2 « J'aime »

Réouverture car j’étais peut-être confus quant à la nature de ce bug… :slight_smile:

Ma reproduction de base :

  • Allez sur g/trust_level_1/manage/interaction
  • Changez « Qui peut voir les membres de ce groupe ? » pour autre chose que « Tout le monde » et enregistrez
  • Actualisez la page

Attendu : cela change
Réel : cela revient à « Tout le monde »

2 « J'aime »

Merci @mdoggydog, j’ai mal lu le rapport. Comme @JammyDodger, je peux reproduire le problème avec la liste déroulante « Qui peut voir les membres de ce groupe ? ».

Une correction est en cours de révision sur

4 « J'aime »

Youpi, merci @pmusaraj — le correctif semble même fonctionner ! :wink:

Il y a un bug structurel sous-jacent qui doit encore être étudié et corrigé : le fait qu’un utilisateur puisse appuyer sur « Enregistrer » sur le frontend et que le frontend signale « Enregistré ! », même lorsque le backend n’a rien enregistré. Je ne suis pas sûr où/comment signaler cela, mais c’est une dette technique qui devrait être suivie quelque part jusqu’à sa résolution.

3 « J'aime »

Ce sujet a été automatiquement fermé après 2 jours. Les nouvelles réponses ne sont plus autorisées.