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] Ajouter un bouton pour renvoyer l'invitation à un centre de certification dans pix admin (PIX-10018) #11203

Conversation

theotime2005
Copy link
Contributor

🥞 Problème

Dans Pix Admin, sur la page d'un centre de certification, dans la section invitation, on ne pouvait que:

  • Envoyer une nouvelle invitation en remplissant les champs email, role et langue,
  • Annuler une invitation envoyée en cliquant sur le bouton.
    Il n'était donc pas possible de renvoyer une invitation déjà existante.

🥓 Proposition

Ajouter un bouton permettant de renvoyer l'email à côté de celui pour annuler l'invitation.

🧃 Remarques

Le bouton est peut-être mal placé.

😋 Pour tester

  • Se rendre sur Pix-admin,
  • Se connecter avec le compte [email protected],
  • Aller sur la page des centre de certifications,
  • Prendre le centre Accessorium,
  • Aller dans l'onglet Invitation,
  • Entrer un vrai mail dans le champ correspondant, et choisissez français pour la langue et automatique pour le rôle,
  • Envoyez l'invitation,
  • Constater que le mail est bien arrivé,
  • Noter l'heure de l'envoi dans le tableau des invitations envoyées,
  • Attendre une à 2 minutes,
  • Dans la colone de droite, sur la ligne de l'email utilisé pour le teste, appuyer sur Renvoyer l'invitation,
  • Constater la bonne réception du nouveau mail,
  • Constater que la date a été mise à jour dans le tableau.

@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 self-assigned this Jan 23, 2025
@theotime2005 theotime2005 force-pushed the pix-10018-add-a-button-to-send-a-new-certif-invitation branch 2 times, most recently from 00d61d5 to 939345c Compare January 24, 2025 22:51
@theotime2005 theotime2005 requested a review from a team as a code owner January 24, 2025 22:51
@bpetetot
Copy link
Contributor

@theotime2005 Je pense que tu as pris un commit par mégarde :) Celui-ci d'une release: 800bb83
Il faudrait que tu rebases et que tu enlèves ce commit

@theotime2005 theotime2005 force-pushed the pix-10018-add-a-button-to-send-a-new-certif-invitation branch from 939345c to fa4d53f Compare January 27, 2025 08:45
@theotime2005 theotime2005 requested a review from a team as a code owner January 27, 2025 08:45
@theotime2005 theotime2005 force-pushed the pix-10018-add-a-button-to-send-a-new-certif-invitation branch 2 times, most recently from 1fb5e1e to 5af780e Compare January 27, 2025 15:43
@theotime2005 theotime2005 added the ⚠️ PR Inheritance This PR inherits a first-to-merge PR and will need a rebase label Jan 28, 2025
@theotime2005 theotime2005 force-pushed the pix-10018-add-a-button-to-send-a-new-certif-invitation branch from 5af780e to f93e623 Compare January 28, 2025 09:12
@theotime2005 theotime2005 reopened this Jan 28, 2025
@theotime2005 theotime2005 force-pushed the pix-10018-add-a-button-to-send-a-new-certif-invitation branch 5 times, most recently from d4cba32 to 9451f41 Compare January 30, 2025 10:39
@theotime2005 theotime2005 removed the ⚠️ PR Inheritance This PR inherits a first-to-merge PR and will need a rebase label Jan 30, 2025
@theotime2005 theotime2005 force-pushed the pix-10018-add-a-button-to-send-a-new-certif-invitation branch from 9451f41 to ca66726 Compare January 30, 2025 10:59
@P-Jeremy
Copy link
Contributor

Test func OK ✅

image

transform: (invitation) => {
return {
...invitation,
language: invitation.locale,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oui, mais pour la partie des centres de certification, c'est language qui est utilisé, donc on s'adapte:)

lego-technix

This comment was marked as resolved.

@EmmanuelleBonnemay EmmanuelleBonnemay requested review from EmmanuelleBonnemay and removed request for a team January 31, 2025 08:32
@theotime2005
Copy link
Contributor Author

Sauf que je ne sais pas exactement dans quel(s) commit(s) les mettre.

@theotime2005 theotime2005 force-pushed the pix-10018-add-a-button-to-send-a-new-certif-invitation branch from 3dc05ea to c411bbf Compare January 31, 2025 09:36
@theotime2005 theotime2005 force-pushed the pix-10018-add-a-button-to-send-a-new-certif-invitation branch from c411bbf to 964facc Compare January 31, 2025 09:37
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 🦊

@pix-service-auto-merge pix-service-auto-merge merged commit 54332b1 into dev Jan 31, 2025
6 of 10 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-10018-add-a-button-to-send-a-new-certif-invitation branch January 31, 2025 09:42
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.

6 participants