Laatste versie PDOK Kaart op test-omgeving PDOK (productie-versie eind september)

[J-kad]

@milovanderlinden @ndkv @Geoneer @raymondnijssen @rduivenvoorde @LuukS @GreenUtil @mprins @BryanDeNijs @JeroenBaltussen

Wij hebben de laatste wijzigingen doorgevoerd op de laatste commit op Github van PDOK kaart. Wij hebben het volgende gewijzigd:

De Kaart is nu volledig backwardscompatible. De achtergrondlagen in de huidige versie van Kaart worden nu ook getoond op basis van de nieuwe versie (in zowel de link als in de html sectie).

Er kan getest worden op: http://test.kaart.pdok.nl/

Deze door ons aangepast versie kunnen wij eenvoudig doorzetten naar productie. De planning is om dit eind september te releasen dus mochten er bevindingen zijn (enkel op de aangepaste functionaliteit, geen nieuwe issues) graag uiterlijk volgende week melden. Het is voor ons overig op dit moment veel werk om volledige nieuwe commits over te nemen. Wij kunnen namelijk niet eenvoudig de code van Github (geonovum sectie) overnemen omdat het binnen de huidige manier van uitrollen van software binnen PDOK niet mogelijk is om omgevingsparamaters hardcoded in de code te laten zitten. Bijvoorbeeld in de “pdok-api.js” van de huidige van de community. Waardes in dit bestand staan hardcoded welke wij niet kunnen teruggeven zonder de code van de community niet meer werkend te maken. Als community moeten we eerst een oplossing voor dit probleem vinden zodat PDOK en de community met dezelfde code werken. PDOK wil graag met een passende oplossing (voorstel) komen op korte termijn.

[JeroenBaltussen]

Eenvoudig kaart gemaakt zonder markers, maar er verschijnt geen beeld.

foutmelding in console:

• Uncaught ReferenceError: pdokProperties is not defined(anonymous function) @ pdok-api.js:12
  api.html?
• mapdiv=map_vialink&zoom=3&showscaleline=true&showmouseposition=true&loc=170000%2C 470000&m…:18 Uncaught TypeError: Pdok.addcss is not a function
  gebruikte link:
  http://test.kaart.pdok.nl/api/api.html?mapdiv=map_vialink&zoom=3&showscaleline=true&showmouseposition=true&loc=170000%2C%20470000&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&pdoklayers=BRT%2CLUFO

[rduivenvoorde]

Jeroen, kan het zijn dat je met een gecachede versie kijkt? Ik heb die melding niet.

En die func zit echt in pdok-api:

https://github.com/Geonovum/pdokkaart/blob/master/api/js/pdok-api.js#L129

ik heb wel een nader probleem (waardoor ik de iconen niet zie) als ik een markertje zet)....
maar dat werkt bij mij wel gewoon:

http://zuidt.nl/pdokkaart

[rduivenvoorde]

Als ik een kml marker zet en de link laat, dan is in die pagina

Pdok.ApiUrl = "undefined/api"

omdat er in de pdok-markers.js url's staan als:

Pdok.ApiUrl + '/markertypes/emblem-notice.png'

gaat dat niet goed.. vraag me af waar die undefined vandaan komt....

Ah dat komt hiervandaan:

// PDOK LOKET CONFIGURATION
Pdok.ApiUrl = pdokProperties.kaarturl + '/api';

die pdokProperties.kaarturl is niet aanwezig in de api.html pagina blijkbaar...

[J-kad]

@JeroenBaltussen : het feit dat je er niet bij kon kan ook te maken hebben dat een collega nog even iets moest uitrollen. Als het goed is kun je er nu in ieder geval gewoon bij.

[J-kad]

@rduivenvoorde : marker wordt inderdaad niet getoond. Bedankt voor je nadere toelichting. Wij gaan hier naar kijken!

[J-kad]

Het issue met de markers is opgelost (zie http://test.kaart.pdok.nl/). Als er verder geen bevindingen zijn dan kan deze versie naar productie.

Wat nog geregeld moet worden om onze code te pushen naar github (https://github.com/Geonovum/pdokkaart) is een account om een branch toe te kunnen voegen. Met ons huidig account kunnen wij dit niet..

Kan iemand dit regelen?

[ndkv]

We zijn bijna klaar met beheerstrategie. Een dezer dagen brengen ik het in uitvoering en zal de boel inrichten zodat jullie een pull request kunnen sturen.

[J-kad]

Allen, zoals besproken met @JeroenBaltussen hebben wij de nieuwe versie van PDOK kaart naar productie gebracht (zonder verder communicatie). Op 30 september zal communicatie (breed) gaan plaatsvinden. @ndkv kun jij een account regelen? (zie één na laatste issue).

[ndkv]

Account voor? Een na laatste issue is #176 maar die bedoel je denk ik niet...?

[J-kad]

@ndkv: een account om een branch toe te kunnen voegen (zie onderstaande). Kun jij dit regelen?

"Wat nog geregeld moet worden om onze code te pushen naar github (https://github.com/Geonovum/pdokkaart) is een account om een branch toe te kunnen voegen. Met ons huidig account kunnen wij dit niet.."

[ndkv]

@beheerPDOK Ik heb jullie write rechten gegeven voor deze repository.