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

Demo in README not work. Mapzen link broken. #188

Open
yqrashawn opened this issue Oct 17, 2016 · 7 comments
Open

Demo in README not work. Mapzen link broken. #188

yqrashawn opened this issue Oct 17, 2016 · 7 comments

Comments

@yqrashawn
Copy link

No description provided.

@markoradak
Copy link

Any news on this..?

@yqrashawn
Copy link
Author

yqrashawn commented Oct 24, 2016

@markoradak You can create an Api key at mapzen and get your link here using the 'buildings' layer only.

@robhawkes
Copy link
Member

Sorry for the delay! The demo links should be working, for example: http://cdn.rawgit.com/UDST/vizicities/master/examples/basic/index.html

Can you point me toward the exact link which isn't working for you?

As for the Mapzen endpoints, these are working from what I can see based on my testing with the demos and on my local machine. Can you re-test and clarify exactly what isn't working and with which branch of the ViziCities repo?

@renaissance-design
Copy link

renaissance-design commented Feb 23, 2017

Hi Rob - looked in on the project and I'm having the same issue.

screenshot 2017-02-23 16 59 29

A little digging reveals that line 18 of main.js needs updating with the new Mapzen endpoints as per the migration guide.

Ref pull request #210

@SymbolixAU
Copy link

I think the topoJSON resource for buildings now needs to point to

https://tile.mapzen.com/mapzen/vector/v1/buildings/{z}/{x}/{y}.topojson?api_key=vector-tiles-NT5Emiw

@am2222
Copy link

am2222 commented Feb 27, 2017

This is because of their imigration into new vector-tile .It seems we must register for a new api_key

@SymbolixAU
Copy link

The old key seems to still work, it just needs to point to a different url

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants