search

jonge-democraten / website

JD website
https://jongedemocraten.nl
MIT License
6 stars 2 forks source link

Add Wageningen to department map #214

Closed bartromgens closed 3 years ago

bartromgens commented 4 years ago

Adds Wageningen to department map by defining it in 'gemeente codes (2013)'.

See https://github.com/jonge-democraten/website/blob/master/website/static/website/json/gemeenten_2013.topo.json for gemeente codes with name and geometry.

bartromgens commented 4 years ago

Example of the new map: wageningen

bartromgens commented 4 years ago

Gemeenten + codes used to define Wageningen:

Wageningen 289 Barneveld 203 Rhenen 340 Veenendaal 345 Buren 214 Tiel 281 Renswoude 339 Scherpenzeel 279 Ede 228 Neder-Betuwe 1740

mathemaat commented 4 years ago

Hoi @bartromgens

Bedankt voor het oplossen! Ik ben alleen niet dat ik de aangewezen persoon om dit te reviewen. Ten eerste weet ik niet precies waar en hoe dit gebruikt wordt. Ten tweede heb ik hier geen werkende versie van de website vanwege dat probleem met Django versie 1.8 (t/m 1.10) dat ik enige tijd geleden gerapporteerd heb.

Uit de mailwisseling begrijp ik dat Pieter en Yvet hier naar kijken. Ik neem dan ook aan dat mijn review dan ook niet meer zo hard nodig is.

bartromgens commented 4 years ago

@mathemaat Ik heb je voornamelijk toegevoegd zodat je bewust bent van een wijziging. Je hoeft er verder niks mee.

Ten tweede heb ik hier geen werkende versie van de website vanwege dat probleem met Django versie 1.8 (t/m 1.10) dat ik enige tijd geleden gerapporteerd heb.

Ik ben niet goed van op de hoogte van dat probleem, maar ik wil best een keer proberen de website naar Django 1.11 to upgraden. Dat zou volgens mij niet heel lastig moeten zijn.

bartromgens commented 3 years ago

Niet meer relevant want kaartje is niet meer online te vinden.