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

[FEATURE] Mettre à jour le champ UpdatedAt quand un membre d'équipe de centre de certification est désactivé (PIX-10548) #10936

Conversation

EmmanuelleBonnemay
Copy link
Contributor

@EmmanuelleBonnemay EmmanuelleBonnemay commented Jan 3, 2025

🎄 Problème

Actuellement, lorqu'un membre d'équipe de centre de certification est désactivé (depuis PixAdmin ou PixCertif), le champ updatedAt de la table certification-center-memberships n'est pas mis à jour.

🎁 Proposition

Mettre à jour ce champ, comme cela se fait pour les Organisations, pour que udpatedAt = disabledAt

🧦 Remarques

L'anonymisation ayant pour effet de désactiver les memberships, elle entre aussi dans le cas décrit plus haut.

🎅 Pour tester

DESACTIVATION

- depuis Pix Certif
- se connecter en tant que certif-pro@example.net
- désactiver un membre du centre de certification de certif-pro
- vérifier en base que le champ updatedAt de la table certification-center-membership a été mis à jour, et qu'il a désormais la même valeur que le champ disabledAt
- depuis Pix Admin
- Se connecter en tant qu'Admin (par exemple superadmin@example.net)
- désactiver un membre d'un centre de centre de certification - par exemple james-paledroits@example.net du centre accessorium
- attendre une minute, puis sur Pix Admin, désactiver James d'un autre centre (accessovolt par exemple).
- vérifier en base que le champ updatedAt de la table certification-center-membership a été mis à jour, et qu'il a désormais la même valeur que le champ disabledAt
- vérifier aussi que la mise à jour du champ UpdatedAt pour le second membership de James n'a pas modifié le premier : pour chaque ligne updatedAt doit avoir la même valeur que disabledAt.

ANONYMISATION

  • depuis Pix Admin
    • anonymiser un utilisateur membre de centre de certification (James Paledroits)
    • vérifier en base que dans le/les membership(s) de James déjà désactivés auparavant (Accessovolt et Accessorium) , aucun champ n'a été mis à jour. (Donc le champ updatedAt de la table certification-center-membership n'a pas été mis à jour).
    • Vérifier que le membership de James qui n'était pas désactivé (auprès du centre Accestral) a été mis à jour et que le champ updatedAt a la même valeur que le champ disabledAt.

@EmmanuelleBonnemay EmmanuelleBonnemay requested a review from a team as a code owner January 3, 2025 17:04
@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 :

@EmmanuelleBonnemay EmmanuelleBonnemay force-pushed the pix-10548-update-Updated-At-field-when-deactivating-certification-center-membership-from-Pix-Admin branch from 87a0fd6 to 167f407 Compare January 8, 2025 14:39
@EmmanuelleBonnemay EmmanuelleBonnemay self-assigned this Jan 8, 2025
@EmmanuelleBonnemay EmmanuelleBonnemay force-pushed the pix-10548-update-Updated-At-field-when-deactivating-certification-center-membership-from-Pix-Admin branch from 167f407 to 3d94431 Compare January 8, 2025 14:46
@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 :

@EmmanuelleBonnemay EmmanuelleBonnemay force-pushed the pix-10548-update-Updated-At-field-when-deactivating-certification-center-membership-from-Pix-Admin branch from 3d94431 to 8239f75 Compare January 8, 2025 15:15
@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 :

@EmmanuelleBonnemay EmmanuelleBonnemay force-pushed the pix-10548-update-Updated-At-field-when-deactivating-certification-center-membership-from-Pix-Admin branch from 8239f75 to 67a54c8 Compare January 8, 2025 17:21
Copy link
Contributor

@bpetetot bpetetot left a comment

Choose a reason for hiding this comment

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

✅ Revu et testé

Copy link
Contributor

@lego-technix lego-technix left a comment

Choose a reason for hiding this comment

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

✅ Lu et testé fonctionnellement avec succès avec Firefox 🦊

@EmmanuelleBonnemay EmmanuelleBonnemay force-pushed the pix-10548-update-Updated-At-field-when-deactivating-certification-center-membership-from-Pix-Admin branch 2 times, most recently from e75d7d1 to c653ceb Compare January 9, 2025 10:39
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-10548-update-Updated-At-field-when-deactivating-certification-center-membership-from-Pix-Admin branch from c653ceb to 55a31b1 Compare January 9, 2025 14:38
@pix-service-auto-merge pix-service-auto-merge merged commit 728f6b1 into dev Jan 9, 2025
7 of 8 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-10548-update-Updated-At-field-when-deactivating-certification-center-membership-from-Pix-Admin branch January 9, 2025 14:45
@Faraopix Faraopix changed the title [FEATURE] Mettre à jour le champ UpdatedAt quand un membre d'équipe de centre de certification est désactivé (PIX-15548) [FEATURE] Mettre à jour le champ UpdatedAt quand un membre d'équipe de centre de certification est désactivé (PIX-10548) Jan 10, 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.

5 participants