-
Notifications
You must be signed in to change notification settings - Fork 0
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
Spatial index widgets #34
base: main
Are you sure you want to change the base?
Conversation
…lDataColumn and spatialDataType
…in getModelSource
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, this is nicely structured! A few comments/questions, still working my way though the code review. :)
@juandjara great work here! I think all that's left after #41 is the one other comment above, removing the As the next step, should we publish this as an alpha release so that external examples can be built on it? If you're ready, feel free to do that any time. The steps would:
|
…to feature/sc-454268/spatial-index-widgets
…to feature/sc-454268/spatial-index-widgets
Co-authored-by: Don McCurdy <[email protected]>
related story: https://app.shortcut.com/cartoteam/story/454268/adapt-carto-api-client-to-be-able-to-work-with-dynamic-spatial-index