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

Is PDOK Kaart backwards compatible? #170

Open
J-kad opened this issue Apr 8, 2015 · 20 comments
Open

Is PDOK Kaart backwards compatible? #170

J-kad opened this issue Apr 8, 2015 · 20 comments

Comments

@J-kad
Copy link

J-kad commented Apr 8, 2015

Goedemiddag,

Is de laatste versie van PDOK Kaart backwards compatible met de eerder versie van PDOK Kaart (versie van PDOK)?

Alvast bedankt!

@rduivenvoorde
Copy link
Collaborator

Hoewel er niet specifiek op gelet is, verwacht ik het wel.

Laat ajb weten wanneer het niet zo is, dan kunnen we dat eventueel nog fixen, zodat dat wel zo blijft.

@J-kad
Copy link
Author

J-kad commented Apr 9, 2015

Wij zullen er naar kijken en laten weten als het niet zo is. Bedankt voor je antwoord!

@J-kad
Copy link
Author

J-kad commented May 22, 2015

Het lijkt er helaas op dat de laatste versie van PDOK Kaart niet backward compatible is. De test is als volgt uitgevoerd:

Wij hebben de nieuwe versie op een test-omgeving gezet. Op de huidige versie (kaart.pdok.nl) hebben wij daarna een kaart gegenereerd met 1 extra laag (natura2000). De URL die gegenereerd wordt is aangepast naar de URL van de testversie. Vervolgens zie je wel een kaartje van NL, maar zonder zoommogelijkheid en keuze voor kaartlagen.

Kan dit aangepast worden?

Alvast bedankt!

@ndkv
Copy link
Contributor

ndkv commented May 22, 2015

Zie je dit:
http://geonovum.github.io/pdokkaart/api/api.html?zoom=2&baselayer=BRT&loc=142500%2C%20470000&pdoklayers=BRT%2CBRTTIJDELIJK%2CBRTGRIJSTIJDELIJK%2CBRTPASTEL%2CLUFO%2CNATURA2000_WMS&markersdef=http%3A%2F%2Fkaart.pdok.nl%2Fapi%2Fjs%2Fpdok-markers.js&layersdef=http%3A%2F%2Fkaart.pdok.nl%2Fapi%2Fjs%2Fpdok-layers.js

On 5/22/2015 2:33 PM, Kaart01 wrote:

Het lijkt er helaas op dat de laatste versie van PDOK Kaart niet
backward compatible is. De test is als volgt uitgevoerd:

Wij hebben de nieuwe versie op een test-omgeving gezet. Op de huidige
versie (kaart.pdok.nl) hebben wij daarna een kaart gegenereerd met 1
extra laag (natura2000). De URL die gegenereerd wordt is aangepast
naar de URL van de testversie. Vervolgens zie je wel een kaartje van
NL, maar zonder zoommogelijkheid en keuze voor kaartlagen.

Kan dit aangepast worden?


Reply to this email directly or view it on GitHub
#170 (comment).

@J-kad
Copy link
Author

J-kad commented May 26, 2015

Ja dat krijg je inderdaad te zien, zonder extra (kaart) mogelijkheden.

@ndkv
Copy link
Contributor

ndkv commented May 26, 2015

En welke commit heb je in testomgeving gezet?

On 26 May 2015 09:10:13 CEST, Kaart01 [email protected] wrote:

Ja dat krijg je inderdaad te zien, zonder extra (kaart) mogelijkheden.


Reply to this email directly or view it on GitHub:
#170 (comment)

Sent from my Android device with K-9 Mail. Please excuse my brevity.

@J-kad
Copy link
Author

J-kad commented May 26, 2015

De commit van 31 maart 2015

@rduivenvoorde rduivenvoorde reopened this Jun 10, 2015
rduivenvoorde added a commit that referenced this issue Jun 10, 2015
@rduivenvoorde
Copy link
Collaborator

Ok, dit is een keiharde bug, die gewoon nooit is opgemerkt...

Gefixed in: 70c3d0a

Ik heb uitgerold op http://zuidt.nl/pdokkaart. Graag retest

@JeroenBaltussen
Copy link

Compliment voor de fix.
Hopelijk hiermee release ready for use.

@J-kad
Copy link
Author

J-kad commented Jun 11, 2015

Mooi om te horen, wij zullen dit gaan testen.

@J-kad
Copy link
Author

J-kad commented Jun 11, 2015

Ik zie in de laatste versie overigens enkel de PDOK achtergrondkaart als standaardlaag in de laagfunctionaliteit staan. In de huidige (productie) versie staan ook nog andere lagen. Kunnen deze ook getoond worden? Of moet ik hiervoor een nieuw issue aanmaken?

@rduivenvoorde
Copy link
Collaborator

ik heb een nieuw issue aangemaakt:

#174

en zal deze afsluiten. Nieuwe issues graag gewoon melden.

@J-kad
Copy link
Author

J-kad commented Jun 19, 2015

@rduivenvoorde @JeroenBaltussen helaas is de laatste commit getest en is PDOK kaart nog niet backwards compatible.. Bevindingen van de ontwikkelaar zijn:

  • Bij een directe link en in een iframe zie ik niet de laag selectie en werkt zoomen niet
  • Via object en broncode werkt de hele kaart niet.

@rduivenvoorde
Copy link
Collaborator

@beheerPDOK @JeroenBaltussen

er moet een duidelijker omschrijving worden gegeven of reproduceerbare scenario's van bugs.

WAT werkt niet bij het kopieren van de kaart? Een snelle test bij werkt hier gewoon.
WELKE browsers/versies worden gebruikt. Liefst met gebruikte code of screendumps.

@rduivenvoorde rduivenvoorde reopened this Jun 20, 2015
@J-kad
Copy link
Author

J-kad commented Jun 23, 2015

@rduivenvoorde @JeroenBaltussen

Was inderdaad wat kort omschreven, excuses :) Hierbij een duidelijkere omschrijving:

Ik stuur je zo je via de mail twee bestanden waarmee de embedded kaart op onze test-omgeving mee is getest en versie van onze productie-omgeving. De versie van productie werkt wel, maar op test zie je de eerder gemelde problemen. In een iframe zie je geen laagselectie en werkt zoomen niet. Via object en broncode werkt de hele kaart niet.

De rechtstreekse link die is gebruikt is:
http://test.kaart.pdok.nl/api/api.html?zoom=7&baselayer=BRT&loc=193713.96%2C%20465749.6&pdoklayers=BRT%2CBRTTIJDELIJK%2CBRTGRIJSTIJDELIJK%2CBRTPASTEL%2CLUFO&markersdef=http%3A%2F%2Ftest.kaart.pdok.nl%2Fapi%2Fjs%2Fpdok-markers.js&layersdef=http%3A%2F%2Ftest.kaart.pdok.nl%2Fapi%2Fjs%2Fpdok-layers.js&mloc=194160.27%2C465870.5&titel=Werkplek&tekst=Op%20de%206e%20verdieping&mt=mt14

Hier werkt de laagselectie en zoomen niet.

@rduivenvoorde
Copy link
Collaborator

@raymondnijssen

ik denk dat ik zie wat het probleem is....

in de nieuwe versie zijn er nieuwe checkboxen (EN daarbij parameters) bijgekomen:
&showzoom=true <= wel of niet de blauwe zoomknoppen +/- tonen(!)
&shownavigation=true <= wel of niet een navigation control aanmaken(!)

Probleem hierbij:

  1. Die laatste is wat mij betreft een naamgevingsprobleem: het is niet SHOWnavigation, maar 'navigation': het bepaalt niet of iets wel of niet zichtbaar is, maar of navigatie (== pannen en zoomen) wel werkt of niet.
  2. blijkbaar is zowel showzoom als shownavigation standaard 'False', dus ze werken/zijn pas zichtbaar als je ze expliciet op true zet. Dit breekt idd backwards compatibilitiet. Ze zouden standaard op True moeten staan, en je kunt ze expliciet uitzetten.

Beide lijken kleine fixes, ik kijk even of en wanneer ik tijd heb

@rduivenvoorde
Copy link
Collaborator

De fout met de mij toegestuurde html is het niet juist hebben gekopieerd van de HTML: er is een oude html file gebruikt met een niet volledige head-deel

rduivenvoorde added a commit that referenced this issue Jun 23, 2015
@rduivenvoorde
Copy link
Collaborator

@beheerPDOK @milovanderlinden @raymondnijssen plz retest

ik heb die pdokextenders file overal uitgesloopt en de 3 classen die erin zitten uiteindelijk toch geappend aan de api zelf (zodat ze niet VOOR de api zitten).

ik heb hier rws apps, pdokkaart en vegetatielegger getest. Laat ajb weten wanneer ik dit issue kan sluiten.

@rduivenvoorde
Copy link
Collaborator

@beheerPDOK dit was een foutje in de defaultwaarden. Ik heb die nu zo aangepast dat de url wel zou moeten werken. Plz retest

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

No branches or pull requests

4 participants