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

fix: prevent users from loading too many s2 cells #1022

Merged
merged 1 commit into from
Jul 20, 2024

Conversation

TurtIeSocks
Copy link
Collaborator

This relates the current zoom level that the user is at to which S2 cells are able to be rendered, preventing users from loading too many cells and locking up their map.

E.g.

  • User has L10, L15, and L20 cells enabled
  • At zoom level 9 or lower, nothing will render
  • At zoom level 10, only the L10 cells will render
  • At zoom level 15, both 10 and 15 will render, etc

@TurtIeSocks TurtIeSocks merged commit 182b9ac into develop Jul 20, 2024
2 checks passed
@TurtIeSocks TurtIeSocks deleted the s2-cell-zoom-req branch July 20, 2024 16:50
Copy link

🎉 This PR is included in version 1.33.0-develop.9 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant