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

Creation formulaire candidature structure #196

Merged
merged 33 commits into from
Aug 29, 2024

Conversation

Alezco
Copy link
Contributor

@Alezco Alezco commented Jul 29, 2024

No description provided.

@Alezco Alezco force-pushed the creation-formulaire-candidature-structure branch from 2ca5a63 to a0df62d Compare July 29, 2024 15:25
src/views/candidature-structure/useSiretApi.js Outdated Show resolved Hide resolved
src/views/candidature-structure/useSiretApi.js Outdated Show resolved Hide resolved
src/views/candidature-structure/useSiretApi.js Outdated Show resolved Hide resolved
src/views/candidature-structure/useSiretApi.js Outdated Show resolved Hide resolved
src/views/candidature-structure/useSiretApi.js Outdated Show resolved Hide resolved
@Ornella452 Ornella452 force-pushed the creation-formulaire-candidature-structure branch from 37c9b5e to 088b8d8 Compare July 30, 2024 09:34
@Alezco Alezco force-pushed the creation-formulaire-candidature-structure branch 2 times, most recently from 35ffb9a to da49fab Compare July 30, 2024 15:46
@Alezco Alezco force-pushed the creation-formulaire-candidature-structure branch from da49fab to 38d30e0 Compare July 30, 2024 15:47
@Alezco Alezco force-pushed the creation-formulaire-candidature-structure branch from 7155a8b to dd0017f Compare August 1, 2024 07:58
src/components/commun/Checkbox.jsx Outdated Show resolved Hide resolved
src/views/candidature-structure/Engagement.jsx Outdated Show resolved Hide resolved
const [companyData, setCompanyData] = useState(null);

const search = (async searchValue => {
if (![6, 7, 14].includes(searchValue.length)) {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Isoler ce tableau dans une variable pour expliciter que ce sont les tailles possibles de SIRET, RIDET, etc

Copy link
Contributor Author

Choose a reason for hiding this comment

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

const TAILLE_SIRET = 7;
const TAILLE_RIDET = 6;

const taillesPossibles = [TAILLE_SIRET, TAILLE_RIDET];


try {
let data;
if (searchValue.length === 6 || searchValue.length === 7) {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Suggested change
if (searchValue.length === 6 || searchValue.length === 7) {
if (searchValue.length === TAILLE_SIRET || searchValue.length === TAILLE_RIDET) {

return data.results[0] ?? null;
}

export const useCompanyFinder = token => {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Suggested change
export const useCompanyFinder = token => {
export const useEntrepriseFinder = token => {

if (searchValue.length === 6 || searchValue.length === 7) {
data = await fetchRidetData(searchValue);
if (data) {
setCompanyData({
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Suggested change
setCompanyData({
setEntrepriseData({

De manière générale, renommer les company en entreprise

setLoading(false);
}
});
const debouncedSearch = useCallback(debounce(search, 300), [search]);
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Mettre le debounce dans le composant plutôt que dans le hook

@Alezco Alezco force-pushed the creation-formulaire-candidature-structure branch from 4c2e989 to e63ed7a Compare August 29, 2024 12:51
@Alezco Alezco merged commit a2dcd52 into recette Aug 29, 2024
3 checks passed
@Alezco Alezco deleted the creation-formulaire-candidature-structure branch August 29, 2024 14:32
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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants