Connexion: Rediriger les utilisateurs d'InclusionConnect vers ProConnect

[calummackervoy]

🤔 Pourquoi ?

InclusionConnect a été remplacé par ProConnect dans login_generic.html, mais ExistingUserLogin présente nos utilisateurs avec la mode de connexion active sur leur compte.

ExistingUserLogin est utilisé quand il y a un conflit des e-mails pendant une connexion avec un SSO, et dans le parcours de connexion candidat.

🚨 À vérifier

• Mettre à jour le CHANGELOG_breaking_changes.md ?

🏝️ Comment tester

• Se connecter avec un candidat où identity_provider == "IC"

💻 Captures d'écran

[calummackervoy]

@tonial j'ai trouvé un autre bug où le bouton inclusion connect n'est pas visible sur la page (la texte est blanche). Je pense que cette condition sera vrai seulement sur l'environment local des devs... on ne peut pas juste supprimer login_inclusion_connect.html et la possibilité de se connecter avec inclusion connect ?

[tonial]

On ne peut pas encore supprimer cette option car IC est encore utilisé sur la démo.
Il faut que je prépare un point avec le métier pour voir ce qu'on fait sur la démo pour permettre la connexion

[tonial]

Ça devrait fonctionner.
Par contre, attention un candidat ne peut pas utiliser IC, donc le cas de test proposé n'est pas bon.

[calummackervoy]

attention un candidat ne peut pas utiliser IC

Ah oui bien sûr, merci. Le test impliqué par le PR (test_login) utilise UserFactory avec kind=IdentityProvider.supported_user_kinds[identity_provider][0] donc je pense que le test est bon - je vais changer le nom du PR et la message du commit