Gifs do Discourse

Ok, obrigado, uma pergunta: a versão mais recente do plugin é compatível com a Stable 2.8.1?

1 curtida

Não tenho certeza, pois não executamos versões estáveis e o plugin de chat ainda está em fase alfa e mudando rapidamente.

3 curtidas

Ok, entendi, muito obrigado.

1 curtida

@RGJ Pergunta, quando o chat rodar em estável, terei a versão mais recente do plugin no meu fórum?

1 curtida

Não, o plugin de chat ainda está em fase alfa e em rápida mudança, e a versão mais recente do plugin de chat não funciona no branch estável do Discourse, então o fixamos em uma versão anterior.

3 curtidas

Ah, perfeito, eu não tinha entendido bem a questão.

1 curtida

Para aqueles que optarem pelo Tenor e ainda não tiverem uma chave (v1), o componente não funcionará, pois utiliza a primeira versão da API, que é legada. As chaves entregues agora são v2 e os endpoints mudaram:

2 curtidas

O suporte ao Tenor foi adicionado por @merefield. Você planeja atualizá-lo para a versão 2.0?

2 curtidas

Oi Falco, não tenho certeza se você está falando do Robert ou de mim :sweat_smile: , pois faz muito mais sentido o Robert. Eu com certeza não estou planejando isso (provavelmente estou fora do meu alcance para fazer isso de forma limpa, mas ei, eu poderia tentar!)

2 curtidas

Este é um link melhor para descrever as alterações: How to migrate from Tenor's V1 API  |  Google for Developers

Receio que esteja no prazo de um projeto no momento, mas ficarei feliz em analisar isso em breve, se ainda estiver pendente.

3 curtidas
FAILED
--------------------
[2022-09-02T18:17:29.927471 #1]  INFO -- : cd /var/www/discourse & su discourse -c 'bundle exec rake themes:update assets:precompile'
ENOENT: no such file or directory, open '/var/www/discourse/plugins/discourse-gifs/plugin.rb'1

Existe algum problema conhecido com este componente no momento?

1 curtida

Olá @Alon1 - parece que você tentou instalar este componente de tema como um plugin no seu arquivo app.yml. Isso não funciona - você deve remover a linha do seu arquivo app.yml.

Mais contexto sobre por que isso só começou a causar erros pode ser encontrado em esta postagem

5 curtidas

Acabei de solicitar um PR com as alterações necessárias para o Tenor V2. As credenciais do Tenor V1 são compatíveis com o V2, então, esperançosamente, isso não será uma alteração que quebra a compatibilidade. Não tenho uma credencial V1 para testar, mas testei com V2 e funcionou muito bem para mim.

Meu forked branch está aberto para outros, mas por favor, use-o por sua conta e risco.

4 curtidas

Ótimo, obrigado! Respondi no Github.

2 curtidas

Na verdade, após testar isso também, acredito que a chave V1 não é compatível e, assim que isso for mesclado, os usuários precisarão registrar seu site como um projeto no Google Cloud Console e adicionar a API do Tenor, gerar uma chave e atualizar suas configurações:

Mas acho que isso não pode ser evitado e a migração para a nova API é uma ótima ideia.

2 curtidas

Vou verificar a adição de um alternador v1/v2 para evitar que isso seja uma alteração drástica.

3 curtidas

Embora isso ainda seja uma alteração disruptiva para cima assim que a API original for desativada? Eu obteria uma orientação da equipe sobre se isso é necessário. No entanto, isso dará mais tempo às pessoas. Vamos discutir no PR.

2 curtidas

Obrigado pelo PR @tnoor-co!

Pensando nisso, como as chaves de API do Tenor V1 são impossíveis de obter agora, mesmo mantendo este componente de tema, a parte do Tenor V1 se tornará um incômodo, pois um desenvolvedor precisa de uma API funcional para testá-la localmente.

Com isso em mente, eu diria que removermos o Tenor V1 do componente agora. Administradores que usam o Tenor como @PaulinaMX podem segurar a atualização e obter uma chave de API V2 antes de atualizar. Também podemos manter um branch tenor-v1 por um tempo para pessoas que precisarem.

O que você acha @merefield?

5 curtidas

Concordo.

Embora tenhamos em mente que @tnoor-co colocou um esforço enorme aqui (e temo pedir mais a ele), minha intuição me diz que deveríamos arriscar e remover o código V1 inteiramente para manter a simplicidade e clareza das configurações e do código no branch principal.

Parece apenas uma grande complexidade para evitar o inevitável :slight_smile:

4 curtidas

Parece bom para mim. Vou enviar uma atualização esta semana para remover completamente a v1.

5 curtidas