← Retour aux issues

Edition OSM: Possibilité d'ajouter un fixme OSM

publié le , mis à jour

⚙️ Ceci est une proposition de changement de code.

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

Avatar Codeberg de pmiossecpmiossec
  • plus de place pour éditer le texte
  • ajout de la fonctionnalité de placeholder
  • autofocus sur le champ d'édition

image

image

image

  1. Avatar Codeberg de etienneJretienneJr

    Hum... ça ne me parait pas adapté, et trompeur. Là ça va ajouter un tag note sur l'élément OSM qu'on est en train d'éditer. Alors que dans le jargon OSM, on s'attendrait à ce que ça crée un élément note aux coordonnées indiquées. Les éléments note sont bien prévus pour enregistrer des améliorations à faire, donc sont bien suivis par les fan de la résolution des notes. Mais le tag note est plutôt prévu pour enregistrer une explication/justification des autres tags de l'élément, pour éviter qu'un contributeur ultérieur remette une mauvaise valeur. Par exemples : https://www.openstreetmap.org/way/785865648 https://www.openstreetmap.org/way/46037554 Il n'est donc pas utilisé pour enregistrer une amélioration à faire, et n'est donc pas suivi par la communauté. Et d'une manière générale, le tag note est à utiliser en dernier recours quand on ne peut vraiment pas utiliser un vrai tag. Donc je pense qu'il ne faut pas autoriser sa modification depuis cartes.app

    En revanche, si t'es motivé, tu peux regarder comment on pourrait poster des notes à une coordonnée donnée. Par exemple après un clic sur la carte. ça doit juste être l'endpoint de l'api qui est différent. Dans un premier temps seulement si l'utilisateur est connecté à son compte OSM. On réfléchira plus tard à comment récupérer de l'info géographique d'utilisateurs anonymes.

  2. Avatar Codeberg de laemlaem

    si t'es motivé, tu peux regarder comment on pourrait poster des notes à une coordonnée donnée. Par exemple après un clic sur la carte.

    Tu veux dire, pour contribuer à un objet pas encore créé, parce que cartes.app ne permet pas encore de créer de lieu ? Dans ce cas, mieux vaut mettre l'effort sur la création de lieu je pense. C'est pas très compliqué, juste un post de l'API et surtout la petit interface qui permet de choisir un type de lieu, soit plus ou moins notre menu catégories :)

  3. Avatar Codeberg de laemlaem

    Et ensuite on ajoutera, si le lieu n'existe pas dans nos catégories : "déposer une note".

  4. Avatar Codeberg de pmiossecpmiossec

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

    Là ça va ajouter un tag note sur l'élément OSM qu'on est en train d'éditer. Alors que dans le jargon OSM, on s'attendrait à ce que ça crée un élément note aux coordonnées indiquées.

    Je ne suis pas un expert OSM donc je veux bien croire que la sémantique et les usages ont déviés car pour moi, la doc indique clairement que ça peut être utilisé pour donner des infos pour une amélioration ("or hints for further improvement."): https://wiki.openstreetmap.org/wiki/Key:note

    Je trouve cela bizarre la pratique de créer un nouvel élément dissocié de l'objet que tu veux améliorer mais soit...

  5. Avatar Codeberg de etienneJretienneJr

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

    la doc indique clairement que ça peut être utilisé pour donner des infos pour une amélioration

    ah oui le wiki ne correspond pas à l'idée que je m'en faisais, j'ai posé la question sur le forum https://forum.openstreetmap.fr/t/des-notes-osm-via-cartes-app/40889/8

  6. Avatar Codeberg de pmiossecpmiossec

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

    ah oui le wiki ne correspond pas à l'idée que je m'en faisais, j'ai posé la question sur le forum https://forum.openstreetmap.fr/t/des-notes-osm-via-cartes-app/40889/8

    J'ai vu les réponses et du coup, j'ai adapté en 'fix me':

    image

  7. Avatar Codeberg de etienneJretienneJr

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

    J'ai vu les réponses et du coup, j'ai adapté en 'fix me':

    Très bien ! Comme notre éditeur est limité (pour l'instant du moins), on va probablement récolter par ce tag des modifications basiques qui seraient/seront très facilement résolues avec un éditeur plus complet, iD par exemple. Donc il faudrait :

    • dans le message rajouter un truc du genre "accédez directement à plus d'options d'édition sur iD" avec le lien direct d'édition.
    • Pour pouvoir suivre plus facilement ces fixme (et par exemple mettre en place un challenge maproulette), il faudrait ajouter un #cartesapp au texte avant de poster.

    OK ?

    J'ai un doute sur A régler mais je n'ai pas d'autre proposition...

  8. Avatar Codeberg de pmiossecpmiossec

    Qu'est-ce que tu en penses @etienneJr de proposer des phrases types pour faciliter l'utilisation quand on n'a pas trop le temps ou quand on est sur mobile...

    image

  9. Avatar Codeberg de etienneJretienneJr

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

    Qu'est-ce que tu en penses @etienneJr de proposer des phrases types pour faciliter l'utilisation quand on n'a pas trop le temps ou quand on est sur mobile...

    J'hésite... d'un côté je trouve que c'est une bonne idée de proposer des phrases types pour faciliter. D'un autre côté, le risque est grand que ce fixme ne soit jamais traité par quelqu'un, d'autant plus si la phrase type n'apporte aucune information utilisable. Il faudrait forcer à mettre une information en plus de la phrase type ? ou bien montrer qu'on attend un complément ?

    Dans tous les cas, pas de phrase type sur les horaires, pour pousser à modifier directement les horaires.

  10. Avatar Codeberg de pmiossecpmiossec

    @etienneJr j'ai retiré l'exemple sur l'horaire et j'ai rajouter un petit texte "(à compléter avec le plus de contexte possible)" Pour le reste, je ne sais pas comment inciter plus...

    image

    Si ça te semble suffisant, je pense qu'on pourrait merger et voir/suivre si c'est et comment c'est utilisé...

    On pourra débrancher facilement si ça pose problème...

  11. Avatar Codeberg de etienneJretienneJr

    C'est top ! Oui tu peux merger. Je pense faire un challenge maproulette pour le suivi.

  12. Avatar Codeberg de laemlaem

    Trop cool cette nouveauté, j'ai pas suivi mais hâte de l'utiliser.

  13. Avatar Codeberg de pmiossecpmiossec

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

    Trop cool cette nouveauté, j'ai pas suivi mais hâte de l'utiliser.

    Y'a un bug :/ Je pouvais pas la tester tant que c'était pas en ligne et je viens de voir que le #cartes.app qui normalement est ajouté normalement automatiquement à la valeur est bien affiché mais pas envoyé :/

    Et j'ai pas le temps de le fixer maintenant. Peut-être ce soir.... (avant ma semaine de vacances)

    /cc @etienneJr

  14. Avatar Codeberg de etienneJretienneJr

    ah zut ! Pour faire des tests, il y a une instance de dev d'OSM https://master.apis.dev.openstreetmap.org/ Elle est complètement indépendante de la prod, donc il faut se créer un compte OSM sur cette instance avant de pouvoir utiliser l'API correspondante. Mais au moins après on peut envoyer tous les changeset qu'on veut.

  15. Avatar Codeberg de pmiossecpmiossec

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

    ah zut ! Pour faire des tests, il y a une instance de dev d'OSM https://master.apis.dev.openstreetmap.org/

    pour cette fois-çi, je pense pas que j'ai le temps de tester ou mettre en place mais j'avais essayé de trouver un moyen mais c'est la redirection après l'authentification qui ne va pas: ça redirige vers 127.0.0.1 au lieu de localhost mais cartes.app ne marche pas sur 127.0.0.1 chez moi 🫤

  16. Avatar Codeberg de pmiossecpmiossec

    Corrigé: y'aura une amélioration à faire plus tard: retirer le champ fixme quand la valeur est vidée...


✏️ Participer à la discussion