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.

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

fix: ajout trim + autocompletion + gestion erreur #227

Merged
merged 4 commits into from
Oct 3, 2024
Merged

fix: ajout trim + autocompletion + gestion erreur #227

merged 4 commits into from
Oct 3, 2024

Conversation

Ornella452
Copy link
Contributor

No description provided.

@Ornella452 Ornella452 changed the title fix: ajout trim fix: ajout trim + autocompletion Oct 2, 2024
@Ornella452 Ornella452 changed the title fix: ajout trim + autocompletion fix: ajout trim + autocompletion + gestion erreur Oct 2, 2024
Alezco
Alezco reviewed Oct 3, 2024
View reviewed changes
src/components/commun/Input.jsx Outdated Show resolved Hide resolved
src/views/candidature-conseiller/CandidatureConseiller.jsx Outdated Show resolved Hide resolved
src/views/candidature-conseiller/CandidatureConseiller.test.jsx Outdated
@@ -661,4 +661,51 @@ describe('candidature conseiller', () => {

vi.useRealTimers();
});

it('quand je remplis le formulaire et q’une erreur se produit alors un message d’erreur s’affiche', async () => {
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
it('quand je remplis le formulaire et q’une erreur se produit alors un message d’erreur s’affiche', async () => {
it('quand je remplis le formulaire et qu’une erreur se produit, alors un message d’erreur s’affiche', async () => {

src/views/candidature-coordinateur/CandidatureCoordinateur.jsx Outdated
@@ -39,10 +39,13 @@ export default function CandidatureCoordinateur() {
const formData = new FormData(event.currentTarget);
const coordinateurData = await buildCoordinateurData(formData, geoLocation, codeCommune);
const resultatCreation = await creerCandidatureCoordinateur(coordinateurData);
if (resultatCreation.status >= 400) {
if (resultatCreation.status && resultatCreation.status >= 400) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Même commentaire ici

src/views/candidature-coordinateur/CandidatureCoordinateur.test.jsx Outdated
@@ -462,5 +462,61 @@ describe('candidature coordinateur', () => {

vi.useRealTimers();
});

it('quand je remplis le formulaire et q’une erreur se produit alors un message d’erreur s’affiche', async () => {
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
it('quand je remplis le formulaire et q’une erreur se produit alors un message d’erreur s’affiche', async () => {
it('quand je remplis le formulaire et qu’une erreur se produit, alors un message d’erreur s’affiche', async () => {

src/views/candidature-structure/CandidatureStructure.jsx Outdated
@@ -40,10 +40,13 @@ export default function CandidatureStructure() {
const formData = new FormData(event.currentTarget);
const structureData = await buildStructureData(formData, geoLocation, codeCommune);
const resultatCreation = await creerCandidatureStructure(structureData);
if (resultatCreation.status >= 400) {
if (resultatCreation.status && resultatCreation.status >= 400) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Pareil ici

src/views/candidature-structure/CandidatureStructure.test.jsx Outdated
@@ -555,5 +555,61 @@ describe('candidature structure', () => {

vi.useRealTimers();
});

it('quand je remplis le formulaire et q’une erreur se produit alors un message d’erreur s’affiche', async () => {
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
it('quand je remplis le formulaire et q’une erreur se produit alors un message d’erreur s’affiche', async () => {
it('quand je remplis le formulaire et qu’une erreur se produit, alors un message d’erreur s’affiche', async () => {

@Ornella452 Ornella452 merged commit 88497a4 into recette Oct 3, 2024
3 checks passed
@Ornella452 Ornella452 deleted the fixForm branch October 3, 2024 13:49
Ornella452 added a commit that referenced this pull request Oct 10, 2024
@Ornella452 @NerOcrO
@dependabot @Alezco @dienamo
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
Reviewers

@Alezco Alezco Alezco approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Ornella452 @Alezco