Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Resolve Consistency with Tags #115

Closed
CurlingMan13 opened this issue Jan 26, 2023 · 3 comments
Closed

Resolve Consistency with Tags #115

CurlingMan13 opened this issue Jan 26, 2023 · 3 comments

Comments

@CurlingMan13
Copy link

There is an issue with being able to copy/paste the ONOSM.org generated notes into the ID editor. I really love the update, but there are some things that fall short.

For example:

  • The phone number tag gets populated with "Telephone=" but OSM uses "Phone="
  • The name tag shows up as "Name: ABC Business" but OSM requires "Name=ABC Business"
  • "Wheelchair Accessibility:" should be "wheelchair="
  • "Website: " should be "Website="
  • "Accepted payment: Visa,Mastercard,Cash" should instead be "payments:visa=yes", "payments:mastercard=yes", etc.

I'm sure I am missing a few others, but these little oversights prevent it from being straight copy-paste into iD (after verification, of course) as intended

@watmildon
Copy link

watmildon commented Feb 5, 2023

YES! This small bit of matching to OSM tags would make these notes much faster to take care of. Looking at the note generation here it seems some work has been done recently by @ArcaneDots which should help for some fields on new notes.

The rest that follow should be read skeptically as I'm not a JS dev at all so could/will be totally wrong.

It seems the intention is to have some portions of the notes be localized into other languages (all the i18n stuff). Presumably this is to the uploader can review the full text of the note. Some fields have already been converted to their equivalent OSM tag (addr:housenumber) so I don't really know how to evaluate if we'd be breaking some workflow considerations by doing away with the translations in the notes all together. Perhaps a happy medium is to present the fully translated note and then under it have all the mapped tag to copy and paste? What is the max size of a note?

Just some loose thoughts.


Right after I typed all that I found this comment on a pull request with similar intent. #99 (comment)

@ArcaneDots
Copy link
Contributor

It has already started the process of normalizing tag and needs to be updated with the recent merge. As @watmildon mentioned its comments include discussions about what fields need to added, validation and cleaned up (Issues #94, #46, #47 and #106) before this PR will be accepted.

@iandees
Copy link
Member

iandees commented Jan 29, 2024

I believe the note content is more consistent with OSM tagging now. As I've mentioned elsewhere the content of the notes will/should never be exactly copy-pasteable because the point is that OSM mappers should attempt to do some verification.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants