Plots2: Antworten auf Kommentare ausblenden, die gespammt oder noch nicht moderiert wurden

Das scheint etwas in meinem Steuerhaus! Ich kann es in einem Moment, in dem ich meiner Arbeit voraus bin, in Unterthemen aufteilen.

Ich habe dies lokal getestet, um es auf ein Ftos zu brechen. Es sieht so aus, als ob das Problem bereits behoben ist. Könnte jemand auch bestätigen, dass dies auf seiner Seite der Fall ist ... @noi5e vielleicht haben Sie es irgendwo in Ihrem Projekt gelöst.

@cesswairimu Hmm !! Ich glaube, jemand anderes hat es getan

@cesswairimu Könnte jemand einen Test dazu schreiben, wenn ich nicht dazu komme? Das ist ein bisschen zu kompliziert für einen FTO, aber ich denke, dass das Schreiben von Tests den Bewerbern viel beibringen kann. Vielleicht sollten wir dieses Thema auch nicht schließen, bis ein Test geschrieben ist

@noi5e Ich stimme zu, ich werde dies aktualisieren, um eher ein Problem beim Schreiben von Tests zu sein. Vielen Dank

Hmm, ich bin mir nicht sicher - vielleicht kann der Test zuerst kommen, um zu bestätigen, ob es wirklich behoben ist?

Da wir nur nach Verwendung filtern können, nicht in der has_many Definition, wie es scheint, und ich sehe hier kein .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

Was denken Sie?

Danke, alle!!! 💯

ah und ich denke bei diesen includes() müsste es einen Namen haben, also vielleicht... .where('comments.status != 0') ?

Ja, ich denke, das ist richtig, siehe Verwendung hier:

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

@jywarren @cesswairimu @noi5e Darf ich daran arbeiten, Tests zu schreiben?

@17sushmita Ja, auf jeden Fall, bitte

Antworten auf Kommentare werden anscheinend nicht nach Spam gefiltert. Wir sollten diese Nachricht sehen:

"Bist du sicher? Der Benutzer kann sich nicht mehr anmelden oder veröffentlichen und seine Inhalte werden außer Kommentaren ausgeblendet.“ scheint falsch zu sein

@jywarren Was bedeutet das, wenn man sagt, dass diese Nachricht falsch erscheint? Soll es nicht angezeigt werden, wenn der Moderator versucht, den Kommentar als Spam zu markieren? Geben Sie ihm bitte etwas mehr Kontext.

@17sushmita Das verwirrt mich auch ein wenig. Ich denke, Jeffrey meinte vielleicht, dass der Text im Bild unten in Gelb angezeigt wurde, anstatt in Moderate first-time comment ? Wenn ja, scheint das falsch zu sein.

Während wir auf die Klärung warten, können Sie noch einen Test schreiben. Ich werde hier nur einige Vorschläge posten, die hilfreich sein könnten.

Zunächst können Sie einen Spam-/Erstmaligen-Poster-Kommentar in die Testdatenbank einfügen, der eine Antwort auf einen anderen Kommentar ist:

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)
})

Dann schreiben Sie den Rest des Tests von dort aus und stellen Sie sicher, dass die Methode, die wir zum Abrufen von Kommentaren verwenden, keine Kommentarantworten mit dem Status 0 oder 4 erhält.

Sowas in der Art! Du wirst wahrscheinlich etwas recherchieren müssen. Hinterlassen Sie hier auf jeden Fall einen Kommentar, wenn Sie nicht weiterkommen oder Hinweise benötigen, wo Sie in der Codebasis suchen können. Wir sind hier um zu helfen!

@noi5e , vielen Dank für deine Hilfe 😃️!! Ich brauche noch eine Klarstellung. Muss ich eine neue Datei für Kommentarantworttests erstellen oder Änderungen in vielleicht /plots2/test/functional/admin_controller_test.rb oder /plots2/test/system/spam2_test.rb oder /plots2/test/functional/comment_controller_test.rb vornehmen oder eine andere Datei?

@17sushmita Ich bin mir da nicht 100% sicher, aber ich denke /test/functional/comment_controller_test.rb . Der andere Ort, an den ich dachte, war ein Komponententest für Kommentare. Gedanken von jemand anderem?

In der Zwischenzeit können Sie gerne damit beginnen, während wir darauf warten, dass andere sich einmischen.

@17sushmita Ich bin mir da nicht 100% sicher, aber ich denke /test/functional/comment_controller_test.rb . Der andere Ort, an den ich dachte, war ein Komponententest für Kommentare. Gedanken von jemand anderem?

In der Zwischenzeit können Sie gerne damit beginnen, während wir darauf warten, dass andere sich einmischen.

Danke, ich habe mit dem Schreiben der Tests begonnen, aber eine Sache, die ich beobachtet habe, ist, dass der Kommentar selbst nach dem Markieren des Kommentars als Spam angezeigt wird und laut Software korrekt ist, da er beim Markieren als Spam eine Warnung generiert, wie von Ausnahme von Kommentaren ausgeblendet." aber ist es wirklich angemessen, als Spam markierte Kommentare anzuzeigen?🤔️

Ist es wirklich angebracht, als Spam markierte Kommentare anzuzeigen?🤔️

Ich glaube nicht!

Dieser Text, den Sie erwähnen, ist ziemlich interessant:

"Sind Sie sicher? Der Benutzer kann sich nicht mehr anmelden oder veröffentlichen und sein Inhalt wird außer Kommentaren ausgeblendet

Es lässt mich denken, dass wir vielleicht früher die Kommentare nicht moderiert haben. Ich denke, dieser Text ist einfach veraltet.

