Feedback zur neuen Review Queue (2019)

Ein Vorschlag für das Bewertungssystem: Wenn Sie einen Beitrag genehmigen, der einen Nutzer wieder aktiviert, könnten Sie dafür eine Benutzernotiz erstellen? Etwas wie '@username hat dieses Konto wieder aktiviert' wäre sehr nützlich. Derzeit sehen wir in den Notizen nur die Hälfte der Geschichte.

3 „Gefällt mir“

Besser ist es, diese einfach komplett zu löschen – das mache ich auch. Meistens sind sie ohnehin fehlerhaft, und ich mag keine lauten Notizen, die keinen Mehrwert bieten.

2 „Gefällt mir“

Aus der Warteschlange für Überprüfungen: Bei dem Typ ‘Gesperrter Beitrag’ erhalte ich beim Versuch, einen Benutzer mit vielen Beiträgen zu löschen, einen 502-Timeout.

Ich bin mir nicht sicher, was die Obergrenze ist, aber bei heutigen Tests war das Konto mit 288 Beiträgen dasjenige mit der geringsten Anzahl, das nicht funktionierte.

Beispiel: Ein Beitrag wurde aufgrund eines Wortes auf der Liste ‘Beobachtete Wörter → Genehmigung erforderlich’ als ‘Gesperrter Beitrag’ gemeldet.

Derzeit stehen folgende Optionen zur Verfügung:
Beitrag genehmigen | Beitrag ablehnen | Benutzer löschen | Bearbeiten

Ich glaube, dass die Ergänzung um die Optionen ‘Schweigen’ und ‘Sperren’ für diese Typen von gesperrten Beiträgen sehr hilfreich wäre. Beispiel: Beitrag ablehnen und Benutzer schweigen oder sperren. Dies würde Administratoren die Wahl geben, einen Benutzer zu schweigen oder zu sperren oder ihn direkt aus der Warteschlange für Überprüfungen vollständig aus der Historie zu entfernen.

Außerdem wäre es sehr gut, wenn das Löschen von Benutzern mit mehr als x Beiträgen aus der Warteschlange für Überprüfungen aufgrund von 502-Fehlern nicht möglich ist, die Optionen ‘Sperren’ und ‘Schweigen’ als Alternativen zur Verfügung stehen.

3 „Gefällt mir“

Ein paar weitere Informationen:

Beim Öffnen von „Nach Thema gruppiert“ aus der Warteschlange erhalte ich diesen Fehler:

Serverfehler
Beim Laden von /review/topics
Fehlercode: 500 Internal Server Error

Hinweis: In der Warteschlange befinden sich etwa 30.000 Einträge. Viele der älteren Einträge wurden von Akismet hinzugefügt, bevor ich es deinstalliert habe.

Problem beim Scrollen/Paginierung (hätte ich wahrscheinlich besser hier gepostet): Review Queue Pagination/Infinite Scrolling after Taking an Action

Bezüglich der Einträge vom Typ „Wartende Beiträge“ und dem 502-Timeout beim Verwenden der Option „Benutzer löschen“. Ich kann den Fehler mit einem Konto mit 166 Beiträgen bestätigen.

Ideen:

  1. Es wäre eine nützliche Zeitersparnis, wenn es in der Warteschlange einen direkten Link zur Admin-Seite des Benutzers gäbe.

  2. Ich glaube, aktuell ist es nicht möglich, sich von der täglichen Erinnerungsmail „x Einträge müssen geprüft werden“ abzumelden. Es wäre hilfreich, wenn man sich abmelden könnte.

2 „Gefällt mir“

Könntest du deine /logs überprüfen und uns mitteilen, was der Fehler ist?

3 „Gefällt mir“

Ok, ich denke, das ist es:

ActiveRecord::SubclassNotFound (Der Mechanismus für Single-Table-Inheritance konnte die Unterklasse ‘ReviewableAkismetPost’ nicht finden. Dieser Fehler wird ausgelöst, weil die Spalte ‘type’ für die Speicherung der Klasse im Fall von Vererbung reserviert ist. Bitte benennen Sie diese Spalte um, wenn Sie nicht beabsichtigt haben, sie für die Speicherung der Vererbungsklasse zu verwenden, oder überschreiben Sie Reviewable.inheritance_column, um eine andere Spalte für diese Information zu verwenden.)
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activerecord-6.0.3/lib/active_record/inheritance.rb:234:in `rescue in find_sti_class’

2 „Gefällt mir“

Können Sie bestätigen, ob Ihr Akismet-Plugin die neueste Version ist – und falls nicht, aktualisieren Sie es?

4 „Gefällt mir“

Vielleicht können alte Elemente, die von anderen Bewertungsarten hinzugefügt wurden, nicht gelesen werden, wenn die Definition des zu Bewertenden verschwindet (z. B. durch Deinstallation eines Plugins). Die Fehler scheinen nach der Deinstallation aufgetreten zu sein:

4 „Gefällt mir“

Ich kann bestätigen, dass Akismet derzeit deinstalliert ist; ich habe es vor einiger Zeit entfernt.

1 „Gefällt mir“

Oh, das ist interessant, wie @featheredtoast vermutet. @Roman, wie denkst du, sollten wir damit umgehen, wenn die Datensätze vorhanden sind, aber das Plugin entfernt wurde?

4 „Gefällt mir“

Ich denke, es ist möglich, festzustellen, welche überprüfbaren Typen herausgefiltert werden müssen, indem man so etwas wie Folgendes macht:

class Reviewable < ActiveRecord::Base
  def self.exclude_types
     db_types = Reviewable.distinct.pluck(:type)

     @exclude_types ||= db_types - Reviewable.types
  end
  
...
end

Anschließend können wir diese Typen verwenden, um einen Standardbereich (default scope) anzuwenden. Wir werden wahrscheinlich einen type-Index zur Tabelle hinzufügen müssen.

5 „Gefällt mir“

@Roman, könntest du das übernehmen, wenn du Zeit hast?

5 „Gefällt mir“

Es tauchen in der Warteschlange für Bewertungen viele unsichtbare Bilder auf. Manche funktionieren einwandfrei, das Verhältnis liegt etwa bei 50/50. Bei der Inspektion wird bei einigen etwas wie Folgendes angezeigt, aber nichts wird dargestellt:

src="/images/transparent.png" alt="" data-orig-src="upload://fwf1zrfwefWEqGer2W3xz1ed.jpeg"

Das tritt sowohl bei Instanzen mit CDN + S3 als auch bei reinem lokalem Speicher auf.

1 „Gefällt mir“

Ja, das Problem betrifft nur Warteschlangen-Beiträge.

Ich habe einen PR mit einer Korrektur, der zur Prüfung aussteht, sodass Bilder bald wieder angezeigt werden sollten.

Ich gebe dir Bescheid, sobald wir ihn gemergt haben.

4 „Gefällt mir“

Die Korrektur ist jetzt auf tests-passed und stable verfügbar.

Es gibt jedoch noch ein weiteres Problem: Abgelehnte Bilder aus Warteschlangenbeiträgen werden in der Prüfwarteschlange weiterhin nicht angezeigt. Das System entfernt sie automatisch, da es nicht notwendig ist, sie zu behalten. Wir planen, sie durch einen Text zu ersetzen, der dies erklärt.

8 „Gefällt mir“

Vielen Dank, dass du das behoben hast, @Roman!

Etwas anderes, das möglicherweise ein Fehler bei tests-passed ist: Szenario: Ein Beitrag in der Warteschlange für die Prüfung wird angenommen, dann geht man zurück und lehnt ihn ab. Der Beitrag bleibt aufgeführt und auf der Website sichtbar.

Edit: Die beiden unteren Absätze dieses Kommentars erklären auch ein weiteres mögliches Problem mit einigen Optionen der Warteschlange für die Prüfung und plattformweiten Geschwindigkeitsbegrenzungen: Discourse No Bump - #27

1 „Gefällt mir“

Etwas anderes, das mir beim ‘auto handle queued age’ aufgefallen ist: Ich habe viele alte Einträge in einigen Review-Warteschlangen, die deutlich älter sind als die Einstellung ‘auto handle queued age’ (Standardwerte), die jedoch nicht automatisch bearbeitet zu werden scheinen. Es sieht so aus, als würden überhaupt keine Einträge automatisch bearbeitet. Ich bin mir nicht sicher, ob ich etwas übersehe.

Außerdem erhalte ich beim Sortieren der Review-Warteschlange nach ‘Created At (reverse)’ einen 500-Fehler. Alle anderen ‘order by’-Filter funktionieren einwandfrei.

1 „Gefällt mir“

Könntest du bitte deine Protokolle überprüfen und uns mitteilen, welcher Fehler auftritt, wenn du die Sortierreihenfolge änderst?

3 „Gefällt mir“

Danke @eviltrout, ja klar. Das ist der Fehler, den ich sehe:

ActiveRecord::SubclassNotFound (Der Mechanismus für die Vererbung in einer einzelnen Tabelle konnte die Unterklasse ‘ReviewableAkismetPost’ nicht finden. Dieser Fehler wird ausgelöst, weil die Spalte ‘type’ für die Speicherung der Klasse im Falle einer Vererbung reserviert ist. Bitte benennen Sie diese Spalte um, wenn Sie nicht beabsichtigt haben, sie zur Speicherung der Vererbungsklasse zu verwenden, oder überschreiben Sie Reviewable.inheritance_column, um eine andere Spalte für diese Information zu verwenden.)
/var/www/discourse/vendor/bundle/ruby/2.6.0/gems/activerecord-6.0.3.1/lib/active_record/inheritance.rb:234:in `rescue in find_sti_class’

Beachten Sie, dass das Akismet-Plugin auf diesem speziellen Forum schon vor geraumer Zeit entfernt wurde.

2 „Gefällt mir“

Ah, es hängt also immer noch damit zusammen. @Roman, es scheint hier immer noch einen Fehler zu geben, der mit den alten überprüfbaren Typen in der Datenbank zusammenhängt?

4 „Gefällt mir“