Plots2: إخفاء الردود على التعليقات التي تم إرسالها بالبريد العشوائي أو التي لم يتم الإشراف عليها بعد

تم إنشاؤها على ١٥ ديسمبر ٢٠٢٠  ·  23تعليقات  ·  مصدر: publiclab/plots2

يبدو أنه لا يتم تصفية الردود على التعليقات بحثًا عن البريد العشوائي. يجب أن نرى هذه الرسالة:

"هل أنت متأكد؟ لن يتمكن المستخدم بعد الآن من تسجيل الدخول أو النشر ، وسيتم إخفاء المحتوى الخاص به باستثناء التعليقات ". يبدو خطأ

مثال على حدوث ذلك: https://publiclab.org/notes/bhamster/09-02-2020/public-lab-virtual-event-on-all-things-microplastics

انظر هذا الرد على تعليق:

image

بدلاً من ذلك ، يجب أن تحتوي على رسالة مثل هذه ، والتي تعمل من أجل التعليقات المنتظمة (تلك التي ترد على منشور ، وليس تعليقًا آخر:

image

مشكلة

(قد يتم إصلاح هذا الآن ... لنكتب اختبارًا لتأكيد ذلك وإصلاح أي إخفاقات قد تنشأ عن الاختبار):

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

https://github.com/publiclab/plots2/search؟q=replied_comments

لا يمكننا فعل ذلك في التعريف لأنها في الواقع علاقة ActiveRecord ، وليست طريقة مخصصة يمكننا إضافة عوامل تصفية إلى: https://github.com/publiclab/plots2/blob/828321c2a8fd91e7a55140a511ed4e6dda718b61/app/models/comment.rb# إل 10

يجب علينا تصفية status = 0 في تعريف comment.replied_comments ، وكذلك لفلترة status = 4 اعرض الرسالة أعلاه.

بعد ذلك ، يجب أن نكتب بعض الاختبارات الأساسية:

https://github.com/publiclab/plots2/blob/876d0fc084064aaecc23f8003630d7d1ab858fa1/test/functional/comment_controller_test.rb#L57 -L61

Ruby help wanted testing

التعليق الأكثر فائدة

@ 17sushmita نعم ، بالتأكيد ، يرجى المضي قدمًا ، سيكون ذلك موضع تقدير كبير!

ال 23 كومينتر

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

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

تضمين التغريدة أعتقد أن شخصًا آخر فعل ذلك

cesswairimu يمكن

@ noi5e أوافق ، سأقوم بتحديث هذا ليكون أكثر من مشكلة في كتابة الاختبار. شكرا

حسنًا ، لست متأكدًا - ربما يأتي الاختبار أولاً لتأكيد ما إذا كان قد تم إصلاحه بالفعل؟

نظرًا لأنه لا يمكننا التصفية إلا عند الاستخدام ، وليس في تعريف has_many ، على ما يبدو ، ولا أرى .where.not(status: 0) على هذه:

https://github.com/publiclab/plots2/blob/ebf57e0cdd1dcc9d5cc884a15e7cd5b5bbe78390/app/views/notes/_comments.html.erb#L8

https://github.com/publiclab/plots2/blob/401874455490c6300270ef54490fe9452e0ca522/app/views/questions/show.html.erb#L69

https://github.com/publiclab/plots2/blob/ebf57e0cdd1dcc9d5cc884a15e7cd5b5bbe78390/app/views/notes/_comment.html.erb#L162

ماذا تعتقد؟

شكرا لكم جميعا!!! 💯

آه ، وأعتقد أنه على تلك includes() ، يجب أن يتم تسميتها ، لذلك ربما ... .where('comments.status != 0') ؟

نعم أعتقد أن هذا صحيح ، انظر الاستخدام هنا:

https://github.com/publiclab/plots2/blob/672d63ab11c2159e28d2cc502c47a3b3c83730b9/app/models/node.rb#L1114

jywarrencesswairimu @ noi5e العمل مايو I على كتابة الاختبارات لهذا؟

@ 17sushmita نعم ، بالتأكيد ، يرجى المضي قدمًا ، سيكون ذلك موضع تقدير كبير!

يبدو أنه لا يتم تصفية الردود على التعليقات بحثًا عن البريد العشوائي. يجب أن نرى هذه الرسالة:

"هل أنت متأكد؟ لن يتمكن المستخدم بعد الآن من تسجيل الدخول أو النشر ، وسيتم إخفاء المحتوى الخاص به باستثناء التعليقات ". يبدو خطأ

jywarren ماذا يعني هذا بقول هذه الرسالة تبدو خاطئة؟ ألا يُفترض أن يتم عرضها عندما يحاول الوسيط وضع علامة على التعليق على أنه بريد عشوائي؟ يرجى إعطاء المزيد من السياق لها.

@ 17sushmita أنا مرتبك قليلاً من ذلك أيضًا. أعتقد أنه ربما قصد جيفري أن النص كان يظهر باللون الأصفر في الصورة أدناه ، بدلاً من Moderate first-time comment ؟ إذا كان الأمر كذلك ، فهذا لا يبدو خطأ.

102243313-05b48f80-3ec9-11eb-919a-d2dd39d84c45

بينما ننتظر التوضيح ، لا يزال بإمكانك المضي قدمًا وكتابة اختبار. سأقوم فقط بنشر بعض الاقتراحات هنا التي قد تكون مفيدة.

لتبدأ ، يمكنك إدراج تعليق بريد عشوائي / تعليق ملصق لأول مرة في قاعدة بيانات الاختبار ليكون ردًا على تعليق آخر:

nodes(node_name).add_comment({
  uid: user_id,
  body: comment_text
  reply_to: parent_comment_id
  status: 0 # comment status: 0 for banned, 4 for first-time poster (a moderator has to approve the comment)
})

ثم اكتب بقية الاختبار من هناك ، مع التأكد من أن الطريقة التي نستخدمها لاسترداد التعليقات لا تحصل على ردود تعليق بالحالة 0 أو 4.

شئ مثل هذا! ربما يتعين عليك إجراء بعض الأبحاث. بالتأكيد اترك تعليقًا هنا إذا واجهتك مشكلة ، أو كنت بحاجة إلى بعض المؤشرات لمعرفة مكان البحث في قاعدة التعليمات البرمجية. نحن هنا للمساعدة!

@ noi5e ، شكرًا جزيلاً على المساعدة 😃️ !! أحتاج إلى توضيح آخر. هل أحتاج إلى إنشاء ملف جديد لاختبارات الرد على التعليقات أو إجراء تغييرات في ربما /plots2/test/functional/admin_controller_test.rb أو /plots2/test/system/spam2_test.rb أو /plots2/test/functional/comment_controller_test.rb أو أي ملف آخر؟

@ 17sushmita لست متأكدًا بنسبة 100٪ من هذا ، لكنني أفكر في /test/functional/comment_controller_test.rb . المكان الآخر الذي كنت أفكر فيه كان اختبار وحدة للتعليقات. افكار من اي شخص اخر؟

في هذه الأثناء ، لا تتردد في بدء العمل على هذا بينما ننتظر حتى يثني الآخرون.

@ 17sushmita لست متأكدًا بنسبة 100٪ من هذا ، لكنني أفكر في /test/functional/comment_controller_test.rb . المكان الآخر الذي كنت أفكر فيه كان اختبار وحدة للتعليقات. افكار من اي شخص اخر؟

في هذه الأثناء ، لا تتردد في بدء العمل على هذا بينما ننتظر حتى يثني الآخرون.

شكرًا ، لقد بدأت في كتابة الاختبارات ، ولكن هناك شيء واحد لاحظته وهو أنه حتى بعد وضع علامة على التعليق على أنه بريد عشوائي ، فإنه يظهر بالفعل ووفقًا للبرنامج فهو صحيح لأنه أثناء وضع علامة على هذا التعليق غير مرغوب فيه ، فإنه يولد تحذيرًا كما هو مذكور بواسطة jywarren أعلاه "هل أنت متأكد؟ لن يتمكن المستخدم بعد الآن من تسجيل الدخول أو النشر ، وسيتم إخفاء محتواه باستثناء التعليقات ." ولكن ، هل من المناسب حقًا عرض التعليقات المميزة بعلامة بريد عشوائي؟ 🤔️

هل من المناسب حقًا عرض التعليقات المميزة بعلامة بريد عشوائي؟ 🤔️

لا أعتقد ذلك!

هذا النص الذي ذكرته ممتع للغاية:

"هل أنت متأكد؟ لن يكون المستخدم قادرًا على تسجيل الدخول أو النشر ، وسيتم إخفاء المحتوى الخاص به باستثناء التعليقات. "

يجعلني أعتقد أنه ربما في اليوم السابق ، لم نكن ندير التعليقات. أعتقد أن هذا النص قد عفا عليه الزمن.

مهما كان الأمر ، أعتقد أننا نتفق جميعًا على أنه يجب علينا بالتأكيد تصفية جميع التعليقات! لذلك يمكننا تجنب spambots التي ذكرها jywarren في الأصل .

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

سنقوم بتقسيم هذا إلى أبعد من ذلك بقليل. محليًا ، قمت بإنشاء حساب جديد ونشرت تعليقًا على ملاحظة. هذا ما أراه عندما أنشر تعليقًا. حتى الان جيدة جدا:

Screen Shot 2021-03-22 at 11 39 30 AM

بصفتي مستخدمًا ، فإن رسالة _ "الموافقة المعلقة من قبل مشرفي المجتمع" _ تجعلني أعتقد أن تعليقي لن يظهر علنًا.

ومع ذلك ، فإن التعليق لا يظهر علنا! عندما أزور نفس الملاحظة في نافذة التصفح المتخفي التي تم تسجيل خروجها من الموقع ، يظهر التعليق تمامًا كما ظهر للمستخدم _cisco_:

Screen Shot 2021-03-22 at 11 43 21 AM

لا يبدو أن حظر المستخدم _cisco_ يغير الأمور كثيرًا أيضًا. تختفي رسالة _ "الموافقة المعلقة" _ ، ولكن لا يزال التعليق قابلاً للعرض من نافذة التصفح المتخفي:

Screen Shot 2021-03-22 at 11 46 06 AM

لذا نعم ، هذا بالتأكيد بحاجة إلى التغيير.

@ 17sushmita انطلق واستمر في العمل على هذا!

أيضًا ، إذا لم تعثر عليه بعد ، فلدينا مستخدم بريد مزعج_ محظور تم إنشاؤه بالفعل في /test/fixtures/users.yml

@ noi5e كان هذا

جانبا ، يبدو أن الردود على التعليقات لا تظهر حاليًا في لوحة تحكم المشرف على الإشراف على الرسائل غير المرغوب فيها:

Screen Shot 2021-03-22 at 11 40 21 AM

وإذا نشر مستخدم cisco ردًا على تعليق ، فلن يجعله يظهر في "المستخدمون النشطون":

Screen Shot 2021-03-22 at 11 40 35 AM

مشكلة محتملة لمشروع Outreachy Spam moderation المذكور في # 9257؟

شكرًا لك @ 17sushmita و @ noi5e على تفريغ هذا @ noi5e صحيح تمامًا - نحتاج فقط إلى رسالة أبسط "تعليق معتدل لأول مرة".

تصفية البريد العشوائي أو ردود التعليقات الخاضعة للإشراف لأول مرة

وصيد رائع لا يزال لم يتم حله بالنسبة للردود.

نقدر سوبر كل مساعدتكم ، كلاكما. 🎉

سأذكر مشكلة لوحة معلومات البريد العشوائي في وصف المشروع الجديد !!!

مرحبًا jywarren @ noi5e ، آسف للسؤال مرة أخرى ولكن كان لدي ارتباك آخر. هل يجب أن نعرض تعليقًا ليس بريدًا عشوائيًا ولكن المستخدم محظور؟ ماذا تفعل في مثل هذه الحالات؟

نعم ، حاليًا لا تؤثر حالة المستخدمين المحظورين على رؤية ملفات
تعليقات. شكرا لك!

يوم الأربعاء ، 24 آذار (مارس) 2021 ، 8:17 صباحًا Sushmita @ . * > كتب:

مرحبًا jywarren https://github.com/jywarren @ noi5e
https://github.com/noi5e ، آسف للسؤال مرة أخرى ولكن كان لدي واحد آخر
الالتباس. هل يجب أن نعرض تعليقًا ليس بريدًا عشوائيًا ولكن المستخدم كذلك
محظور؟ ماذا تفعل في مثل هذه الحالات؟

-
أنت تتلقى هذا لأنه تم ذكرك.
قم بالرد على هذا البريد الإلكتروني مباشرة ، وقم بعرضه على GitHub
https://github.com/publiclab/plots2/issues/8854#issuecomment-805774370 ،
أو إلغاء الاشتراك
https://github.com/notifications/unsubscribe-auth/AAAF6J4WSMFWF64K72GXR5TTFHJ6NANCNFSM4U4TWDIQ
.

جانبا ، يبدو أن الردود على التعليقات لا تظهر حاليًا في لوحة تحكم المشرف على الإشراف على الرسائل غير المرغوب فيها:

Screen Shot 2021-03-22 at 11 40 21 AM

وإذا نشر مستخدم cisco ردًا على تعليق ، فلن يجعله يظهر في "المستخدمون النشطون":

Screen Shot 2021-03-22 at 11 40 35 AM

مشكلة محتملة لمشروع Outreachy Spam moderation المذكور في # 9257؟

@ noi5e لا يمكنني إعادة إنتاج إصدار ردود التعليقات محليًا. لقد قمت بتسجيل الدخول كمستخدم جديد ، وقمت بالرد على التعليق ، وقمت بتسجيل الخروج ثم قمت بتسجيل الدخول كمسؤول ، ثم قمت بتمييز الرد على التعليق على أنه بريد عشوائي. ذهبت إلى صفحة الإشراف على الرسائل غير المرغوب فيها ووجدت الرد على التعليق تحت التعليقات.

image

image

هل هناك شيء مفقود؟

مرحبًا TildaDares ، أعتقد أنه تم إصلاح هذه المشكلة في # 9381

شكرا jcads

هل كانت هذه الصفحة مفيدة؟
0 / 5 - 0 التقييمات