Wie auch immer, ich denke, wir sind uns alle einig, dass wir auf jeden Fall alle Kommentare filtern sollten! So können wir die Spambots vermeiden, die @jywarren ursprünglich erwähnt hat .

Ich habe das gerade lokal getestet und für mich sieht es so aus, als ob das nicht behoben wurde. Wir filtern derzeit keinen Spam oder erstmalig moderierte Kommentare ANTWORTEN ... Bitte beachten Sie, dass regelmäßige Kommentare (die keine ANTWORTEN auf andere Kommentare sind) gefiltert werden, daher gilt dies nur für Antworten. (Ich habe auf Kommentare getestet, die KEINE Antworten sind, und diese werden moderiert)

Werde das noch etwas weiter aufschlüsseln. Vor Ort habe ich ein neues Konto erstellt und einen Kommentar zu einer Notiz gepostet. Das sehe ich, wenn ich einen Kommentar poste. So weit, ist es gut:

Als Benutzer lässt mich die Meldung _"Genehmigung durch Community-Moderatoren ausstehend"_ vermuten, dass mein Kommentar nicht öffentlich angezeigt wird.

Allerdings ist der Kommentar öffentlich erscheinen! Wenn ich dieselbe Notiz in einem Inkognito-Fenster aufrufe, das von der Site abgemeldet ist, wird der Kommentar genau so angezeigt, wie er dem Benutzer _cisco_ angezeigt wurde:

Das Sperren des Benutzers _cisco_ scheint auch nicht viel zu ändern. Die Meldung _"Ausstehende Genehmigung"_ wird ausgeblendet, aber der Kommentar ist weiterhin in einem Inkognito-Fenster sichtbar:

Also ja, das muss sich definitiv ändern.

@17sushmita Mach weiter und arbeite weiter daran!

Wenn Sie es noch nicht gefunden haben, haben wir einen gesperrten _spammer_-Benutzer, der bereits in /test/fixtures/users.yml

@noi5e Das war eine sehr klare Erklärung. Ich habe auch sehr ähnlich auf meinem lokalen System getestet und die Ergebnisse waren die gleichen, was mich dazu veranlasste. Also werde ich zuerst daran arbeiten, die Spam-Kommentare zu entfernen und dann einen Test schreiben, ob es behoben ist oder nicht.

Abgesehen davon sieht es so aus, als würden Kommentarantworten derzeit nicht im Spam-Moderations-Dashboard des Administrators angezeigt:

Und wenn Benutzer cisco eine Kommentarantwort postet, werden sie nicht in den aktiven Benutzern angezeigt:

Potenzielles Problem für das in #9257 erwähnte Outreachy-Spam-Moderationsprojekt?

Danke @17sushmita und @noi5e für das so gründliche Auspacken. Ich entschuldige mich für den unklaren Hinweis in meiner ursprünglichen Nachricht, aber @noi5e ist völlig korrekt - wir brauchen nur die einfachere Nachricht "Mäßiger

Filtern von Spam oder erstmalig moderierten Kommentaren ANTWORTEN

Und ein toller Fang, der für ANTWORTEN noch ungelöst ist.

Ich danke Ihnen beiden für Ihre Hilfe. 🎉

Ich werde das Spam-Dashboard-Problem in der neuen Projektbeschreibung beachten!!!

Hallo @jywarren @noi5e , Entschuldigung für die nochmalige Nachfrage, aber ich hatte noch eine Verwirrung. Sollen wir einen Kommentar anzeigen, der kein Spam ist, aber der Benutzer gesperrt ist? Was tun in solchen Fällen?

Ja, derzeit hat der Status gesperrter Benutzer keinen Einfluss auf die Sichtbarkeit ihrer
Kommentare. Dankeschön!

Am Mi, 24. März 2021, 08:17 Sushmita @ . * > schrieb:

Hallo @jywarren https://github.com/jywarren @noi5e
https://github.com/noi5e , Entschuldigung für die nochmalige Nachfrage, aber ich hatte noch einen
Verwechslung. Sollten wir einen Kommentar anzeigen, der kein Spam ist, aber der Benutzer ist es?
verboten? Was tun in solchen Fällen?

—
Sie erhalten dies, weil Sie erwähnt wurden.
Antworten Sie direkt auf diese E-Mail und zeigen Sie sie auf GitHub an
https://github.com/publiclab/plots2/issues/8854#issuecomment-805774370 ,
oder abmelden
https://github.com/notifications/unsubscribe-auth/AAAF6J4WSMFWF64K72GXR5TTFHJ6NANCNFSM4U4TWDIQ
.

Abgesehen davon sieht es so aus, als würden Kommentarantworten derzeit nicht im Spam-Moderations-Dashboard des Administrators angezeigt:

Und wenn Benutzer cisco eine Kommentarantwort postet, werden sie nicht in den aktiven Benutzern angezeigt:

Potenzielles Problem für das in #9257 erwähnte Outreachy-Spam-Moderationsprojekt?

@noi5e Ich kann das Problem mit den Kommentarantworten lokal nicht reproduzieren. Ich habe mich als neuer Benutzer angemeldet, eine Kommentarantwort erstellt, mich abgemeldet und dann als Admin angemeldet und dann die Kommentarantwort als Spam markiert. Ich ging auf die Spam-Moderationsseite und fand die Kommentarantwort unter Kommentare.

Gibt es etwas, das ich vermisse?

Hallo @TildaDares , ich glaube, dieses Problem wurde in #9381 behoben

Danke @jcads