Oui, je vois maintenant le problème, nous affichons toujours le bouton Discourse Connect même si l’invitation a déjà été utilisée, ainsi que pour un utilisateur existant qui n’a pas utilisé l’invitation.
Invitation non encore utilisée pour un utilisateur existant avec DiscourseConnect activé :
Je pense que la bonne chose à faire est de masquer le bouton « Accepter l’invitation » dans le premier cas et le bouton « Continuer » dans le second cas, bien que je ne sois pas entièrement sûr, car passer par le chemin de connexion SSO accepte également l’invitation discourse/app/controllers/session_controller.rb at 13db6234940e50f66bc712a054003e362ae9dbcd · discourse/discourse · GitHub . @sam, avez-vous des réflexions à ce sujet ? Je suis juste un peu réticent à apporter d’autres modifications à cette interface utilisateur/système pour le moment…
Le changement d’interface utilisateur semble assez simple. Je sais à quel point c’est ennuyeux, mais s’il s’agit simplement de masquer un bouton conditionnellement, je recommande de le faire, mais sans rétroportage.
Désolé, je pense que @hyphalos peut simplement utiliser CSS ici pour contourner le problème, non ? Cacher les éléments qui posent problème ? Je préférerais ne pas faire de rétroportation, nous sommes trop proches de la sortie de la version 3.0.
Ce n’est pas vraiment réalisable car un bouton différent doit être caché en fonction de l’état :
Cependant, si c’est trop risqué avant la 3.0, alors nous ne pouvons pas faire grand-chose ici, je vais simplement apporter la correction dans main et elle se répercutera sur la version stable dans la nouvelle année.
