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

Amélioration du système de vérification géographique #4780

Merged
merged 2 commits into from
Jan 6, 2025
Merged

Amélioration du système de vérification géographique #4780

merged 2 commits into from
Jan 6, 2025

Conversation

jenovateurs
Copy link
Contributor

@jenovateurs jenovateurs commented Dec 24, 2024 •
edited
Loading

Amélioration du système de vérification de cohérence des points géographiques.
attached_to_institution des éléments liés à une epci.
A contrôler que cela fonctionne bien. Besoin d'un petit coup de main @yasmine-glitch si possible.
Merci d'avance.

@jenovateurs jenovateurs linked an issue Dec 24, 2024 that may be closed by this pull request
Mieux gérer le test de vérification geographique #4779
Closed
@jenovateurs jenovateurs changed the title Amélioration du système de vérification géographique et correct… Amélioration du système de vérification géographique Dec 24, 2024
@github-actions github-actions bot added this to the BC actuel milestone Dec 24, 2024
Shamzic
Shamzic reviewed Dec 30, 2024
View reviewed changes
tools/test-benefits-geographical-constraint-consistency.ts Outdated
@@ -31,12 +31,34 @@ function testGeographicalRelevancy(benefit) {
)
})
if (conditionGeo) {
if (
conditionGeo.type === "attached_to_institution" ||
conditionGeo.values.length > 1
Copy link
Contributor

Choose a reason for hiding this comment

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

J'ai peut-être mal saisi l'objectif de cette modification mais cette condition ne me semble pas logique car dans le cas où il y a plusieurs values dans conditionGeo, alors tu quittes la méthode en retournant isValid: true et la vérification avec conditionGeo.values.every n'est pas traitée par la suite.

Ça voudrait dire qu'une aide avec plusieurs values est automatiquement valide et que seules les aides avec une seule value sont vérifiées ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Très bonne question.
Après avoir réanalyser le code et peut-être que ma réponse devrait être ajouté dans le wiki. On ne peux pas faire de vérification sur les listes d'epci, region, ...
On peux uniquement vérifier la cohérence entre un élément et une institution.
Car pour les listes d'éléments, c'est plus complexe. Par exemple une aide d'une métropole qui prend en conditions certaines communes et d'autres non inclus dans la métropole.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

pour le every on pourrait le modifier pour gérer directement le premier et seul élément.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Cela répond à ta question ?

Copy link
Contributor

@Shamzic Shamzic Dec 31, 2024 •
edited
Loading

Choose a reason for hiding this comment

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

pour le every on pourrait le modifier pour gérer directement le premier et seul élément.

Je pense que ce serait plus compréhensible de le refacto ainsi oui

Cela répond à ta question ?

Oui parfait merci (:

@yasmine-glitch
Copy link
Contributor

@jenovateurs j'ai testé en local avec le code postal d'Arzon et j'ai bien la tarification solidaire de l'agglo de Vannes qui s'affiche. J'ai testé avec le code postal de Bischheim et j'ai bien le FAJ de l'eurométropole de Strasbourg.
Tout semble fonctionner.

@jenovateurs jenovateurs merged commit fe8d3e6 into main Jan 6, 2025
28 checks passed
@jenovateurs jenovateurs deleted the 4779-mieux-gérer-le-test-de-vérification-geographique branch January 6, 2025 15:42
Shamzic added a commit that referenced this pull request Feb 5, 2025
@Shamzic @yasmine-glitch
@dependabot @jenovateurs @EmileRolley
Migration jest en vitest + upgrade vue (#4820)
2eecbe5 
* Met à jour l'aide au dépôt de garantie de la région CVL. (#4773)

* Maj l'aide BAFA de la Caf de Meurthe et Moselle (#4783)

* montre l'aide au permis du pas de calais.

* Màj l'aidee bafa de la Caf de Meurthe et Moselle.

* Chore(deps): Bump next from 14.2.10 to 14.2.15 in /contribuer (#4772)

* Chore(deps): Bump next from 14.2.10 to 14.2.15 in /contribuer

Bumps [next](https://github.com/vercel/next.js) from 14.2.10 to 14.2.15.
- [Release notes](https://github.com/vercel/next.js/releases)
- [Changelog](https://github.com/vercel/next.js/blob/canary/release.js)
- [Commits](vercel/next.js@v14.2.10...v14.2.15)

---
updated-dependencies:
- dependency-name: next
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

* feat: changement sur l'action pour récupérer le sentry url avec dependabot

* feat: rollback pull request target

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Jeremy PASTOURET <pastouret.jeremy@gmail.com>

* Chore(deps): Bump openfisca-france in /openfisca (#4770)

Bumps [openfisca-france](https://github.com/openfisca/openfisca-france) from 169.7.1 to 169.13.0.
- [Release notes](https://github.com/openfisca/openfisca-france/releases)
- [Changelog](https://github.com/openfisca/openfisca-france/blob/master/CHANGELOG.md)
- [Commits](openfisca/openfisca-france@169.7.1...169.13.0)

---
updated-dependencies:
- dependency-name: openfisca-france
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* fix: fusion du authToken (partagé entre le front et le back) & configure le projet sentry front sur le build de vite.js (#4771)

* fix: par defaut valeur à 0 pour eviter une erreur dans open-fisca (#4776)

* Ajoute le revenu minimum étudiant de la ville de Gravelines (#4785)

* ajoute la ville de graveline.

* ajoute le revenu minimum étudiant de la ville de gravelines.

* Update ville_gravelines_aide_aux_etudiants.yml

* Mise à jour du paquet `mesaidesvelo` avec le nouveau paquet `@betagouv/aides-velo` (#4705)

* pkg: remove 'aides-velo' and add '@betagouv/aides-velo'

* refactor(aides-velo)!: use the '@betagouv/aides-velo' package

* fix(lib)!: IndividuMethods.age now returns undefined if date_naissance is undefined

* refactor(data/benefits)!: refactor aides-velo-generator with @betagouv/aides-velo

* fix(data): add missing institutions needed by @betagouv/aides-velo

* fix: nitpicks

* ci: change dependabot to watch on @betagouv/aides-velo instead of aides-velo

* fix(data): remove dependency to @lib/benefits/compute-aides-velo.js

* fix(lib): remove custom paths imports

* fix(lib): add missing individu-properties for interetsVelo

* Revert "fix(lib)!: IndividuMethods.age now returns undefined if date_naissance is undefined"

This reverts commit fd6cc3b.

* fix: corrige le test aides-vélo avec l'âge undefined

* refactor: compute-aides-velo

* chore: resize logos institutions & fix extensions

* fix: image logo de la ville Taillain Médoc en format png

* chore: fixe une limite de la description des aides à 600 pour autoriser les descriptions plus longues du package publicodes-aides-velo

* refactor: typage et structure de aides-velo-generator

* fix: typage dateDeValeur du mock de test d'une Situation

* chore: update betagouv/aides-velo, package-lock.json & package.json

* fix: ajoute nouvelles institutions manquantes + logos

* fix: limite description fixée à 450

* fix: màj taille max de la description des aides
En attente du merge betagouv/publicodes-aides-velo#20

* feat: cache la checkbox de l'aide vélo adapté aux PMR

* refactor: ordre dépendances

* feat: ajoute les informations du type de vélo dans le label de l'aide (via les infos en description)

* refactor: filtre du label des aides vélo

* refactor: supprime l'occurence "Ville de" dans les titres des aides vélo

* chore: met à jour le numéro de version du paquet aides vélo

* fix: ajoute les institutions et les logos manquants

---------

Co-authored-by: Emile Rolley <emile.rolley@tuta.io>

* Ajout groupe de boutons de démarrage de simulation depuis la page "Toutes les aides" (#4743)

* feat: ajout boutons démarrage simulation (page liste des aides)

* refactor: renommage variable plus précis

* feat: alignement responsive - boutons de démarrage de simulation (liste des aides)

* feat: ajoute la class tertiary du DSFR au back-button

* refactor: liste-aides

* Ajout d'une question sur les aides pour les études à l'étranger dans la FAQ (#4788)

* feat: ajout d'une question sur les aides pour les études à l'étranger dans la FAQ

* Update src/views/faq.vue

Co-authored-by: Jérémy PASTOURET <pastouret.jeremy@gmail.com>

---------

Co-authored-by: Jérémy PASTOURET <pastouret.jeremy@gmail.com>

* Chore(deps): Bump next from 14.2.15 to 14.2.21 in /contribuer (#4791)

Bumps [next](https://github.com/vercel/next.js) from 14.2.15 to 14.2.21.
- [Release notes](https://github.com/vercel/next.js/releases)
- [Changelog](https://github.com/vercel/next.js/blob/canary/release.js)
- [Commits](vercel/next.js@v14.2.15...v14.2.21)

---
updated-dependencies:
- dependency-name: next
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Ajout de la mutuelle communale du rhône (#4789)

* bump(open-fisca) - mise à jour de version

* feat: ajout de la mutuelle communale d'auvergne

* Amélioration du système de vérification géographique (#4780)

* feat: amélioration du système de vérification géographique et correction d'erreurs

* refacto: simplifier le code de vérification géographique

* Mise à jour de titre (doublon) de deux dispositifs de Bordeaux (#4766)

* feat: maj du libellé d'un dispositif en doublon

* fix: mise à jour du label de l'aide

* Ajout aide bafa Villefranche-sur-saone (#4793)

* ajout de l'aide au bafa de villefranche

* fix: ajout des points à la fin des conditions

* Update data/benefits/javascript/ville_villefranche_sur_saone_bafa_bafd.yml

Co-authored-by: Simon Hamery <shamery@free.fr>

* Update data/benefits/javascript/ville_villefranche_sur_saone_bafa_bafd.yml

Co-authored-by: Simon Hamery <shamery@free.fr>

* chore: resize logo villefranche

---------

Co-authored-by: Simon Hamery <shamery@free.fr>

* maj mutuelle région ara. (#4794)

* fix: mise à jour de l'EPCI exclu (métropole de Lille) pour les FSL du département Nord (#4795)

@yasmine-glitch Merci les précisions !

* Met à jour 4 aides de la région Pays de la Loire. (#4781)

* Met à jour 4 aides de la région Pays de la Loire.

* mises à jour.

* mises à jour.

* Update region_pays_de_la_loire-bourses-pour-les-élèves-et-étudiant-e-s-en-formation-sanitaire-et-sociale.yml

* Met à jour 2 aides (#4796)

* met à jour la carte illico solidaire pour limiter les faux positifs

* Met à jour lien de l'aide bafa de la caf de la haute-garonne.

* Chore(deps): Bump openfisca-france in /openfisca (#4799)

Bumps [openfisca-france](https://github.com/openfisca/openfisca-france) from 169.13.0 to 169.14.6.
- [Release notes](https://github.com/openfisca/openfisca-france/releases)
- [Changelog](https://github.com/openfisca/openfisca-france/blob/master/CHANGELOG.md)
- [Commits](openfisca/openfisca-france@169.13.0...169.14.6)

---
updated-dependencies:
- dependency-name: openfisca-france
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* màj vae hauts de france (#4801)

* Cache le RME de Gravelines + Màj l'aide Bafa de Caf Meurthe-et-Moselle (#4798)

* cache le revenu minimum étudiant de Gravelines.

* Met à jour l'aide Bafa de la Caf de Meurthe et Moselle.

* Cache trois aides de la région Pays de la Loire. (#4802)

* met à jour la description de la bourse de college pour davantage de clarté. (#4803)

* Cache deux aides et Met à jour une aide. (#4804)

* cache la bourse aux apprentis du val d'oise. A revérifier fin février.

* Met à jour l'aide au BAFA-BAFD de la Caf d'Ille-et-Vilaine.

* cache l'aide à l'autonomie des jeunes de la caf de Saône-et-Loire.

* feat: suppression suivi (#4810)

* Mise à jour de dépendances mineures (#4739)

* chore: mise à jour de dépendances mineurs

* fix: readapation du code en réutilisation la documentation officiel d'OpenID

* feat: premier niveau de connexion

* fix: redirection au login MCP (page accompagnement)

* ajout du rate limit

---------

Co-authored-by: Shamzic <shamery@free.fr>

* fix: empêche la fermeture de la modale de chargement des résultats lors d'un clic extérieur (#4800)

Source: https://storybook.systeme-de-design.gouv.fr/?path=/docs/modal--docs

* Rework page d'accueil [WIP] (#4679)

* feat: rework page d'accueil

* refactor: amélioration de la section "Comment ça marche?"

* fix: test 2e2 home-page

* feat: divise la page Home en sections

* feat: ajoute une section faq

* refactor: home / hero-section

* refactor: home page

* refactor: déplace et refacto du bouton d'accès à la liste des aides

* refactor: hero-section

* feat: ajoute les liens des partenaires

* fix: accessibilité - axe

* fix: lint

* style: améliore le dégradé du titre de la hero-section

* style: supprime les bullet points des balises <li>

* feat: ajoute la recherche par mot clef dans la liste des aides via un query param + màj liste des aides (section benefit showcase)

* refactor: et => et/ou

* feat: cache certaines sections de la page d'accueil dans l'iframe

* refactor: label bouton d'accès à la liste des aides

* refactor: label et contenu dans la section "service public"

* refactor: suppr. sous-descriptif section "comment ça marche ?"

* feat: ajoute du lazy loading sur les images de la section "partenaires"
https://developer.mozilla.org/fr/docs/Web/HTML/Element/img#attr-loading

* refactor: optimisation images section partenaires - utilisation extension webP + fallback en jpg

* refactor: découpage logo jeunes.gouv.fr

* test suppression section partenaires

* test sans le gif de présentation

* test sans le gif de présentation et sans la section partenaires

* chore: réactive les sections partenaires et showcase

* feat: ajoute lien vers page confidentialité + ajout back-button (confidentialite.vue)

* refactor: mise à jour liens (features-section.vue)

* refactor: mise à jour liens (features-section.vue)

* refactor: contenu (features-section.vue)

* refactor: contenu (features-section.vue)

* refactor: Ajoute les liens vers la CSS et l'aide Départ 18-25 (showcase page d'accueil)

* refactor: ordre des liens (showcase page d'accueil)

* refactor: accessibilité liens (showcase page d'accueil)

* refactor: réintroduit le terme "service public" (features-section.vue)

* refactor: retire le point d'accompagnement dans le section "Comment ça marche?"

* refactor: met à jour la section des partenaires

* refactor: met à jour des logos section des partenaires

* refactor: renommage méthode (rebase avec main)

* refactor: formations -> formation

* refactor: réordonner les logos des partenaires

* refactor: amélioration images JPEG (partenaires)

* chore: désactive les partenaires liés à des communes

* refactor: désactive partenaire rdv service-public et ajoute un commentaire

* Améliore la disposition des logos - section partenaires de la page d'accueil (#4815)

* refactor: améliore la disposition des logos (section partenaires de la page d'accueil)

* refactor: taille des logos des partenaires réduite au format smartphone

* Mise à jour des dépendances de Mongo (#4740)

* chore: maj des dépendances mongo

* fix: breacking changes de la maj de mongo

* fix: erreur d'instanciation avec mongo

* feat: passage en mode async/await pour la nouvelle version de mongoose

* ajout d'un mock autour des survey

* feat: mise à jour de la pertinance des tests

* mise à jour du test

* Met à jour les aides AVF et AVE de la Caf de la Gironde. (#4808)

* Met à jour les aides BAFA de a Caf de Touraine (Indre-et-Loire) (#4807)

* feat: migration jest => vitest

* feat: rajoute les tests historiquement exclus

* chore: update package-lock.json

* test:  ajuste la vérification de taille de la description - aide au covoiturage (temporaire)

* tryfix: cd - unit tests

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Yasmine Berrada <yasmineberrada91@gmail.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Jeremy PASTOURET <pastouret.jeremy@gmail.com>
Co-authored-by: Emile Rolley <emile.rolley@tuta.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Shamzic Shamzic Shamzic approved these changes

@yasmine-glitch yasmine-glitch yasmine-glitch approved these changes

Assignees

@jenovateurs jenovateurs

Labels
None yet
Projects
None yet
Milestone
BC janv-février 2025
Development

Successfully merging this pull request may close these issues.

Mieux gérer le test de vérification geographique
3 participants
@jenovateurs @yasmine-glitch @Shamzic