Fix + refactor formulaire #225
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…de postal au lieux de code commune
…ller + rectif wording
Ornella452
force-pushed
the
refactoFormulaireApiGeo
branch
from
64ea604 to
9cf453b
Compare
Alezco
reviewed
Oct 1, 2024
| onChange={debounce(event => searchByName(event.target.value))} | ||
| onChange={debounce(async event => { | ||
| searchByName(event.target.value); | ||
| const codeCommune = await villes.find(({ codesPostaux, nom }) => |
There was a problem hiding this comment.
Essayer de déplacer ce bout de code dans buildConseillerData et simplifier ce composant
src/views/candidature-conseiller/CandidatureConseiller.test.jsx
Outdated
Show resolved
Hide resolved
src/views/candidature-conseiller/CandidatureConseiller.test.jsx
Outdated
Show resolved
Hide resolved
src/views/candidature-conseiller/CandidatureConseiller.test.jsx
Outdated
Show resolved
Hide resolved
src/views/candidature-conseiller/CandidatureConseiller.test.jsx
Outdated
Show resolved
Hide resolved
...onfirmation-email-candidature-conseiller/PageConfirmationEmailCandidatureConseiller.test.jsx
Outdated
Show resolved
Hide resolved
Ornella452
force-pushed
the
refactoFormulaireApiGeo
branch
from
01a5934 to
c681535
Compare
Ornella452
force-pushed
the
refactoFormulaireApiGeo
branch
from
c681535 to
8578f5b
Compare
Alezco
approved these changes
Oct 2, 2024
Ornella452
added a commit
that referenced
this pull request
Oct 10, 2024
* 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
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.