search

laem / cartes

L'appli Web de cartes grand public
https://cartes.app
GNU Affero General Public License v3.0
104 stars 13 forks source link

Lien brand:wikadata Undefined #194

Closed cayenne17 closed 2 days ago

cayenne17 commented 4 months ago

Hello,

J'ai remarqué que sur les objets contenant un tag OSM brand:wikidata, il y a un lien.

Le lien pointe vers https://www.wikidata.org/wiki/Undefined Il devrait pointer vers https://www.wikidata.org/entity/$1 (voir : https://wiki.openstreetmap.org/wiki/Item:Q99)

laem commented 4 months ago

Pourrais-tu donner un exemple d'URL où cela survient ?

laem commented 4 months ago

Ou d'entité OSM

cayenne17 commented 4 months ago

@laem Total access : https://cartes.app/?lieu=w297421327 volvo : https://cartes.app/?lieu=w73079301 Domino's pizza : https://cartes.app/?lieu=n5578159363

(partout où il y a du brand:wikidata)

Jean-BaptisteC commented 1 month ago

https://github.com/laem/futureco/blob/master/app/voyage/useWikidata.ts

Jean-BaptisteC commented 3 weeks ago

Il semble que ces liens ne sont plus affichés sur les fiches des commerces

cayenne17 commented 3 weeks ago

@Jean-BaptisteC toujours présent image juste que ça pointe sur https://www.wikidata.org/wiki/Undefined au lieu de https://www.wikidata.org/wiki/Q3070922

cayenne17 commented 2 days ago

Je vois que maintenant ça emmène vers la bonne page.

@laem @Jean-BaptisteC : Est-ce qu'il y a eu quelque chose de fait ?

Je pense que le problème peut-être fermé

Jean-BaptisteC commented 2 days ago

Oui le correctif a été merge, l'issue peut être clôturé

fouss commented 2 days ago

My bad je suis un utilisateur de gitlab mais je ne maitrise pas trop le fonctionnement des PR, désolé si j'ai pas tout bien fait au bon endroit !

Jean-BaptisteC commented 2 days ago

Il manquait juste le fixes avant le numéro de l'issue dans la PR pour fermer l'issue automatiquement une fois qu'elle est merge :)