← Retour aux issues

feat: rework the caterogies search with fuse.js

publié le , mis à jour

⚙️ Ceci est une proposition de changement de code.

Rendez-vous sur sa page Coderberg pour l'osculter.

  1. Avatar Codeberg de n4n5n4n5

    https://codeberg.org/cartes/web/pulls/2625#issuecomment-20761967

    Ça marche pour cet exemple, mais c'est parce que tu rends l'algorithme plus tolérant, tout simplement. Il se pourrait que ça fasse apparaitre des catégories qui n'ont que peu à voir (faux positifs), non ?

  2. Avatar Codeberg de laemlaem

    Ahlala, notre recherche côté client est vraiment pétée. Depuis quand on affiche un résultat religieux d'un côté, et un résultat de riches d'un autre, avant la catégorie recherchée ? J'ai vérifié, rien à voir avec les modifs récentes de Fuse. Ça fait longtemps donc.

    Y a du boulot 😓

    image

  3. Avatar Codeberg de etienneJretienneJr

    @laem wrote in https://codeberg.org/cartes/web/pulls/2717#issuecomment-21315650:

    Depuis quand on affiche un résultat religieux d'un côté, et un résultat de riches d'un autre, avant la catégorie recherchée ?

    il y aurait un fix tout simple pour ce cas précis, ça serait de les renommer en "halal" et "étoilé" sans le mot restaurant, qui est de toute façon inutile vu qu'on est dans le groupe restaurant et que l'icone est claire. Mais attention, ça va modifier la key de ces 2 catégories, donc il faudra modifier en même temps tous les fichiers de traduction. (c'était pour éviter ça que, il y a qq mois, j'avais proposé de définir des keys qui soient indépendantes du name...)

  4. Avatar Codeberg de laemlaem

    (dslé j'ai pas de temps pour ce sujet, j'aimerais bien bosser dessus, mais pas maintenant)

  5. Avatar Codeberg de n4n5n4n5

    Bon je suis entrain de ré-écrire cette MR

  6. Avatar Codeberg de n4n5n4n5

    Cleanup (type, refactor) https://codeberg.org/cartes/web/pulls/2999 avant une tentative

  7. Avatar Codeberg de n4n5n4n5

    un autre cleanup

    https://codeberg.org/cartes/web/pulls/3001

    je pense qu'il faudrait un CategoryMenu et un CategoryMenuSearch

    actuellement c'est les meme composant, et si je dis pas de bétises, les catégories sont triés par ordre de...couleur ?!

    Je trouve pas ça logique l'ordre de couleur sur une recherche de catégorie

  8. Avatar Codeberg de n4n5n4n5

    Exemple concret

    Je pense que ici,

    image

    Pour moi, ça devrait être comme ça: les catégories qui matchent le plus en haut (ceci est un croquis, pas codé..encore)

    image

    @etienneJr @laem vous être d'accord ?

  9. Avatar Codeberg de esmenardesmenard

    Avec auto-école tout en haut, c'est la seule catégorie dont le nom commence par auto

  10. Avatar Codeberg de n4n5n4n5

    @esmenard wrote in https://codeberg.org/cartes/web/pulls/2717#issuecomment-22685743:

    Avec auto-école tout en haut, c'est la seule catégorie dont le nom commence par auto

    Pour moi non, le nombre de match exact d'abord, ici les services ont deux catégories qui match

  11. Avatar Codeberg de n4n5n4n5

    C'est en ligne, il faudra pimper un peu le truc peut etre

    image

    Il faudra ensuite:

    • revoir fileredCategoriesMaxCount
    • mettre vraiement le fusing (virer les regex), qui etait le premier but de cette MR

    Et ça fix le pb de @laem

    image


✏️ Participer à la discussion