Skip to content

Conversation

@nikakhov
Copy link

No description provided.

@hpinkos
Copy link
Contributor

hpinkos commented Nov 30, 2016

Thanks @nikakhov. Can you send us a Contributor Licence Agreement (CLA) so we can review this? You can find instructions here: https://github.com/AnalyticalGraphicsInc/cesium/blob/master/CONTRIBUTING.md#contributor-license-agreement-cla

@pjcozzi
Copy link
Contributor

pjcozzi commented Dec 5, 2016

@nikakhov were you able to send in a CLA? Perhaps I missed it?

@nikakhov
Copy link
Author

nikakhov commented Dec 5, 2016

Let me share a few thoughts. Perhaps we are not yet at that point where this code should be merged. In the issue discussion, I mentioned how this fix is incomplete and needs to be supplemented with additional changes, such as setting minimum and maximum zoom limits.

@pjcozzi
Copy link
Contributor

pjcozzi commented Jan 2, 2017

@nikakhov please open a new pull request when you think this is ready, thanks again for supporting Cesium.

@pjcozzi pjcozzi closed this Jan 2, 2017
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.

3 participants