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.
uniquement les commerces pour commencer
l'icône associée est là https://codeberg.org/cartes/web/pulls/1806
etienneJr
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 ? 😉 )etienneJr
ça me parait bon, je fusionne.
LySioS
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 :)
pmiossec
Avoir rajouté cette categorie a un effet de bord négatif.
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
etienneJr
@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 decategoriesles catégories dont on ne veut pas qu'elles soient trouvées. https://codeberg.org/cartes/web/src/commit/da2e14786431c7576cbe8d3643c87409258797cc/components/categories.ts#L116-L172D'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 ...
pmiossec
@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...
laem
Trop stylée comme idée :)
laem
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.
laem
Détrompez-moi mais cette catégorie "sans horaires" aujourd'hui elle n'est pas découvrable, si ?
etienneJr
Si si, en cherchant "horaire" dans la barre de recherche.
pmiossec
Un autre cas avec notre cher https://www.radiateurs-carre.fr/ et pourtant les horaires sont renseignés https://cartes.app/?allez=Radiateurs%20Carr%C3%A9|w1476184415|6.81489|47.63427#17.5/47.634261/6.814889
pmiossec
Mince, le fait d'avoir renseigné le 'website' dans OSM a changé l'affichage 🫤
LySioS
@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
pmiossec
@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:menuqd je voulais justewebsite. J'ai failli péter un cable car sur mobile c'est extremement penible à contourner!LySioS
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 😵
etienneJr
@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, etHoraires manquants, elles ne peuvent plus être trouvées à partir des tags. Si d'autres catégories sont concernées, il faut rajoutercannot-be-found-from-tags: truedanscategories.yamlLySioS
c'est noté, je pensais à faire un filtre sur les produits bio (resto, supermarché, boulangerie, etc) j'y penserai