Skip to content

Add translatable strings for Swimming Pool field#1008

Merged
tyrasd merged 3 commits intoopenstreetmap:mainfrom
arch0345:pool
Feb 5, 2024
Merged

Add translatable strings for Swimming Pool field#1008
tyrasd merged 3 commits intoopenstreetmap:mainfrom
arch0345:pool

Conversation

@arch0345
Copy link
Copy Markdown
Contributor

@arch0345 arch0345 commented Sep 20, 2023

Currently, the swimming pool field is only being used for the Swimming Pool preset, however the most common values for swimming_pool=* are ones that aren't relevant for its use in combination with leisure=swimming_pool (ie yes, no, outdoor, indoor).

This PR also adds a preset for leisure=hot_tub and adds a deprecation rule for leisure=swimming_pool + swimming_pool=hot_tub

Also added translatable strings for Swimming Pool field
@github-actions
Copy link
Copy Markdown

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

@arch0345
Copy link
Copy Markdown
Contributor Author

Hey @tyrasd, would you be able to review this PR?

@tyrasd tyrasd added new-value adds value(s) to existing field deprecating labels Feb 5, 2024
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.

looks good to me, thanks 👍

@tyrasd tyrasd merged commit b526b49 into openstreetmap:main Feb 5, 2024
@arch0345 arch0345 deleted the pool branch February 5, 2024 16:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

deprecating new-preset new-value adds value(s) to existing field

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants