Insignias de invitación por invitarte a ti mismo

UX
24

Son efectivamente insignias del sistema y no se pueden editar desde la interfaz de usuario (aunque son visibles). Esa línea tendría que añadirse con una PR. :+1:

3 Me gusta
27

¿Te importa si intento hacer esa PR? Creo que solo necesito cambiar el SQL en discourse/lib/badge_queries.rb líneas 156-167.

3 Me gusta
30

Hola. Después de que me quitaran mis insignias hace un día, de repente volvieron. ¿Alguien puede solucionar este problema?

1 me gusta
31

Volverán ya que las consultas se ejecutan diariamente y sigues ‘calificando’. :slight_smile: Tendríamos que eliminarlos manualmente todos los días hasta que se solucione el error, lo cual no es realista.

He comprobado y podemos hacer que esto sea pr-welcome :+1: (Tendrías que hacer las tres).

4 Me gusta
32

Entendido, gracias. Debería poder hacerlo

Y no veo por qué necesitaría hacer 3. El de los enlaces de invitación es una función que devuelve el recuento si leí la función correctamente.

3 Me gusta
33

No miré el código, así que puede que tengas razón. :slightly_smiling_face:

3 Me gusta
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

Aquí está. Parece que es solo el 1. Y revisé todo el archivo y no hay nada más con invitaciones.

Hice el PR

5 Me gusta
35

Mirando la PR (y ten en cuenta que las PR no son mi fuerte), parece que la has codificado de forma rígida para que sea solo la versión del Promotor al cambiar #{count.to_i} por 1?

Mi ejemplo anterior fue solo una prueba de concepto y necesitaría ser adaptado. :+1:

(además, mirando el texto en ese onebox, soy ‘JammyDodger’ no ‘Jammy Dogger’… :slight_smile:)

8 Me gusta
36

Lo siento. Corregí tu nombre

4 Me gusta
39

¡Gracias por tu contribución! He fusionado tu PR

Además, una pequeña nota: normalmente requerimos pruebas para las modificaciones, pero al ver que no había ninguna antes, simplemente fusionaremos esto y lo rellenaremos.

Espero ver más de ti en el futuro :hugs:

8 Me gusta
40

Este tema se cerró automáticamente después de 2 días. Ya no se permiten nuevas respuestas.