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.

Sign up for GitHub

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

Jump to bottom

Feat: Conformation email structure #219

Merged

Ornella452 merged 5 commits into recette from conformationEmailStructure

Sep 20, 2024

Contributor

Ornella452 commented Sep 19, 2024

No description provided.

Ornella452 added 2 commits

September 19, 2024 20:14


          bouton confirmation pour l'inscription structure

bd7f77c


          renommage component pour dissocier avec celle version structure

490d48a

Alezco reviewed

View reviewed changes

src/views/confirmation-email-candidature-structure/ConfirmationEmailCandidatureStructure.jsx Outdated

		import { useParams } from 'react-router-dom';
		import { useApiConfirmationEmailCandidatureStructure } from './useApiConfirmationEmailCandidatureStructure';

Contributor

Alezco Sep 20, 2024

Ligne vide en trop

src/views/confirmation-email-candidature-structure/ConfirmationEmailCandidatureStructure.jsx Outdated

+                useEffect(() => {
+                  window.scrollTo({ top: 0, behavior: 'smooth' });
+                }, []);
+                const ClickBoutonConfirmer = async () => {

Contributor

Alezco Sep 20, 2024

Ce nom de fonction dit comment on l'appelle mais pas ce qu'elle fait. Comme nom, je propose plutôt confirmerEmailCandidatureStructure

...iews/confirmation-email-candidature-structure/useApiConfirmationEmailCandidatureStructure.js Outdated

		@@ -0,0 +1,19 @@

Contributor

Alezco Sep 20, 2024

Quelques lignes vides en trop dans ce fichier aussi

src/views/confirmation-email-candidature-structure/PageConfirmationEmailCandidature.test.jsx Outdated

+                it('quand j’affiche la page de confirmation de l’email validée alors le titre et les informations de la page s’affichent', () => {
+                  // WHEN
+                  render(<ConfirmationEmailCandidatureStructure />);
+                  vi.spyOn(window, 'scrollTo').mockImplementation();

Contributor

Alezco Sep 20, 2024

Je vois que cette ligne se répète. A mettre dans un beforeEach ?

Contributor Author

Ornella452 Sep 20, 2024

je supprime finalement, car j'ai toujours l'erreur Error: Not implemented: window.scrollTo

src/views/confirmation-email-candidature-structure/PageConfirmationEmailCandidature.test.jsx Outdated

+                  expect(envoyer).toBeInTheDocument();
+                });
+                it('quand l’utilisateur clique sur le bouton et que le lien est valide alors un message success s’affiche', async () => {

Contributor

Alezco Sep 20, 2024

Suggested change

      
              it('quand l’utilisateur clique sur le bouton et que le lien est valide alors un message success s’affiche', async () => {
          
              it('quand l’utilisateur clique sur le bouton et que le lien est valide alors un message de succès s’affiche', async () => {

src/views/confirmation-email-candidature-structure/PageConfirmationEmailCandidature.test.jsx Outdated

+                  const envoyer = screen.getByRole('button', { name: 'Confirmer' });
+                  expect(envoyer).toBeInTheDocument();
+                  // eslint-disable-next-line testing-library/no-unnecessary-act
+                  await act(() => {

Contributor

Alezco Sep 20, 2024

Par curiosité, est-ce que ça fonctionne sans le act ? Parfois il n'est pas obligatoire

Contributor Author

Ornella452 Sep 20, 2024

oui obligatoire 👍

src/views/confirmation-email-candidature-structure/PageConfirmationEmailCandidature.test.jsx Outdated

+                  );
+                  expect(messageError).toBeInTheDocument();
+                  const messageSuccess = screen.queryByText('Votre email a été confirmé et votre inscription est maintenant active.' +

Contributor

Alezco Sep 20, 2024

Même commentaire ici : je pense qu'il n'y a pas besoin de tester la non-présence de tous les messages possibles, juste la présence du message recherché

src/views/confirmation-email-candidature-structure/PageConfirmationEmailCandidature.test.jsx Outdated

+                  const messageSuccess = screen.queryByText('Votre email a été confirmé et votre inscription est maintenant active.' +
+                  ' Vous recevrez un mail d’activation de votre espace structure lorsque votre candidature aura été validée.', { selector: 'p' });
+                  expect(messageSuccess).not.toBeInTheDocument();
+                  const messageErrorGenerale = screen.queryByText('Une erreur s’est produite veuillez réessayer plus tard.', { selector: 'p' });

Contributor

Alezco Sep 20, 2024

Suggested change

      
                const messageErrorGenerale = screen.queryByText('Une erreur s’est produite veuillez réessayer plus tard.', { selector: 'p' });
          
                const messageErreurGenerale = screen.queryByText('Une erreur s’est produite veuillez réessayer plus tard.', { selector: 'p' });

src/views/confirmation-email-candidature-structure/PageConfirmationEmailCandidature.test.jsx Outdated

		});


		it('quand l’utilisateur clique sur le bouton et qu’une erreur innatendu se produit alors un message d’erreur s’affiche', async () => {

Contributor

Alezco Sep 20, 2024

Suggested change

      
              it('quand l’utilisateur clique sur le bouton et qu’une erreur innatendu se produit alors un message d’erreur s’affiche', async () => {
          
              it('quand l’utilisateur clique sur le bouton et qu’une erreur innatendue se produit alors un message d’erreur s’affiche', async () => {

src/views/confirmation-email-candidature-structure/PageConfirmationEmailCandidature.test.jsx Outdated

+                  );
+                  expect(messageError).toBeInTheDocument();
+                  const messageError403 = screen.queryByText('Impossible de valider l’email, le lien a expiré ou est invalide.', { selector: 'p' });

Contributor

Alezco Sep 20, 2024

Même commentaire ici sur le test des messages d'erreur

Ornella452 added 3 commits

September 20, 2024 15:11


          fix nommage

ad6ff11


          retour commentaire

d9b1616


          fix lint

2bccb30

Alezco approved these changes

View reviewed changes

Ornella452 merged commit f179f38 into recette

3 checks passed

Ornella452 deleted the conformationEmailStructure branch

September 20, 2024 16:07

Ornella452 added a commit that referenced this pull request


          MEP (#233)

84923d2

* Optimisation du build

* build(deps): bump the minor-and-patch group with 2 updates

Bumps the minor-and-patch group with 2 updates: [@reduxjs/toolkit](https://github.com/reduxjs/redux-toolkit) and [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite).


Updates `@reduxjs/toolkit` from 2.2.6 to 2.2.7
- [Release notes](https://github.com/reduxjs/redux-toolkit/releases)
- [Commits](reduxjs/redux-toolkit@v2.2.6...v2.2.7)

Updates `vite` from 5.3.4 to 5.3.5
- [Release notes](https://github.com/vitejs/vite/releases)
- [Changelog](https://github.com/vitejs/vite/blob/main/packages/vite/CHANGELOG.md)
- [Commits](https://github.com/vitejs/vite/commits/v5.3.5/packages/vite)

---
updated-dependencies:
- dependency-name: "@reduxjs/toolkit"
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: minor-and-patch
- dependency-name: vite
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: minor-and-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

* Creation formulaire candidature structure (#196)

* :feat intialisation des tests formulaire candidature structure

* test formulaire partie 'information contact' + 'votre besoin CN'

* Retours de code review

* Ajout d'un scroll vers la bonne section du formulaire

* :feat ajout premier bloc formulaire informations contact structure

* Refacto des informations de structure

* partie Votre motivation + test

* :feat ajout premier bloc formulaire informations contact structure

* Refacto des informations de structure

* Retours de code review

* partie engagement + test

* partie submit + test + rectif lint

* :feat intialisation des tests formulaire candidature structure

* test formulaire partie 'information contact' + 'votre besoin CN'

* Retours de code review

* Ajout d'un scroll vers la bonne section du formulaire

* partie Votre motivation + test

* :feat ajout premier bloc formulaire informations contact structure

* Refacto des informations de structure

* partie engagement + test

* partie submit + test + rectif lint

* :feat ajout premier bloc formulaire informations contact structure

* Refacto des informations de structure

* Retours de code review

* Correction des tests

* quelques rectif required + config min + ponctuation

* retour commentaire , required pour la checkbox

* :feat encart information structure

* :fix refacto nommage retours commentaires

---------

Co-authored-by: dienamo <[email protected]>
Co-authored-by: Ornella Ourfi <[email protected]>

* Feat: Creation formulaire candidature structure coordo (#198)

* Refacto formulaire conseiller

* :feat intialisation des tests formulaire candidature structure

* test formulaire partie 'information contact' + 'votre besoin CN'

* Retours de code review

* Ajout d'un scroll vers la bonne section du formulaire

* partie Votre motivation + test

* :feat ajout premier bloc formulaire informations contact structure

* Refacto des informations de structure

* partie engagement + test

* :feat ajout premier bloc formulaire informations contact structure

* Refacto des informations de structure

* Correction des tests

* Ajout du formulaire coordinateur

---------

Co-authored-by: dienamo <[email protected]>
Co-authored-by: Ornella Ourfi <[email protected]>

* Modifications formulaire conseiller (#206)

* Refacto du formulaire conseiller

* Fix merge conflicts

* Fix: input date avec la règle minimum date du jour (#201)

* On ne peut pas valider le formulaire de candidature d'un conseiller si on n'a pas rempli au moins une situation

Le test n'était pas correct car il testait que la phrase s'affiche mais le formulaire se validait quand même.

* Simplification du formulaire de candidature conseiller en réduisant le nombre de rendu quand on coche une situation

* rectif formulaire CN pour règle métier minimum date du jour

* resolve test failed

* Fix tests

---------

Co-authored-by: Fabien Mercier <[email protected]>
Co-authored-by: Alezco <[email protected]>

* Appel du back-end depuis le formulaire (#207)

* Appel du back-end depuis le formulaire

* Ajout de la page de formulaire validé

* Correction des problèmes de validation

* Ajout de tests pour la candidature validée

* Retours de code review

* Retours supplémentaires de code review

* WIP : test d'erreur à faire fonctionner

* Fix test stub

* Ajout d'un test de changement de page

* Amélioration des tests (#211)

* Amélioration de npm install

* Ajout d'une vérification de captcha avec hCaptcha (#213)

* Ajout du captcha sur le formulaire coordo (#216)

* :feat recherche entreprise par siret ou ridet (#209)

* :feat rechereche entreprise par siret ou ridet

* :feat ajout du loader dans l'input

* :feat tests de recherche par siret ou ridet

* :feat refactorisation pour avoir les valeurs de formData directement

* :feat refactorisation du hook de gestion des données insee ou adresse

* :feat mise à jour des composants pour récuperation dans formdata

* :fix retours commentaires et migration scss-->css

* Feat: confirmation email (#215)

* confirmation email

* lint

* Connexion au back-end du formulaire structure (#218)

* Refacto

* Ajout des informations de la ville

* Feat: Conformation email structure (#219)

* bouton confirmation pour l'inscription structure

* renommage component pour dissocier avec celle version structure

* fix nommage

* retour commentaire

* fix lint

* fix : regex + nommage etc.. (#221)

* Ajout du lien au back-end sur le formulaire coordinateur (#220)

* Ajout du lien au back-end sur le formulaire coordinateur

* Ajout de tests

* Correction de l'appel à la GeoAPI

---------

Co-authored-by: Ornella <[email protected]>

* Fix: url api (#222)

* fix url api

* fix return function pour renvoyer les données codeCommune, codeRegon etc..

* Fix + refactor formulaire (#225)

* 1er refactor (codeCommune en paramètre + rectif de récuperation du cdde postal au lieux de code commune

* import alert dsfr

* renommage composant candidature-validee => candidature-validee-conseiller + rectif wording

* copy candidature validee conseiller en version pour structure

* resolve warning Error: Not implemented: window.scrollTo

* test pour récupérer la valeur après le build (version avant refacto api geo)

* ajout de 2 tests pour le téléphone dans le cas où c'est rempli par l'utilisateur

* refacto codeCommune + api adresse

* correction test

* fix correction required champs

* refactor window scrollTo

* retour commentaire

* refactor code commune + ajout du '*' pur l'input required à true

* fix get info commune

* fix: ajout trim + autocompletion + gestion erreur (#227)

* fix trim

* aucompletion off pour le form conseiller

* gestion dans le cas failed to fetch

* retour commentaire

* recif wording + png (#228)

* recif wording message erreur

* remplace img png par avif

* fix avif en png (#229)

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: Fabien Mercier <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Benjamin Morali <[email protected]>
Co-authored-by: dienamo <[email protected]>
Co-authored-by: dienamo <[email protected]>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet