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

Allow disused:type in Osmose #2094

Closed
wants to merge 1 commit into from
Closed

Conversation

Famlam
Copy link
Collaborator

@Famlam Famlam commented Dec 5, 2023

After writing my reply in issue 2093 I remembered we have a copy of the JOSM rule (with extra -osmoseDetail), so we can edit it ourselves. This does not fix the issue (different plugin), it's just related to it

p.s. this is JOSMs reasoning against it, they advocate state=* https://josm.openstreetmap.de/ticket/23194

Based on osm-fr#2093 (comment)

We have our own copy of JOSMs rule, with extra context (-osmoseDetail)
@frodrigo
Copy link
Member

frodrigo commented Dec 9, 2023

Not sure about this. There is few disused:type and not documented in the wiki.

@Famlam
Copy link
Collaborator Author

Famlam commented Dec 9, 2023

Ok, it seemed to me like a better approach to avoid more "validator bypasses" like type=disused:X. On the other hand, as long as JOSM keeps its rule, the issue will continue to exist.

@Famlam Famlam closed this Dec 9, 2023
@Famlam Famlam deleted the disused-type branch December 9, 2023 19:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants