Service + Composant + Container pour la recherche commune / département

[Floby]

Abstract

Déplace l'intelligence de la recherche et des données sous-jacentes dans une classe sans fioriture attachée au State.
Réification de SearchRequest qui décrit les critère de recherche et de tri dans un type cohérent

See in action : https://dev.vitemado.se/148-merge/

Nouvelles fonctionnalités

Aucune

Régressions connues

Aucune 😅

Ce que ça fait

• Réutilisation d'un maximum de logique pour la recherche de la localisation, département comme commune.
• Tests sur ce composant autocomplete et le service associé
• création d'un composant "container" <vmd-search /> qui s'occupe d'injecter le bon service au composant <vmd-commune-or-departement-selector />
• notamment une fonction suggest: (prefix: string) => Promise<Departement | Commune> qui encapsule la logique de recherche. Dans les tests, juste un stub et c'est rigolo
• Memoization des requêtes d'autocomplete avec typescript-memoize qui marche mieux car l'implémentation actuelle ne fonctionnait pas pour des appels fait sur le même tick. (et aussi c'est plus clair ce que ça fait quand on le lit (et c'est testé donc c'est pas magique))
• réification des recherches dans un type SearchRequest = SearchRequestByCommune | SearchRequestByDepartement ça permet de mieux s'assurer de la complétude des implémentations au regard des 2 variantes car le compilateur râle. ça permet aussi de récupérer cet objet depuis <vmd-search /> aussi bien que de le désérialiser depuis l'url (ou de query params 👀 )
• ET DU COUP ENFIN SURTOUT PARCE QUE C'ETAIT ÇA LE SUJET A LA BASE : ça permet d'afficher des résultats de recherche même quand on n'utilise pas l'autocomplete 😅 pratique pour Créer un embed.html permettant d'embedder des résultats de VMD #122

Bonus

• Tests automatisés sur le composant <vmd-commune-or-departement-selector />
• Co-localisation des logiques de recherche textuelle/numérique et des optimisations sous-jacentes

(dans un prochain épisode)

• L'opportunité d'avoir d'autres moyens d'initier une recherche et ses résultats avec une service du type

type SearchService = (search: SearchRequest) => Promise<SearchResult>`

[fcamblor]

Pour info, j'ai rebase la branche par-dessus ma branche tomorrow_appointments, c'est dispo sur la branche features/rebased-component-autocomplete

J'ai l'impression que ça marche pas trop mal ... juste un doute si le click sur l'onglet chronodose n'est pas un peu plus long après le rebase qu'avant (note: pour tester, il faut soit modifier sa date systeme pour se positionner au 12/05, ou bien overrider la fonction fonctionaliteChronodosesActive() du vmd-rdv

[Floby]

je vais jeter un œil, j'ai moins de temps que prévu aujourd'hui :/

…

On Tue, 11 May 2021 at 03:30, Frédéric Camblor ***@***.***> wrote: Pour info, j'ai rebase la branche par-dessus ma branche tomorrow_appointments, c'est dispo sur la branche features/rebased-component-autocomplete J'ai l'impression que ça marche pas trop mal ... juste un doute si le click sur l'onglet chronodose n'est pas un peu plus long après le rebase qu'avant (note: pour tester, il faut soit modifier sa date systeme pour se positionner au 12/05, ou bien overrider la fonction fonctionaliteChronodosesActive() du vmd-rdv — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#148 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AABZQMUSJBBVXRLOR5I6VFDTNCCFBANCNFSM44BTTYMA> .