Permissions oubliées dans l'onglet de sécurité de la catégorie

Suite à la discussion sur Ajout automatique des autorisations / Invite pour les autorisations non enregistrées :

Je pense qu’il faut aller un peu plus loin dans cette direction.

Si vous avez une modification en attente (1) et que vous cliquez sur « Enregistrer », nous devriez vous bloquer et vous indiquer que vous avez des modifications en attente à appliquer, via une fenêtre du type « Êtes-vous sûr ? ».

6 « J'aime »

Faut-il vraiment cliquer sur Enregistrer ? Je pense que cliquer sur le bouton Ajouter devrait suffire ?

3 « J'aime »

J’aime la fonction Enregistrer. C’est une étape supplémentaire pour s’assurer que les autorisations sont correctes. Sans elle, si je sélectionne le mauvais groupe dans la liste déroulante et que je clique sur :heavy_plus_sign:, ils obtiennent immédiatement l’accès, sans que j’aie l’occasion de corriger l’erreur.

3 « J'aime »

Une chose qui rend cela légèrement plus compliqué, c’est que le bouton « Enregistrer la catégorie » s’applique actuellement au contenu de tous les onglets/onglets. L’avertissement devrait probablement s’afficher si vous avez apporté des modifications à N’IMPORTE QUOI dans n’importe quelle section, si vous cliquez sur la croix.

6 « J'aime »

Salut @Osama, peux-tu prendre celui-ci ? Cela devrait être assez simple, mais fais-le-nous savoir si tu rencontres des problèmes.

4 « J'aime »

PR pour cela : UX: Show a warning when admin clicks save without adding group when changing category permissions by OsamaSayegh · Pull Request #7947 · discourse/discourse · GitHub

Capture d’écran :

Le bouton « Enregistrer la catégorie » déclenchera l’avertissement si l’utilisateur modifie l’un des menus déroulants sans cliquer sur le bouton plus. Ils peuvent cliquer à nouveau sur Enregistrer, ce qui ignorera le changement en attente et enregistrera quand même. Cliquer sur le bouton plus réinitialisera le déclencheur ; ainsi, s’ils ne modifient pas les menus déroulants après avoir cliqué sur le bouton plus, l’avertissement ne sera pas déclenché.

J’ai également changé la couleur du bouton plus en couleur principale afin qu’il se distingue un peu plus.

Est-ce que cela te semble correct ?

7 « J'aime »