Skip to content

Conversation

ilselvaggio
Copy link

@matheusgomesms
Copy link
Collaborator

Thank you!

Is this layer working? I didn't manage to make it work on iD, JOSM, or even in their GeoServer.

"attribution": {
"url": "https://geoservices.buergernetz.bz.it/geokatalog",
"text": "© Autonome Provinz Bozen/Provincia Autonoma di Bolzano CC-BY 4.0",
"required": false
Copy link
Collaborator

Choose a reason for hiding this comment

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

if the data is CC BY then attribution is required so should be required: true

Copy link
Collaborator

@andrewharvey andrewharvey left a comment

Choose a reason for hiding this comment

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

Thanks for the PR and apologies it's been missed.

The service is working for me, but it looks like it would work better as an overlay above imagery rather than a background map. Could you set overlay: true? Per the schema at

"overlay": {

"name": "South Tyrol Cadastre",
"start_date": "2024-09-02",
"end_date": "2024-09-02",
"best": true
Copy link
Collaborator

Choose a reason for hiding this comment

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

Could we remove best: true this should be for imagery layers only.

"EPSG:32632"
],
"type": "wms",
"category": "photo",
Copy link
Collaborator

Choose a reason for hiding this comment

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

could you change the category to either map or other since it's not an orthophoto. https://github.com/osmlab/editor-layer-index/blob/gh-pages/CONTRIBUTING.md#categories

"id": "SouthTyrol-Cadastre",
"name": "South Tyrol Cadastre",
"start_date": "2024-09-02",
"end_date": "2024-09-02",
Copy link
Collaborator

Choose a reason for hiding this comment

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

If the data is continually updated, best leave off the start/end dates. Unless you know the data is fixed at this time period?

@andrewharvey
Copy link
Collaborator

Could you also please remind me where the CC BY waiver is?

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.

3 participants