لا يمكن تغيير member_visibility_level للمجموعات التلقائية

[mdoggydog]

لا يمكن تغيير رؤية الأعضاء للمجموعات التلقائية عبر واجهة مستخدم إدارة المجموعات. وبشكل أكثر تحديدًا: تسمح واجهة المستخدم بتحديد قيمة جديدة، وتفيد بـ “تم الحفظ!” بسعادة عند الضغط على “حفظ التغييرات”، ولكن يتم تجاهل الطلب بصمت بواسطة الواجهة الخلفية.

يبدو أن هذا تراجع تم تقديمه في طريقة group_params() في GroupsController في FIX: Add `errors` field if group update confirmation by udan11 · Pull Request #16260 · discourse/discourse · GitHub

[manuel (Manuel Kostka)]

وأيضًا صادفت هذا للتو وأجد أنه من المحير أنني أستطيع تغيير إعداد الرؤية، ولكن بعد ذلك يتم تجاهله بصمت.

هل لا توجد طريقة لإظهار المسؤولين ومستويات الثقة للجميع في صفحة المجموعات؟

[JammyDodger]

يبدو أن هناك نسخة من هذا الطلب Feature request مع المزيد من المناقشة في:

Respect the visibility settings of all automatic groups

(One person’s feature request is another person’s bug report… ) The “Groups” index page (e.g., /groups or /g), should respect the visibility settings of the automatic groups, and display them accordingly. It does this for the Moderators group, but not the others. This is due to a hard-coded exception in GroupsController.index(), which causes the other automatic groups to never appear on the index page when viewed by non-staff users, no matter what the visibility settings are…

[pmusaraj (Penar Musaraj)]

لقد اختبرت هذا للتو ولا يمكنني إعادة إنتاج المشكلة بالضبط، ولكن قد نكون نختبر أشياء مختلفة.

هنا في meta، قمت بتغيير رؤية مجموعة trust_level_4 من المستخدمين المسجلين إلى الجميع. قبل التغيير، كان زيارة https://meta.discourse.org/g/trust_level_4 يعيد 404، وبعد التغيير يعيد قائمة الأعضاء في المجموعة. لقد عكس هذا التعديل هنا منذ ذلك الحين بحيث يعود هذا المسار إلى 404 للمستخدمين المجهولين، ولكن تغيير الرؤية يعمل. تم اختبار نفس الشيء محليًا أيضًا.

هل المشكلة هي أنه بعد تغيير رؤية المجموعة، لا يتم إدراج المجموعة في /groups للمستخدمين العاديين؟ هذا لأسباب تتعلق بتقليل الفوضى (إنه ليس تراجعًا حديثًا، هكذا تم تصميمه في عام 2017)… لا يمكن للمستخدمين العاديين الانضمام إلى المجموعات التلقائية، لذلك لا يوجد الكثير من الفائدة في إدراجها في الدليل حتى عندما تسمح الرؤية بذلك.

[JammyDodger]

@pmusaraj

(إنها ليست مشكلة حديثة، بل هكذا تم تصميمها في عام 2017)…

مع أخذ هذا في الاعتبار، أعتقد أنه يمكننا إغلاق النسخة Bug من هذا الموضوع وتركيز المحادثة في النسخة Feature على Respect the visibility settings of all automatic groups.

[JammyDodger]

إعادة فتح الموضوع لأنني ربما كنت مرتبكًا بشأن ما تتضمنه هذه المشكلة…

@mdoggydog

مهلاً… لم يتم إصلاح هذه المشكلة - لا تزال موجودة حتى في 3.2.0.beta5-dev. وهي مشكلة بحد ذاتها - مستقلة عن السلوك في طلب الميزة المذكور. (وقد تسللت إلى الكود في عام 2022، بشكل مستقل عن قرار التصميم المتخذ في عام 2017. ولكن، من المحتمل أن القرارات التي اتخذت في عام 2017 جعلت من الصعب ملاحظة هذه المشكلة.)

هذه المشكلة ليست متعلقة بإعداد “من يمكنه رؤية هذه المجموعة؟” (visibility_level)، بل تتعلق بإعداد “من يمكنه رؤية أعضاء هذه المجموعة؟” (members_visibility_level).

حاول تغيير “من يمكنه رؤية أعضاء هذه المجموعة؟” لمجموعة تلقائية… اضغط على حفظ… ثم قم بتحديث صفحة الويب الخاصة بك وانظر ما إذا كان قد تم حفظ التغيير بالفعل أم لا…

إعادة الإنتاج الأساسية الخاصة بي:

• انتقل إلى g/trust_level_1/manage/interaction
• قم بتغيير “من يمكنه رؤية أعضاء هذه المجموعة؟” إلى أي شيء آخر غير “الجميع” واحفظ
• قم بتحديث الصفحة

المتوقع - يتغير
الفعلي - يعود إلى “الجميع”

[pmusaraj (Penar Musaraj)]

شكراً @mdoggydog، لقد أخطأت في قراءة التقرير. مثل @JammyDodger، يمكنني إعادة إنتاج المشكلة باستخدام القائمة المنسدلة “من يمكنه رؤية أعضاء هذه المجموعة؟”.

التصحيح قيد المراجعة على

FIX: Allow staff to change group members visibility level for automatic groups

[mdoggydog]

رائع، شكرًا لك @pmusaraj — يبدو أن الإصلاح يعمل أيضًا!

هناك خطأ هيكلي أساسي لا يزال بحاجة إلى التحقيق والإصلاح: حقيقة أن المستخدم يمكنه الضغط على “حفظ” في الواجهة الأمامية وستبلغ الواجهة الأمامية عن “تم الحفظ!”، حتى عندما لم يحفظ الواجهة الخلفية أي شيء. لست متأكدًا من أين/كيف يمكن تقديم ذلك، ولكنه دين تقني يجب تتبعه في مكان ما حتى يتم إصلاحه.