Invita badge invitando te stesso

UX
24

Sono effettivamente badge di sistema e non possono essere modificati dall’interfaccia utente (anche se sono visibili). Quella riga dovrebbe essere aggiunta con una PR. :+1:

3 Mi Piace
27

Ti dispiace se provo a fare quella PR. Penso che debba solo cambiare l’SQL in discourse/lib/badge_queries.rb alle righe 156-167

3 Mi Piace
30

Ciao. Dopo che i miei badge sono stati rimossi un giorno fa, sono improvvisamente ricomparsi. Qualcuno può risolvere questo problema?

1 Mi Piace
31

Torneranno poiché le query vengono eseguite quotidianamente e tu ti qualifichi ancora. :slight_smile: Dovremmo rimuoverli manualmente ogni giorno finché il bug non verrà corretto, il che non è realistico.

Ho controllato e possiamo renderlo pr-welcome :+1: (Dovresti fare tutte e tre le cose)

4 Mi Piace
32

Capito, grazie. Dovrei essere in grado di farlo

E non vedo perché dovrei fare la 3. Quella per i link di invito è una funzione che restituisce il conteggio, se ho letto correttamente la funzione.

3 Mi Piace
33

Non ho guardato il codice, quindi potresti avere ragione. :slightly_smiling_face:

3 Mi Piace
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

Ecco qui. Sembra che sia solo l’1. E ho esaminato l’intero file e non c’è nient’altro con gli inviti

Ho creato la PR

5 Mi Piace
35

Guardando il PR (e tenendo presente che i PR non sono il mio forte), sembra che tu l’abbia codificato in modo fisso solo per la versione Promoter cambiando #{count.to_i} in 1?

Il mio esempio sopra era solo una prova di concetto e dovrebbe essere adattato. :+1:

(inoltre, guardando il testo in quel onebox, sono ‘JammyDodger’ non ‘Jammy Dogger’… :slight_smile:)

8 Mi Piace
36

Mi dispiace. Ho corretto il tuo nome

4 Mi Piace
39

Grazie per il tuo contributo! Ho unito la tua PR

Inoltre, una piccolissima nota: solitamente richiediamo test per le modifiche, ma visto che prima non ce n’erano, uniremo questa e la riempiremo in seguito.

Spero di vederti ancora in futuro :hugs:

8 Mi Piace
40

Questo argomento è stato chiuso automaticamente dopo 2 giorni. Non sono più consentite nuove risposte.