martin
(Martin Brennan)
25
はい、ここで問題がわかりました。招待がすでに引き換えられている場合でも、また招待を引き換えていない既存のユーザーに対しても、Discourse Connect ボタンが表示されています。
DiscourseConnect が有効な既存ユーザーの招待がまだ引き換えられていない場合:
DiscourseConnect が有効な既存ユーザーの招待がすでに引き換えられている場合:
最初のケースでは「Accept Invitation」ボタンを、2番目のケースでは「Continue」ボタンを非表示にするのが正しいと思いますが、SSO ログイン経由で招待が受け入れられることもあるため、確信が持てません。discourse/app/controllers/session_controller.rb at 13db6234940e50f66bc712a054003e362ae9dbcd · discourse/discourse · GitHub 。@sam さん、何かご意見はありますか?現時点では、この UI/システムへの変更はあまり行いたくありません…
「いいね!」 1
sam
(Sam Saffron)
26
UIの変更は非常に単純に見えます。これがどれほど煩わしいかは承知していますが、条件付きでボタンを非表示にするだけであれば、バックポートせずに実装することをお勧めします。
「いいね!」 1
martin
(Martin Brennan)
27
@hyphalos の問題は、stable を使用していることです。この場合、修正をバックポートする必要があります。
sam
(Sam Saffron)
28
申し訳ありませんが、@hyphalos はここで CSS を使用して、問題のある要素を非表示にするだけで回避できるのではないでしょうか? 3.0 リリースに近すぎるため、何かをバックポートすることは避けたいです。
「いいね!」 1
martin
(Martin Brennan)
29
これは実際には実行可能ではありません。なぜなら、状態に応じて非表示にする必要があるボタンが異なるからです。
しかし、3.0より前ではリスクが高すぎる場合、ここでできることはあまりありません。mainで修正を行い、新年にはstableにも反映されるようにします。
「いいね!」 2
martin
(Martin Brennan)
クローズされました:
30
3.1をリリースする予定ですので、これはもはや関連性がありません。