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

add reservoir_type=tailings #144

Merged
merged 2 commits into from Oct 17, 2022
Merged

Conversation

Marc-marc-marc
Copy link
Contributor

@Marc-marc-marc Marc-marc-marc commented Feb 27, 2021

add several combinaison with the depreciated tag reservoir_type=tailings

closes #142

add several combinaison with the depreciated tag reservoir_type=tailings
@tyrasd tyrasd changed the title add reservoir_type=tailings close #142 add reservoir_type=tailings Dec 14, 2021
tyrasd
tyrasd previously requested changes Dec 14, 2021
Copy link
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.

For this to work well, we also need a preset for the new man_made=tailings_pond tag. otherwise users would be presented with the rather generic preset for a "man made feature" after upgrading the tags. @Marc-marc-marc can you please add that here?

@tyrasd tyrasd added the enhancement New feature or request label Dec 14, 2021
@matkoniecz
Copy link
Contributor

can you please add that here?

I guess that it would be also OK with separate PR? If someone else would want to do this or @Marc-marc-marc is stuck with git/github interface?

@tyrasd
Copy link
Member

tyrasd commented Dec 16, 2021

👍 sure, for me it would be also fine if someone else took over (if it's ok with @Marc-marc-marc of course). Just base your PR/branch of from this one.

@tyrasd tyrasd added deprecating new-preset and removed enhancement New feature or request labels Jan 6, 2022
@tyrasd tyrasd linked an issue Jan 6, 2022 that may be closed by this pull request
@tyrasd tyrasd force-pushed the main branch 2 times, most recently from 9d3204d to 49f529e Compare June 22, 2022 16:19
tyrasd added a commit that referenced this pull request Oct 17, 2022
@tyrasd tyrasd removed the new-preset label Oct 17, 2022
@tyrasd tyrasd dismissed their stale review October 17, 2022 09:43

I've added the preset for this tag in 562c26f

@tyrasd tyrasd merged commit 0b9f443 into openstreetmap:main Oct 17, 2022
@Marc-marc-marc Marc-marc-marc deleted the patch-1 branch October 20, 2022 10:14
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.

Suggest tag upgrade for tailings ponds Suggest tag upgrade for tailings ponds
3 participants