Skip to content

Add Track & Field presets#817

Merged
tyrasd merged 5 commits intoopenstreetmap:mainfrom
arch0345:athletics
Mar 14, 2023
Merged

Add Track & Field presets#817
tyrasd merged 5 commits intoopenstreetmap:mainfrom
arch0345:athletics

Conversation

@arch0345
Copy link
Copy Markdown
Contributor

@arch0345 arch0345 commented Mar 13, 2023

Adds presets for the following Track & Field events:

  • Discus
  • Hammer Throw
  • High Jump
  • Javelin
  • Long Jump
  • Pole Vault
  • Running
  • Shot Put
  • Sprinting
  • Steeplechase
  • Triple Jump

I made the preset for athletics=running unsearchable as sport=running is more popular.

@github-actions
Copy link
Copy Markdown

🍱 Preview the tagging presets of this pull request here: https://pr-817--ideditor-presets-preview.netlify.app/id/dist/#locale=en.

@tyrasd tyrasd added new-preset new-field create a new field (see add-field for cases where field from presets is added to new entries) labels Mar 13, 2023
Comment thread data/presets/leisure/track/athletics/long_jump.json
Comment thread data/presets/leisure/pitch/athletics/shot_put.json
Also fixed file name for Sprint Runway preset and name for High Jump Pit preset
Copy link
Copy Markdown
Member

@tyrasd tyrasd left a comment

Choose a reason for hiding this comment

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

Maybe we should keep the generic Pitch/Track fields (like lit, access ) for the track & field subpresets (see suggested change below).

We should also add athletics as an option to the sport_racing_nonmotor field, shouldn't we?

Comment thread data/presets/leisure/track/_athletics.json
Comment thread data/presets/leisure/pitch/_athletics.json
@tyrasd tyrasd merged commit cab8278 into openstreetmap:main Mar 14, 2023
@arch0345 arch0345 deleted the athletics branch March 14, 2023 17:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new-field create a new field (see add-field for cases where field from presets is added to new entries) new-preset

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants