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

Additional fields for crossing=traffic_signals presets #453

Merged
merged 1 commit into from
May 12, 2022

Conversation

arch0345
Copy link
Contributor

@arch0345 arch0345 commented May 9, 2022

Adds fields for:

based on keys listed on this wiki article. I added the latter 3 to moreFields since they're not as commonly used as the former.

Adds fields for:
* `button_operated`
* `traffic_signals:arrow`
* `traffic_signals:countdown`
* `traffic_signals:minimap`
* `traffic_signals:sound`
* `traffic_signals:vibration`
@tyrasd tyrasd merged commit 158e071 into openstreetmap:main May 12, 2022
@tyrasd
Copy link
Member

tyrasd commented May 12, 2022

Thanks!

@arch0345 arch0345 deleted the traffic-signals branch May 12, 2022 19:30
Copy link
Contributor

@1ec5 1ec5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is mostly belated copyediting for clarity, but one of the strings has it backwards. (Sorry for the delay; I only saw this now because the strings showed up in Transifex.)

data/fields/button_operated.json Show resolved Hide resolved
data/fields/traffic_signals/arrow.json Show resolved Hide resolved
data/fields/traffic_signals/countdown.json Show resolved Hide resolved
data/fields/traffic_signals/sound.json Show resolved Hide resolved
data/fields/traffic_signals/sound.json Show resolved Hide resolved
data/fields/traffic_signals/sound.json Show resolved Hide resolved
data/fields/traffic_signals/sound.json Show resolved Hide resolved
arch0345 added a commit to arch0345/id-tagging-schema that referenced this pull request May 14, 2022
@arch0345
Copy link
Contributor Author

Thanks for your feedback @1ec5, I've opened up #456 to resolve these issues

@Dimitar5555
Copy link
Contributor

It looks like the new fields are not linked correctly. None of them show up in the iD instance on openstreetmap.org nor in version that is published on https://ideditor.org/latest-id/. These are the messages that appear in the console and they are mostly identical for all types of crossings.
image

tyrasd added a commit that referenced this pull request May 27, 2022
tyrasd added a commit that referenced this pull request Sep 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants