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
Poland geoportal update #1063
Poland geoportal update #1063
Conversation
@rbuffat could you help with this failed check? I'm having trouble understanding what the problem is. When trying the test locally I get no errors: |
@rbuffat OK. Thanks! In that case since this is not an error on my side I'm requesting a merge |
@rbuffat but over here something is still wrong |
WMS Access constraints:
|
I do not see what is wrong with that, could you elaborate on your replay? According to my information we can use these layers. |
@Asteliks Nothing wrong. I just try to document the situation that hopefully in X years it is possible to understand the context regarding license etc. See also: #960 (comment) and https://wiki.openstreetmap.org/wiki/Geoportal.gov.pl https://www.geoportal.gov.pl/regulamin:
|
Thanks @Asteliks ! |
And thank you too @rbuffat |
Thanks for adding this, but all the descriptions and names should be in English, please fix it. |
All the layer names have been settled with the management of Geoportal and can not be changed under any circumstances. They need to be able to find these layers in our system with ease and by the names given to us. |
Don't worry, I will translate them into Polish like they are now. But for other languages, we have to provide English text to translate - translations into other languages are from English only.
|
Well, what can I tell you. You are a bit lite for the party, the changes have already been merged. This could be updated in the next PR. As for the descriptions they are also required the way they are. |
@maro-21 This sounds like a reasonable suggestion. Would you be willing to make a PR for this? Otherwise, I could try to create one based on your suggestions. |
Note for future changes: |
This PR solves issue #1060 and adds three new sources form Geoportal 2: