Sì, ora vedo il problema, stiamo ancora mostrando il pulsante Discourse Connect anche se l’invito è già stato riscattato, e anche per un utente esistente che non ha riscattato l’invito.
Invito non ancora riscattato per utente esistente con DiscourseConnect abilitato:
Penso che la cosa corretta sia nascondere il pulsante “Accetta invito” nel primo caso e il pulsante “Continua” nel secondo caso, anche se non sono del tutto sicuro, dato che passare attraverso il percorso di accesso SSO accetta anche l’invito discourse/app/controllers/session_controller.rb at 13db6234940e50f66bc712a054003e362ae9dbcd · discourse/discourse · GitHub . @sam hai qualche idea al riguardo? Sono solo un po’ riluttante a apportare ulteriori modifiche a questa interfaccia utente/sistema al momento…
La modifica dell’interfaccia utente sembra piuttosto semplice, so quanto possa essere fastidioso, ma se si tratta solo di nascondere un pulsante in modo condizionale, consiglio di farlo, ma senza effettuare il backporting.
Mi dispiace molto, penso che @hyphalos possa semplicemente usare il CSS qui per aggirare il problema, no? Nascondere gli elementi che sono un problema? Preferirei non fare un backport di nulla, siamo troppo vicini al rilascio della 3.0.
Questo non è realmente fattibile poiché è necessario nascondere un pulsante diverso a seconda dello stato:
Tuttavia, se è troppo rischioso prima della 3.0, non c’è molto che possiamo fare qui, farò solo la correzione in main e si ripercuoterà sullo stabile nel nuovo anno.
