Skip to content

Add preset for man_made=geoglyph#2062

Merged
matkoniecz merged 2 commits intoopenstreetmap:mainfrom
ak8abhinay:add-man_made-geoglyph-preset
Apr 15, 2026
Merged

Add preset for man_made=geoglyph#2062
matkoniecz merged 2 commits intoopenstreetmap:mainfrom
ak8abhinay:add-man_made-geoglyph-preset

Conversation

@ak8abhinay
Copy link
Copy Markdown
Contributor

Adds a new preset for man_made=geoglyph.

wiki - https://wiki.openstreetmap.org/wiki/Tag:man_made%3Dgeoglyph

Fixes #14

@github-actions
Copy link
Copy Markdown

🍱 Your pull request preview is ready

Please use this preview to check your changes. Ideally use the test documentation template and document your test results by commenting on the PR. This will speed up the review process for everyone.

FYI, once this PR is merged, you can use the iD Editor Preview to test your changes in interaction with all other changes.

@ak8abhinay
Copy link
Copy Markdown
Contributor Author

Here is a screenshot of the preset working in the iD editor,

image

Comment thread data/presets/man_made/geoglyph.json
@matkoniecz matkoniecz added the waitfor-requested-changes This PR is waiting for an update to incorporate requested changes from a PR review. label Mar 24, 2026
Copy link
Copy Markdown
Collaborator

@matkoniecz matkoniecz left a comment

Choose a reason for hiding this comment

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

missing icon is sad but I do not really have an idea for one

Image Image Image

@matkoniecz matkoniecz removed the waitfor-requested-changes This PR is waiting for an update to incorporate requested changes from a PR review. label Apr 12, 2026
@matkoniecz matkoniecz merged commit a41228c into openstreetmap:main Apr 15, 2026
5 checks passed
@matkoniecz
Copy link
Copy Markdown
Collaborator

Thanks for the PR! It is now merged but note that before you will see it in iD few things need to happen.

iD tagging schema needs to get release (see https://github.com/openstreetmap/id-tagging-schema/releases - latest one mentioned there is just a draft). And later needs to be pulled in by iD.

Thanks again for your contribution! If you are interested in making other one it would be really welcome!

@matkoniecz matkoniecz changed the title Add preset for man_made=geoglyph Add preset for man_made=geoglyph Apr 15, 2026
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.

Consider adding man_made=geoglyph to presets

3 participants