CartoDB / torque

Temporal mapping for CARTO
http://cartodb.github.com/torque/
BSD 4-Clause "Original" or "Old" License
397 stars 129 forks source link

Can we request with torque.json instead of json.torque? #190

Open rochoa opened 9 years ago

rochoa commented 9 years ago

Windshaft provider requests torque tiles with json.torque format/extension: https://github.com/CartoDB/torque/blob/2.11.3/lib/torque/provider/windshaft.js#L478

To be more consistent, could we change that to torque.json?

fdansv commented 9 years ago

I think in Torque the change would be easy to perform, and I like better torque.json. Are there any other implications apart from changing the provider in Torque, @javisantana ?

javisantana commented 9 years ago

no, but we'd need to keep backwards compatibility in the tiler

rochoa commented 9 years ago

Windshaft will support both.

javisantana commented 9 years ago

so that's ok, we can change it

javisantana commented 9 years ago

I was wondering why I had the idea of calling json.torque instead of torque.json in the same way utfgrid is grid.json. That's why code reviews are good