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 une modale de confirmation des invitations sur Pix Orga (pix-14535) #11201

Conversation

EmmanuelleBonnemay
Copy link
Contributor

@EmmanuelleBonnemay EmmanuelleBonnemay commented Jan 23, 2025

🥞 Problème

Il arrive que des utilisateurs de Pix Orga invitent des prescrits à rejoindre des organisations en tant que membres d'équipes.

🥓 Proposition

Ajouter une modale de confirmation lors de la création d'une invitation à rejoindre une organisation.

🧃 Remarques

  • La gestion d'erreur a été rajoutée côté front pour éviter l'ouverture de la modale en cas de données non valides dans le formulaire.
  • Les traductions sont provisoires

😋 Pour tester

  • Se connecter sur Pix Orga en tant qu'admin
  • Aller sur onglet "Equipe", puis "Inviter un membre"
  • Ne pas remplir le champ email, cliquer sur "Inviter"
  • constater l'affichage d'une erreur "Ce champ est obligatoire"
  • Remplir le champ avec des données non valides (par exemple : '[email protected];etienne-format.incorrect')
  • Constater l'affichage de l'erreur "Les données que vous avez soumises ne sont pas au bon format".
  • Remplir le champ avec des données valides
    (par exemple ' [email protected],[email protected] ')
  • Constater la présence d'une pop-up de confirmation et l'envoi des deux invitations aux adresses indiquées

@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-14535-add-warning-modal-to-organisation-invitation-process branch from 320416c to 3db8210 Compare January 23, 2025 14:29
@EmmanuelleBonnemay EmmanuelleBonnemay self-assigned this Jan 23, 2025
@P-Jeremy
Copy link
Contributor

P-Jeremy commented Jan 24, 2025

Test func champs vide OK ✅
image

Test func mail non conforme OK ✅
image

Test func modale ouverte OK ✅
image

Test func confirmation envoi OK ✅
image

// then
assert.dom(await screen.findByText(errorMessage)).exists();
assert.dom(screen.queryByRole('dialog')).doesNotExist();
});
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggestion: saut de ligne

name: t('pages.team-new-item.invite-button'),
});
await click(inviteButton);
await waitForDialog();
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggestion: Ajouter saut de ligne

orga/app/components/team/invite-form.hbs Outdated Show resolved Hide resolved
orga/app/styles/components/team/invite-form.scss Outdated Show resolved Hide resolved
orga/tests/acceptance/team-creation-test.js Outdated Show resolved Hide resolved
@EmmanuelleBonnemay EmmanuelleBonnemay force-pushed the pix-14535-add-warning-modal-to-organisation-invitation-process branch from 3db8210 to ae8ea5e Compare January 27, 2025 08:22
@EmmanuelleBonnemay EmmanuelleBonnemay force-pushed the pix-14535-add-warning-modal-to-organisation-invitation-process branch from ae8ea5e to 7161dfc Compare January 27, 2025 08:28
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-14535-add-warning-modal-to-organisation-invitation-process branch from 7161dfc to 33aebf7 Compare January 27, 2025 13:13
@pix-service-auto-merge pix-service-auto-merge merged commit c1a6ec8 into dev Jan 27, 2025
8 of 10 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-14535-add-warning-modal-to-organisation-invitation-process branch January 27, 2025 13:19
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