search

tmcw / geojson.net

https://www.placemark.io/
ISC License
150 stars 32 forks source link

Update to zeit.co/now 2.0 #47

Closed jgimbel closed 5 years ago

jgimbel commented 5 years ago

Zeit's now has upgraded to 2.0. This updates the now.json to use version 2. now 2.0 runs on aws lambda, which means the server needs to be stateless, and not use memory session. To make the server stateless, and not need a database for sessions, I implemented a jwt cookie authentication.

While testing the new authentication, I found the github import function was not hooked up anymore. I added the basic geojson functionality, but this will still need more UI will be needed for error handling, and handling of wkt and esri shapefiles.

PR example

vercel[bot] commented 5 years ago

This pull request is automatically deployed with Now. To access deployments, click Details below or on the icon next to each push.

Joel-MMCC commented 5 years ago

This does fix #46 (WSOD at the demo geojson.net website), but the #45 issue (URL parameters ignored) is still in effect. Pasting a URL copied from geojson.io and changing just the TLD to “.net” complete with zoom level and a coordinates pair into the browser still shows the whole Earth centered on Lat/Long 0,0. Obviously the parameters to pre-create geometry also don’t work.

Example: this URL should show you the “Four Corners” of the USA (the only spot where four states meet) at a tight zoom level. At geojson.NET, it does not: https://geojson.NET/#map=17/36.99887/-109.04478

This otherwise identical URL at the older geojson.IO server works fine: https://geojson.IO/#map=17/36.99887/-109.04478

jgimbel commented 5 years ago

It would appear that the url structure has changed with geojson.net The correct link would be https://geojson.net/#17/36.99887/-109.04478 However, this is not working either. Thank you for letting me know. I will look into this tonight.