Links de convite (botão Aceitar Convite) com SSO não funcionando para a versão estável v2.8.10

Sim, agora vejo o problema aqui, ainda estamos mostrando o botão Discourse Connect mesmo que o convite já tenha sido resgatado, e também para um usuário existente que não resgatou o convite.

Convite ainda não resgatado para usuário existente com DiscourseConnect ativado:

Convite já resgatado para usuário existente com DiscourseConnect ativado:

Eu acho que o correto é ocultar o botão “Aceitar Convite” no primeiro caso e o botão “Continuar” no segundo caso, embora eu não tenha certeza, já que passar pelo caminho de login SSO também aceita o convite discourse/app/controllers/session_controller.rb at 13db6234940e50f66bc712a054003e362ae9dbcd · discourse/discourse · GitHub. @sam você tem alguma opinião sobre isso? Eu só estou um pouco relutante em fazer mais alterações nesta interface/sistema no momento…

1 curtida

A mudança na UI parece bem direta, sei o quão irritante isso é, mas se for apenas ocultar um botão condicionalmente, recomendo fazê-lo, mas sem backport.

1 curtida

O problema aqui para o @hyphalos é que eles estão usando a versão estável, então, neste caso, teríamos que fazer um backport da correção.

Desculpe, acho que @hyphalos pode simplesmente usar CSS aqui para contornar, não? Ocultar os elementos que são um problema? Eu preferiria não fazer backport de nada, estamos muito perto do lançamento 3.0.

1 curtida

Isso não é realmente viável, pois um botão diferente precisa ser ocultado dependendo do estado:

No entanto, se for muito arriscado antes do 3.0, não há muito que possamos fazer aqui, farei a correção em main e ela será repassada para a versão estável no próximo ano.

2 curtidas

Estamos prestes a lançar a versão 3.1, isso não é mais relevante.