Skip to content

Conversation

@theotime2005
Copy link
Contributor

@theotime2005 theotime2005 commented Jun 13, 2025

🔆 Problème

Lorsqu'on renvoie une invitation à rejoindre une organisation depuis Pix admin, si on met de nouveaux paramètres tels que le rôle et la locale, ceux-ci ne sont pas modifiés par rapport à la première invitation. Et on est alors obliger d’annuler l'invitation puis d’en recréer une autre avec les nouveaux paramètres désirés.

⛱️ Proposition

Modifier la fonction organizationInvitationService.createOrUpdateOrganizationInvitation pour qu’en cas d’organizationInvitation existante le role et la locale soient aussi modifiés et pas uniquement la « modificationDate » (updatedAt).

Considérations de sécurité

Le choix a été fait de continuer d’utiliser la même fonction organizationInvitationService.createOrUpdateOrganizationInvitation pour les usecases create-organization-invitations, create-organization-invitation-by-admin et resend-organization-invitation plutôt que de définir une nouvelle fonction organizationInvitationService.createOrUpdateOrganizationInvitationForAdmin.

En effet dupliquer la fonction organizationInvitationService.createOrUpdateOrganizationInvitation pour lui permettre de modifier le role et la locale n’a pas d’intérêt en terme de sécurité puisque la sécurité est réalisée au niveau du usecase resend-organization-invitation qui ne va pas passer de role ou de locale.

🌊 Remarques

RAS

🏄 Pour tester

  • Se rendre sur Pix admin,
  • Se connecter avec le compte [email protected],
  • Se rendre sur l'organisation Accis,
  • Aller dans la section Invitation,
  • Envoyer une invitation à [email protected] en mettant le rôle MEMBER et la locale francophone,
  • Constater que l'invitation et bien créée et apparaît dans le tableau,
  • Renvoyer une invitation à [email protected] mais avec le rôle ADMIN et la locale Français,
  • Constater que l'invitation est mise à jour en base.

Tester également le cas de renvoi d’une invitation dans Pix Orga pour vérifier qu’il n’y a pas de régression car le usecase resend-organization-invitation.usecase.js a également été modifié.

@theotime2005 theotime2005 requested a review from a team as a code owner June 13, 2025 07:45
@theotime2005 theotime2005 self-assigned this Jun 13, 2025
@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@theotime2005 theotime2005 force-pushed the pix-18065-update-some-informations-in-organization-invitation branch 2 times, most recently from 608209d to 187e58e Compare June 13, 2025 09:44
@theotime2005 theotime2005 added 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally labels Jun 13, 2025
@P-Jeremy
Copy link
Contributor

Test func ok ✅

@theotime2005 theotime2005 force-pushed the pix-18065-update-some-informations-in-organization-invitation branch from 187e58e to baff424 Compare June 20, 2025 07:21
@Libouk Libouk force-pushed the pix-18065-update-some-informations-in-organization-invitation branch 2 times, most recently from bfaade3 to 70055b3 Compare June 20, 2025 14:05
@lego-technix lego-technix force-pushed the pix-18065-update-some-informations-in-organization-invitation branch 4 times, most recently from 2fd2987 to 91e043e Compare June 24, 2025 07:26
@P-Jeremy
Copy link
Contributor

Test func toujours OK ✅ 🎉

@bpetetot bpetetot added Tech Review OK and removed 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally labels Jun 24, 2025
@bpetetot bpetetot added Func Review OK PO validated functionally the PR 🚀 Ready to Merge labels Jun 24, 2025
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-18065-update-some-informations-in-organization-invitation branch from 91e043e to 005dfc7 Compare June 24, 2025 16:38
@pix-service-auto-merge pix-service-auto-merge merged commit 8641db7 into dev Jun 24, 2025
10 of 11 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-18065-update-some-informations-in-organization-invitation branch June 24, 2025 16:45
pix-service pushed a commit that referenced this pull request Jun 25, 2025
# [5.143.0](v5.142.0...v5.143.0) (2025-06-25)

### 🚀 Amélioration

- [#12604](#12604) Afficher une modale avant de quitter la page de fin de parcours (PIX-18023)
- [#12625](#12625) Ajouter un filtre sur l'api maddo pour remonter les dernières participations mise à jour (PIX-18288)
- [#12626](#12626) ajouter une route PATCH users/{id} pour enrichir un utilisateur anonyme avec des données d'authentification (Pix-18022)
- [#12639](#12639) Détacher les profile-rewards de l'organisation lors de la supression de learner (PIX-18113).
- [#12640](#12640) Lister par défaut les feature-toggles.
- [#12619](#12619) Repenser l'affichage des niveaux sur la page d'analyse de Pix Orga (PIX-18337)
- [#12635](#12635) v2 du module tmp-ia-fonctionnement-debut pour revue PYO

### 🐛 Correction

- [#12632](#12632) Corriger une typo dans le message de MER
- [#12528](#12528) Mettre à jour les informations d'une invitation à rejoindre une organisation lors d'un renvoie avec de nouveaux paramètres (PIX-18065)
- [#12621](#12621) Réparer l'affichage du nombre de tentatives de connexions restantes (PIX-18405)

### 🏗️ Tech

- [#12627](#12627) Bump audit-logger dependencies
- [#12568](#12568) Montée de version EmberJS pour PixAdmin

### ☕ Autre

- [#12638](#12638) Corriger l'exemple de titre de PR
@pix-service
Copy link
Contributor

🎉 This PR is included in version 5.143.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@lego-technix lego-technix changed the title [BUGFIX] Mettre à jour les informations d'une invitation à rejoindre une organisation lors d'un renvoie avec de nouveaux paramètres (PIX-18065) [BUGFIX] Mettre à jour les informations d'une invitation à rejoindre une organisation lors d'un renvoi avec de nouveaux paramètres (PIX-18065) Sep 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants