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

update cinnabar style to 7.0.1 #323

Merged
merged 2 commits into from
Jun 14, 2017

Conversation

matkoniecz
Copy link
Member

fixes part of #199 (display barriers that are important during survey)

type: TopoJSON #GeoJSON, MVT, TopoJSON
url: https://tile.mapzen.com/mapzen/vector/v1/all/{z}/{x}/{y}.topojson
#url: '//tile.dev.mapzen.com/mapzen/vector/v1/all/{z}/{x}/{y}.topojson'
#url: //localhost:8080/vector/all/{z}/{x}/{y}.json
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This must stay MVT!

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Interesting. I missed it, but I tested it and it works nicely (at my phone at least)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, it accepts any of these formats. But the binary MVT is smaller, this makes a difference with this many users.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Switched back to MVT, it should be now ready.

@westnordost westnordost merged commit f4abb97 into streetcomplete:master Jun 14, 2017
@westnordost
Copy link
Member

cool, thanks

@westnordost
Copy link
Member

f4abb97#commitcomment-22538246

Could you please change the url to the mvt url as well and use http (not https)?

@matkoniecz
Copy link
Member Author

done and documented in #328

@matkoniecz matkoniecz deleted the cinnabar-update branch June 14, 2017 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants