ideditor / schema-builder

🏗🏷 Create tagging schemas for iD
ISC License
12 stars 16 forks source link

Docs: Add details to `taginfoProjectInfo` #102

Closed tordans closed 1 year ago

tordans commented 1 year ago

This extends the docs for taginfoProjectInfo, based on the feedback in https://github.com/openstreetmap/id-tagging-schema/issues/919#issuecomment-1586022643


Closes https://github.com/openstreetmap/id-tagging-schema/issues/919

1ec5 commented 1 year ago

This is a good change, but people looking at the iD Tagging Schema project in taginfo won’t know to look in this repository. If this build script appends the decoder ring to the description property, people would see it directly inside taginfo:

https://github.com/ideditor/schema-builder/blob/8f6c4ee3b3f525892b33315e211fe925011b53e5/lib/build.js#L491

tordans commented 1 year ago

@1ec5 agreed, but lets move your comment in a new issue; they can be handled separately, right?

1ec5 commented 1 year ago

What’s the purpose of putting the decoder ring here, in the documentation about build options, if it only matters to people who view the project file in taginfo?

tyrasd commented 1 year ago

If this build script appends the decoder ring to the description property, people would see it directly inside taginfo:

implemented in c0f3fb5f742ae42c86280fc428606f9b2b2a3a0b