Einladungs-Badges durch Selbsteinladung

UX
24

Es handelt sich tatsächlich um System-Badges, die nicht über die Benutzeroberfläche bearbeitet werden können (obwohl sie sichtbar sind). Diese Zeile müsste mit einem PR hinzugefügt werden. :+1:

3 „Gefällt mir“
27

Macht es Ihnen etwas aus, wenn ich versuche, diesen PR zu erstellen? Ich glaube, ich muss nur die SQL-Abfrage in discourse/lib/badge_queries.rb Zeilen 156-167 ändern.

3 „Gefällt mir“
30

Hallo. Nachdem meine Abzeichen vor einem Tag entfernt wurden, sind sie plötzlich wieder aufgetaucht. Kann jemand dieses Problem beheben?

1 „Gefällt mir“
31

Sie werden zurückkommen, da die Abfragen täglich ausgeführt werden und Sie sich immer noch „qualifizieren“. :slight_smile: Wir müssten sie jeden Tag manuell entfernen, bis der Fehler behoben ist, was unrealistisch ist.

Ich habe nachgesehen und wir können daraus einen pr-welcome machen :+1: (Sie müssten alle drei erledigen)

4 „Gefällt mir“
32

Verstanden, danke. Das sollte ich schaffen.

Und ich sehe nicht, warum ich 3. machen müsste. Die für Einladungslinks ist eine Funktion, die, wenn ich die Funktion richtig lese, die Anzahl zurückgibt.

3 „Gefällt mir“
33

Ich habe mir den Code nicht angesehen, daher könnten Sie Recht haben. :slightly_smiling_face:

3 „Gefällt mir“
34 
  def self.invite_badge(count, trust_level)
    ~SQL
      SELECT u.id user_id, current_timestamp granted_at
      FROM users u
      WHERE u.id IN (
        SELECT invited_by_id
        FROM invites i
        JOIN invited_users iu ON iu.invite_id = i.id
        JOIN users u2 ON u2.id = iu.user_id
        WHERE i.deleted_at IS NULL AND u2.active AND u2.trust_level >= #{trust_level.to_i} AND u2.silenced_till IS NULL
        GROUP BY invited_by_id
        HAVING COUNT(*) >= #{count.to_i}
      ) AND u.active AND u.silenced_till IS NULL AND u.id > 0 AND
        (:backfill OR u.id IN (:user_ids) )
    SQL
  end

Hier ist es. Es scheint, als wäre es nur die 1. Und ich habe die ganze Datei durchgesehen und nichts anderes mit Einladungen gefunden.

Habe den PR erstellt

5 „Gefällt mir“
35

Betrachtet man den PR (und bedenke, dass PRs nicht meine Stärke sind), sieht es so aus, als hättest du ihn nur für die Promoter-Version fest codiert, indem du #{count.to_i} zu 1 geändert hast?

Mein obiges Beispiel war nur ein Proof of Concept und müsste angepasst werden. :+1:

(Außerdem, wenn ich mir den Text in diesem Onebox ansehe, bin ich ‘JammyDodger’ und nicht ‘Jammy Dogger’… :slight_smile:)

8 „Gefällt mir“
36

Mein Fehler. Ich habe deinen Namen korrigiert.

4 „Gefällt mir“
39

Vielen Dank für deinen Beitrag! Ich habe deinen PR zusammengeführt.

Außerdem eine kleine Anmerkung: Wir benötigen normalerweise Tests für Änderungen, aber da vorher keine vorhanden waren, werden wir dies einfach zusammenführen und nachträglich ergänzen.

Ich hoffe, in Zukunft mehr von dir zu sehen :hugs:

8 „Gefällt mir“
40

Dieses Thema wurde nach 2 Tagen automatisch geschlossen. Neue Antworten sind nicht mehr möglich.