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.

Sign up for GitHub

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

Jump to bottom

VNext move all spatial types to spatial namespace #1102

Closed
j82w opened this issue Dec 13, 2019 · 4 comments
Closed

VNext move all spatial types to spatial namespace #1102

j82w opened this issue Dec 13, 2019 · 4 comments
Labels
VNext Next Major version of SDK

Comments

@j82w
Copy link
Contributor

j82w commented Dec 13, 2019

There is a mix of spatial types in the root namespace that should be moved to the spatial namespace.
@j82w j82w added Required for V4 GA VNext Next Major version of SDK and removed Required for V4 GA labels Dec 13, 2019
@j82w j82w changed the title Move all spatial types to spatial namespace VNext move all spatial types to spatial namespace Dec 13, 2019
@ealsur
Copy link
Member

ealsur commented Dec 17, 2019

This was already done in #1072

Any new Spatial types will go to the Spatial namespace in vNext

@j82w
Copy link
Contributor Author

j82w commented Dec 17, 2019

@ealsur take a look at PR #1097 . It adds new types Geo types that are under the common namespace.

@ealsur
Copy link
Member

ealsur commented Dec 17, 2019

Yup, saw it, but that change is not in vNext. Any new change I port to vNext, if its on the client surface/rest contract, I do the namespace adjustments 😄

@j82w
Copy link
Contributor Author

j82w commented Dec 17, 2019

I'm closing this issue then.

@j82w j82w closed this as completed Dec 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
VNext Next Major version of SDK
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ealsur @j82w