ترقيم/تمرير لا نهائي في قائمة المراجعة بعد اتخاذ إجراء

لاحظت أنه بعد معالجة عنصر في قائمة المراجعة، مثل “اعتماد المنشور”، يصبح من المستحيل التمرير إلى الأمام (تحميل المزيد عبر التمرير اللانهائي). على الأقل في حالتي عند استخدام خيار “مرتّب حسب:” بـ “تاريخ الإنشاء”.

المتصفح: فايرفوكس سطح المكتب
ديسكورش: 2.5.0.beta5

إعجاب واحد (1)

أنا متأكد تمامًا من أننا قمنا بتحسين هذه الصفحة إلى حد كبير خلال العام الماضي وأصلحنا مشكلة التحميل.

هل يمكنك الإشارة إلى إعادة الفتح إذا كانت هذه المشكلة لا تزال قائمة.

إعجاب واحد (1)

@markersocial، هل يمكنك مشاركة أي تفاصيل جديدة؟ يبدو أن هذا لا يزال يحدث بالنسبة لك.

سيكون من المفيد معرفة الفلاتر المحددة التي تستخدمها، حتى نتمكن من محاولة إعادة إنتاج هذه المشكلة.

إعجابَين (2)

بالتأكيد :slight_smile:

لذا قمت باختبار هذا مرة أخرى على الإصدار 2.8.0beta4 وما زال يحدث معي مع ترتيب الفرز الافتراضي (النقاط). أعتقد أن الأمر نفسه ينطبق على أي خيارات فرز أخرى بناءً على الاختبارات السابقة.

لإعادة إنتاج المشكلة، تحتاج فقط إلى وجود 11 عنصرًا على الأقل في قائمة المراجعة. إذا قمت بالتعامل مع أي عنصر في أعلى القائمة (موافقة/رفض وما إلى ذلك)، فلن يكون من الممكن بعد ذلك التمرير لأسفل والتنقل إلى العنصر الأخير في القائمة. يجب تحديث القائمة بالكامل بالكامل من أجل القدرة على التنقل مرة أخرى.

إعجابَين (2)

ممتاز، شكرًا للتفاصيل! @sam، تمكنت من تكرار المشكلة. تكرار سهل كما اقترح @markersocial - علم ما لا يقل عن 11 منشورًا، افتح قائمة المراجعة، تعامل مع العلم الأول، ثم مرر للأسفل.

3 إعجابات

دعني أبدأ بالقول إنني لست على دراية بـ Ember، لذا قد أستخدم مصطلحات غير دقيقة هنا وهناك.

بعد حل عنصر reviewable، يتم إزالته من قائمة reviewables الحالية عن طريق استدعاء reject:

https://github.com/discourse/discourse/blob/main/app/assets/javascripts/discourse/app/controllers/review-index.js#L107

يرجع reject مصفوفة. عند تعيين متغير reviewables إلى مصفوفة newList (السطر 114)، يتم استبدال الفئة التي تحتوي على طرق إضافية مثل loadMore() بمصفوفة أساسية. (تعديل: هذا هو سبب عمل التحميل بشكل صحيح قبل حل العنصر ولكن ليس بعده)

إصلاح محتمل هو استبدال this.set("reviewables", newList); بـ this.reviewables.set("content", newList);.

3 إعجابات

تم إصلاح هذا الآن :arrow_down:

5 إعجابات