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.

Sign up for GitHub

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

Jump to bottom

Leaflet voorbeeld werkt niet #39

Open
Erikvv opened this issue Jul 7, 2023 · 4 comments
Open

Leaflet voorbeeld werkt niet #39

Erikvv opened this issue Jul 7, 2023 · 4 comments

Comments

@Erikvv
Copy link

Erikvv commented Jul 7, 2023 •
edited
Loading

Ik heb dit script wat aan moeten passen om het werkend te krijgen

  • URL geodata.nationaalgeoregister.nl werkt niet meer
  • jQuery vervangen door standaard Web API

Huidige versie: https://gist.github.com/ndkv/a9f903c1579ff7609638
Mijn suggestie: https://gist.github.com/Erikvv/b8d141de32850163ee0ffa03517c5d72
@ndkv
Copy link
Contributor

ndkv commented Jul 10, 2023 via email

@Erikvv
Copy link
Author

Erikvv commented Jul 10, 2023

Er wordt hier naar gelinkt op regel 53, 55 en 59

PDOK-NGR-documentatie/docs/quickstart.rst

Line 53 in d276eab

Het `GeoJSON resultaat <http://geodata.nationaalgeoregister.nl/bag/wfs?service=WFS&request=GetFeature&typeName=bag:pand&count=100&startIndex=0&outputFormat=json>`_ kun je bijv. in Leaflet `visualiseren <https://cdn.rawgit.com/ndkv/a9f903c1579ff7609638/raw/01e13989c298330715b8b59194bd1f6512ab475b/index.html>`_ m.b.v. van de ``L.geoJson()`` functie.

Zal ik een PR aanbieden waarbij ik het als code block er in zet ipv als link naar een gist? Volgens mij heeft Spinx ook HTML syntax highlighting.

@ndkv
Copy link
Contributor

ndkv commented Jul 10, 2023

Ik ping @lvdbrink @fterpstra @thijsbrentjens even omdat ik geen pull requests (meer) kan beheren.

@lvdbrink
Copy link
Member

lvdbrink commented Jul 10, 2023 •
edited
Loading

Ik wil wel een PR mergen als @ndkv het even wil reviewen van tevoren.

Erikvv added a commit to Erikvv/PDOK-NGR-documentatie that referenced this issue Jul 14, 2023
@Erikvv
Leaflet: update van voorbeeld
8213fca 
Het leaflet voorbeeld werkte niet meer. Om het werkend te krijgen is het
volgende gewijzigd:

* CDN van de JS en CSS aangepast
* Versie van leaflet aangepast naar de laatste
* Domein van de WFS service aangepast

Verder zijn de volgende best practices toegepast

* JQuery vervangen door Fetch API
* CSS absolute postionering vervangen door Flexbox
* JavaScrip var verangen door const

Ik heb ook het coordinatensysteem gelijkgetrokken zodat het voorbeeld
enkel EPSG:4326 bevat.

Lost op:
- Geonovum#8
- Geonovum#39
@Erikvv Erikvv mentioned this issue Jul 14, 2023
Open
Erikvv added a commit to Erikvv/PDOK-NGR-documentatie that referenced this issue Jul 14, 2023
@Erikvv
Leaflet: update van voorbeeld
530acef 
Het leaflet voorbeeld werkte niet meer. Om het werkend te krijgen is het
volgende gewijzigd:

* CDN van de JS en CSS aangepast
* Versie van leaflet aangepast naar de laatste
* Domein van de WFS service aangepast

Verder zijn de volgende best practices toegepast

* JQuery vervangen door Fetch API
* CSS absolute postionering vervangen door Flexbox
* JavaScrip var verangen door const

Ik heb ook het coordinatensysteem gelijkgetrokken zodat het voorbeeld
enkel EPSG:4326 bevat.

Lost op:
- Geonovum#8
- Geonovum#39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@ndkv @Erikvv @lvdbrink