"Concedido por" em emblemas não exibe o nome do staff

Olá a todos.

Versão do Discourse 3.2.0.beta2-dev (1a70817962)

Ao emitir manualmente emblemas para nossos membros através das páginas de administração, o nome do staff “Concedido por” não está sendo exibido:

Possivelmente relacionado a: Grant badge reason not added to the granted badge :thinking:

3 curtidas

Olá :wave:

Se eu entendi corretamente, ele registra apenas os system_users aqui. Se um membro da equipe conceder um distintivo a um usuário, ele será colocado nos Registros de Ações da Equipe. /admin/logs/staff_action_logs

No entanto, acho que seria um ótimo recurso adicionar isso aos distintivos concedidos manualmente pela equipe.

3 curtidas

Ele também mostra todos os registros criados por funcionários. Você está certo sobre o log.

Pelo que vejo, o template está pronto para exibir o nome do concedente, mas o código espera que a carga útil (criação e listagem) inclua uma lista de users para que o nome possa ser descoberto com base no ID do concedente. Acho que isso nunca foi implementado. :thinking:

4 curtidas

Este commit é provavelmente relevante, eu acho :thinking: Infelizmente, a discussão do tópico meta sobre ele não está mais disponível.

4 curtidas

Não é este? :thinking:

1 curtida

Eu o ressuscitei depois de ler o comentário do Don esta manhã. :zombie: :slight_smile:

5 curtidas

Ahhhh, eu pensei que tinha ganhado privilégios extras ou algo assim :rofl:

3 curtidas

Obrigado por relatar isso, corrigido em FIX: display and link to correct grant_by users for granted badges by tyb-talks · Pull Request #24021 · discourse/discourse · GitHub :slight_smile:

3 curtidas

Brilhante, obrigado @kelv :smiley:

2 curtidas

@kelv, com base na mensagem de commit FIX, isso foi realmente um bug?

Em vez de algo que não foi implementado?

Nunca consegui encontrar um commit que quebrasse o sistema, então não tenho certeza se foi um recurso planejado que nunca foi implementado ou um bug :sweat_smile: De qualquer forma, pensei que a interface do usuário parecia ter a intenção explícita de ter isso (pelo cabeçalho da coluna Concedido por), então optei por ir com CORREÇÃO.

2 curtidas

Bem, a correção funciona muito bem, obrigado novamente :smiley:

3 curtidas

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