Skip to content

MB-66517: Added circles support for geometry collections#27

Merged
abhinavdangeti merged 3 commits into
masterfrom
circles
May 15, 2025
Merged

MB-66517: Added circles support for geometry collections#27
abhinavdangeti merged 3 commits into
masterfrom
circles

Conversation

@Likith101

Copy link
Copy Markdown
Member

No description provided.

Comment thread geojson/geojson_shapes_util.go Outdated
Comment thread geojson/geojson_shapes_util.go Outdated
abhinavdangeti
abhinavdangeti previously approved these changes May 14, 2025
@CascadingRadium

CascadingRadium commented May 15, 2025

Copy link
Copy Markdown
Member

shouldn't you add Circle in

func (gc *GeometryCollection) UnmarshalJSON(

also that method is missing EnvelopeType as well, add both Circle and Envelope

Comment thread geojson/geojson_shapes_util.go
@abhinavdangeti abhinavdangeti merged commit ed70de5 into master May 15, 2025
12 checks passed
@abhinavdangeti abhinavdangeti deleted the circles branch May 15, 2025 14:00
abhinavdangeti added a commit to blevesearch/bleve that referenced this pull request May 15, 2025
Linked Geo PR - blevesearch/geo#27

---------

Co-authored-by: Abhinav Dangeti <abhinav@couchbase.com>
CascadingRadium pushed a commit to blevesearch/bleve that referenced this pull request Aug 21, 2025
Linked Geo PR - blevesearch/geo#27

---------

Co-authored-by: Abhinav Dangeti <abhinav@couchbase.com>
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.

4 participants