search

mapbox / mbtiles-spec

specification documents for the MBTiles tileset format
https://www.mapbox.com/help/an-open-platform/#mbtiles
Other
610 stars 99 forks source link

Remove `type` from metadata spec #28

Open willwhite opened 12 years ago

willwhite commented 12 years ago

TileMill doesn't write this and I don't know of any clients that read this now.

robpvn commented 11 years ago

There's a saying that goes " required is forever", and I think that removing something that is currently required is not a good idea once the spec is out in the wild. After all, there might be applications reading/writing this that you don't know about.

tmcw commented 11 years ago

@robpvn in protobuf, yes. But if we can find a relatively large segment of clients that aren't implementing this part of the spec, it makes sense not to have unnecessary historical baggage.

incanus commented 11 years ago

Uh, version it?

robpvn commented 11 years ago

Fair enough. Would it be cut out completely from the spec or moved to suggested? Because I can still see the rationale for having information about overlay or baselayer in an application that opens multiple MBTiles datasources. Obviously, if it is cut out from the spec it would still be possible for people to use it as an option but it would be less, shall we say, discoverable for implementors.