Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Corrige les régressions sur la mutualisation du code de recherche et la génération des sitemaps #220

Merged
merged 5 commits into from
May 18, 2021

Conversation

fcamblor
Copy link
Collaborator

Fix pour une régression introduite entre #217 et #194 sur du code censé être mutualisé entre la partie applicative et la partie tooling

@fcamblor fcamblor requested review from bilelz and Luwangel May 17, 2021 21:32
@fcamblor
Copy link
Collaborator Author

J'avais un problème de compilation jest qui avait du mal à charger les fichiers .mjs sous forme de modules.

Après m'être pas mal cassé les dents sur ce problème, j'ai finalement fait en sorte de convertir le communes-import.mjs en un fichier Typescript, ce qui a permis de ne plus avoir du tout de fichiers mjs (tout est alors typescript, ce qui est finalement bien plus simple :-) )

@fcamblor
Copy link
Collaborator Author

Je suis également tombé sur un bug dans la génération des sitemaps de #194 (2 urls étaient positionnées dans les sitemaps département, au lieu de 2 entrées)

Copy link
Collaborator

@Luwangel Luwangel left a comment

Choose a reason for hiding this comment

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

Merci pour ce gros boulot 👍
En effet avec les différents rebase j'étais passé à côté de l'harmonisation d'une des méthodes utilitaires.
Tu as également bien fait de convertir tout en ts, c'est bien plus simple de tout harmoniser.
Bref, c'est top :)

@Luwangel Luwangel changed the title Mutualisation code de recherche Corrige les régressions sur la mutualisation du code de recherche et la génération des sitemaps May 18, 2021
@Luwangel Luwangel merged commit c1dccd8 into dev May 18, 2021
@Luwangel Luwangel deleted the fix-regression-for-search-terms branch May 18, 2021 08:31
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants