mapbox / tilejson-spec

JSON format for describing map tilesets.
255 stars 52 forks source link

version 3.0.0 #36

Closed mapsam closed 3 years ago

mapsam commented 6 years ago

This is the main branch for version 3.0.0 development of the tilejson specification. Descriptions of the scope of v3 can be found in #35. All open tickets that are under discussion for v3 are labelled as v3.

Lead authors are @mapsam and @GretaCB

We'll be making separate pull requests for each field into this branch so we can keep discussions about individual properties properly located. If you have comments about the general structure of the 3.0.0 specification, feel free to leave them here!

TODO - this list will be updated as we make pull requests and merge into this branch

Overall

Notes/additions/updates

Add content/review structure sections

Post release TODO

cc @mapbox/core-tech

mapsam commented 6 years ago

@GretaCB do you have an idea of what order we should put the fields in the Structure section in? Right now they are listed alphabetically, but they could also go in order of "REQUIRED" first and "OPTIONAL" second, or something else entirely.

cc @flippmoke who may have some thoughts here

flippmoke commented 6 years ago

@mapsam I think required first might be best, but alphabetical works as well. I would just make sure its quickly clear what is required somehow so that it can be "checked" off if you were reading the spec and implementing it.

GretaCB commented 6 years ago

"REQUIRED" first and "OPTIONAL" second

👍 agreed, and alphabetical within those sections?

mapsam commented 6 years ago

and alphabetical within those sections?

Sounds good! Thanks both of ya 🙏 Let's plan on re-ordering at the end so we can focus on writing now. Will add as a checkbox before we merge.

GretaCB commented 6 years ago

Update

Both @mapsam and I (@gretacb) are out on vacation for the next two weeks, therefore this push will be temporarily paused until we're back. Of course comments and perspectives are absolutely welcome in the meantime, we'd love to hear your thoughts.

Super excited to come back refreshed and ready to push this through. Thanks again for everyone's patience and contributions so far, and have a wonderful couple of weeks! 👋 🏖

mapsam commented 6 years ago

@GretaCB let's plan on including the fillzoom property in v3 as well, since we can benefit from a single point of reference for this complicated topic. I'll make a PR into this branch so we can discuss there! https://github.com/mapbox/tilejson-spec/issues/27

mapsam commented 6 years ago

Update: Carol and I have added all planned updates and changes based on external and internal input. Next steps are to get more eyes and reviews on the final draft. Tagging @samanpwbb @ian29 @alliecrevier @flippmoke @stevage for starters, please feel free to review if you have time (but no pressure if you don't)!

Our plan is to have this merged into master next week.

mapsam commented 6 years ago

Also cc @andrewharvey if you'd like to give this a review!

lukasmartinelli commented 4 years ago

Why is this not merged?! @mapsam

stevage commented 3 years ago

Has this been officially abandoned?

dqunbp commented 3 years ago

Is it officially abandoned?

mapsam commented 3 years ago

Thank you all for the reviews and feedback!

This was abandoned a couple years back due to a shift in work priorities. Apologies for not being clear about this. I've spent some time cleaning up the PR and think this is ready to be merged. For transparency, please note that the api.mapbox.com servers have not been updated to reflect "tilejson": "3.0.0" at this time.