Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Invitations: utiliser automatiquement toutes les invitations valides disponibles à la connexion [GEN-2221] #5067

Merged
merged 2 commits into from
Nov 12, 2024

Conversation

tonial
Copy link
Contributor

@tonial tonial commented Nov 10, 2024

🤔 Pourquoi ?

Cela permet d'être sur que les prescripteurs et employeurs utilisant ProConnect et Inclusion Connect soient correctement associés à leur structures.

🍰 Comment ?

Décrivez en quelques mots la solution retenue et mise en oeuvre, les difficultés ou problèmes rencontrés. Attirez l'attention sur les décisions d'architecture ou de conception importantes.

🚨 À vérifier

  • Mettre à jour le CHANGELOG_breaking_changes.md ?

🏝️ Comment tester

Les instructions pour reproduire le problème, les profils de test, le parcours spécifique à utiliser, etc. Si vous disposez d'une recette jetable, mettre l'URL pour tester dans cette partie.

💻 Captures d'écran

@tonial tonial self-assigned this Nov 10, 2024
Used in Inclusion Connect and ProConnect login flow
@tonial tonial force-pushed the alaurent/improve_invitations branch from e3fc80f to 563d0db Compare November 10, 2024 06:24
Copy link
Contributor

@xavfernandez xavfernandez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cela me semble un comportement étonnant : ce n'est pas parce que je suis invité quelque part que j'ai envie d'y aller 😬.

Si je comprends bien la PR, les utilisateurs de ProConnect (i.e. tous les prescripteurs et employeurs) rejoindront automatiquement toutes structures dans lesquelles ils ont été invitées ?

@tonial
Copy link
Contributor Author

tonial commented Nov 12, 2024

Oui, c'est bien ça.

Je pense que c'est un moindre mal : il y a peu de chance que tu te fasses inviter par erreur à une structure.
Je vérifie avec le métier

@tonial
Copy link
Contributor Author

tonial commented Nov 12, 2024

Zohra pense aussi que ce n'est pas un problème, je merge !

@tonial tonial added this pull request to the merge queue Nov 12, 2024
Merged via the queue into master with commit 55cde29 Nov 12, 2024
9 checks passed
@tonial tonial deleted the alaurent/improve_invitations branch November 12, 2024 11:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants