← Retour aux issues

Ajout catégorie "horaires manquants"

publié le , mis à jour

⚙️ Ceci est une proposition de changement de code.

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

Avatar Codeberg de LySioSLySioS

uniquement les commerces pour commencer

l'icône associée est là https://codeberg.org/cartes/web/pulls/1806

  1. Avatar Codeberg de etienneJretienneJr

    merci d'avoir lancé l'action ! J'ai récupéré l'icone et fait les modifs idoines pour que le script qui rajoute le fond coloré fonctionne. Et dans la requête overpass, j'ai enlevé les shop=vacant. ça donne ça (à un endroit que tu devrais reconnaitre ? 😉​ ) image

  2. Avatar Codeberg de etienneJretienneJr

    ça me parait bon, je fusionne.

  3. Avatar Codeberg de LySioSLySioS

    Nickel !

    je reconnais la forme de la place, sinon, j'aime pas trop aller à Perpi, encore moins la cartographier, c'est trop gros et trop dense :)

  4. Avatar Codeberg de pmiossecpmiossec

    Avoir rajouté cette categorie a un effet de bord négatif.

    Pour https://cartes.app/?cat=demandez-angela&allez=Holy%2BCut%7Cn13227707095%7C-0.57109%7C44.83269#15/44.83269/-0.57109/15/40

    le type affiché en haut ne devrait pas être "horaire manquant" mais soit "Demander Angela" car c'est la categorie selectionnée, soit "Coiffeur" car c'est le vrai type.

    D'ailleur, même probleme sans selectionner de categorie: https://cartes.app/?allez=Holy+Cut%7Cn13227707095%7C-0.57106%7C44.83268#17/44.83268/-0.57106/15/40

  5. Avatar Codeberg de etienneJretienneJr

    @pmiossec aaaahhh zut je m'étais douté en le faisant que ça pourrait arriver. Mais ça ne s'était pas produit sur les qq lieux que j'avais testés, donc je ne m'étais pas plus inquiété. J'aurais dû tester plus.

    La fonction findCategory(tags) est ici. Je pense que ça solution sera de soustraire de categories les catégories dont on ne veut pas qu'elles soient trouvées. https://codeberg.org/cartes/web/src/commit/da2e14786431c7576cbe8d3643c87409258797cc/components/categories.ts#L116-L172

    D'ailleurs, même probleme sans sélectionner de catégorie

    le type affiché en haut ne devrait pas être "horaire manquant" mais soit "Demander Angela"

    normal, car pour l'instant la catégorie sélectionnée n'est pas prise en compte. Ce sont uniquement les tags qui servent à recalculer la catégorie. Quand il y a 1 seule catégorie sélectionnée, ça serait malin de la récupérer en effet. Quand il y a plusieurs catégories sélectionnées, ça se complique ...

  6. Avatar Codeberg de pmiossecpmiossec

    @etienneJr wrote in https://codeberg.org/cartes/web/pulls/1807#issuecomment-11839644:

    normal, car pour l'instant la catégorie sélectionnée n'est pas prise en compte. Ce sont uniquement les tags qui servent à recalculer la catégorie. Quand il y a 1 seule catégorie sélectionnée, ça serait malin de la récupérer en effet. Quand il y a plusieurs catégories sélectionnées, ça se complique ...

    En y (re)reflechissant, la catégorie sélectionné ne devrait pas être prise en compte donc le calcul avec les tags actuel est bon (juste le bug à corriger). Car quand on ouvre la fiche lieu, on veut vraiment voir le type du POI pas une caractéristique qu'il a même si c'est celle qui nous a permis de le trouver...

  7. Avatar Codeberg de laemlaem

    Trop stylée comme idée :)

  8. Avatar Codeberg de laemlaem

    Au passage, sachez que hier j'ai fait un changement pour afficher les catégories cachées dans l'annuaire /lieux.

    C'est pour l'indexation.

  9. Avatar Codeberg de laemlaem

    Détrompez-moi mais cette catégorie "sans horaires" aujourd'hui elle n'est pas découvrable, si ?

  10. Avatar Codeberg de etienneJretienneJr

    Si si, en cherchant "horaire" dans la barre de recherche.

  11. Avatar Codeberg de pmiossecpmiossec

    Mince, le fait d'avoir renseigné le 'website' dans OSM a changé l'affichage 🫤

  12. Avatar Codeberg de LySioSLySioS

    @pmiossec wrote in https://codeberg.org/cartes/web/pulls/1807#issuecomment-11862150:

    Mince, le fait d'avoir renseigné le 'website' dans OSM a changé l'affichage :face_with_diagonal_mouth:

    C'est pcq tu as supprimé la clé shop car repair

    L'analyse ne fonctionne que sur les shops

  13. Avatar Codeberg de pmiossecpmiossec

    @LySioS wrote in https://codeberg.org/cartes/web/pulls/1807#issuecomment-11864505:

    C'est pcq tu as supprimé la clé shop car repair

    👍 merci d'avoir reparé: je ne sais pas comment c'est possible car j'ai normalement juste ajouté un tag 🤔

    Par contre je sais que l'interface OSM a un serieux problème d'UX avec la fonctionnalité d'autocompletion aussi bien pour les tags que le commentaire. Il me completait toujours en website:menu qd je voulais juste website. J'ai failli péter un cable car sur mobile c'est extremement penible à contourner!

  14. Avatar Codeberg de LySioSLySioS

    Idem, j'ai fait la correction sur mobile, meme gageure.

    je ne sais pas comment c'est possible car j'ai normalement juste ajouté un tag

    ID n'est pas assez bien pensé, ça permet voire encourage trop d'erreurs de saisie. Je ne parle meme pas des champs évidents manquants et des horaires 😵

  15. Avatar Codeberg de etienneJretienneJr

    @pmiossec wrote in https://codeberg.org/cartes/web/pulls/1807#issuecomment-11831451:

    Avoir rajouté cette categorie a un effet de bord négatif.

    c'est corrigé dans #1840, pour les 2 catégories transversales Demandez Angela, et Horaires manquants, elles ne peuvent plus être trouvées à partir des tags. Si d'autres catégories sont concernées, il faut rajouter cannot-be-found-from-tags: true dans categories.yaml

  16. Avatar Codeberg de LySioSLySioS

    c'est noté, je pensais à faire un filtre sur les produits bio (resto, supermarché, boulangerie, etc) j'y penserai


✏️ Participer à la discussion