[FRONT] Fix advanced search : add type collectivite GRP#480
Merged
jb-delafosse merged 3 commits intomainfrom Dec 12, 2025
Merged
[FRONT] Fix advanced search : add type collectivite GRP#480jb-delafosse merged 3 commits intomainfrom
jb-delafosse merged 3 commits intomainfrom
Conversation
Contributor
✅ Review app deployedCheck how your changes look like!
|
jb-delafosse
requested changes
Dec 12, 2025
front/utils/utils.ts
Outdated
| @@ -194,6 +194,7 @@ export function stringifyCommunityType(type: CommunityType): string { | |||
| if (type === CommunityType.EPT) return 'Établissement public territorial'; | |||
| if (type === CommunityType.Metropole) return 'Groupement'; | |||
Collaborator
There was a problem hiding this comment.
Peut être supprimé : on ne retounera jamais "MET, CTU, CA, CC et EPT", toutes ces subtilités sont dans une seule catégorie GRP pour groupement.
Si on fait un peu l'historique :
- 10 Mars 2025 - [FRONT] Select community type for map filtering #60 : le frontend a tous les types avec toute la granularité
- 30 Mars 2025 - [BACK] Reformat OFGL #110 : le backend map tous les types détaillé vers MET
- 7 aout 2025 : [BACK] Refactor/ofgl lazy #346 : REfacto technique coté back, pas de changement sur les types.
- 3 Sept 2025 : [BACK] Met to GRP #415 : On envoie tous les types sur GRP et plus MET coté back.
| /** Etablissement public territorial */ | ||
| EPT = 'EPT', | ||
| /** Groupement */ | ||
| GRP = 'GRP', |
Collaborator
There was a problem hiding this comment.
Idem tu peux supprimer les types pas utilisé imo.
jb-delafosse
approved these changes
Dec 12, 2025
Collaborator
jb-delafosse
left a comment
jb-delafosse
left a comment
There was a problem hiding this comment.
Good Job ! App is back on !
jb-delafosse
force-pushed
the
fix-advanced-search
branch
from
9fadf19 to
ef81436
Compare
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Ajout d'un nouveau type de collectivité : Groupement
Il y a l'air d'avoir eu une mise à jour. La répartition de la table collectivité par type donne ça maintenant. Il y a beaucoup moins de diversité. Je me suis contenté d'ajouter le nouveau type.
97 DEP
34823 COM
17 REG
1266 GRP