Convidar emblemas ao convidar a si mesmo

Experiência do Usuário
24

Eles são, de fato, distintivos do sistema e não podem ser editados pela interface do usuário (embora sejam visíveis). Essa linha precisaria ser adicionada com um PR. :+1:

3 curtidas
27

Se importaria se eu tentasse fazer esse PR. Acho que só preciso mudar o SQL em discourse/lib/badge_queries.rb linhas 156-167

3 curtidas
30

Olá. Depois que meus emblemas foram removidos um dia atrás, eles voltaram de repente. Alguém pode corrigir esse problema?

1 curtida
31

Eles voltarão, pois as consultas são executadas diariamente e você ainda se ‘qualifica’. :slight_smile: Teríamos que removê-los manualmente todos os dias até que o bug seja corrigido, o que é irrealista.

Verifiquei e podemos fazer isso pr-welcome :+1: (Você precisaria fazer os três)

4 curtidas
32

Entendido, obrigado. Eu devo ser capaz de fazer isso

E não vejo por que eu precisaria fazer 3. A de links de convite é uma função que retorna a contagem se eu li a função corretamente.

3 curtidas
33

Eu não olhei o código, então você pode estar certo. :slightly_smiling_face:

3 curtidas
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

Aqui está. Parece que é apenas o 1. E procurei em todo o arquivo e nada mais com convites

Fiz o PR

5 curtidas
35

Olhando o PR (e lembrando que PRs não são meu forte), parece que você codificou diretamente para ser apenas a versão Promoter, alterando o #{count.to_i} para 1?

Meu exemplo acima foi apenas uma prova de conceito e precisaria ser adaptado. :+1:

(Além disso, olhando o texto naquele onebox, eu sou ‘JammyDodger’ e não ‘Jammy Dogger’… :slight_smile:)

8 curtidas
36

Desculpe. Corrigi seu nome

4 curtidas
39

Obrigado pela sua contribuição! Mesclei seu PR

Além disso, uma observação muito pequena é que normalmente exigimos testes para modificações, mas como não havia nenhum antes, vamos apenas mesclar isso e preencher depois.

Espero ver mais de você no futuro :hugs:

8 curtidas
40

Este tópico foi fechado automaticamente após 2 dias. Novas respostas não são mais permitidas.