Skip to content

Conversation

@russcam
Copy link
Contributor

@russcam russcam commented Aug 2, 2016

  • Add Relation to geo_shape queries

    GeoShapeRelation was in the codebase but looks like it was missed in the geo refactoring

  • Add contains to geoshape relations.

Needs to be forward ported to 5.x

Add Contains to geoshape relations
@gmarz
Copy link
Contributor

gmarz commented Aug 2, 2016

LGTM

@michaelbudnik
Copy link
Contributor

michaelbudnik commented Aug 5, 2016

Any ETA on a release which will contain this fix?

@gmarz gmarz merged commit 03d5ac4 into 2.x Aug 5, 2016
@gmarz gmarz deleted the fix/2.x-georelation branch August 5, 2016 16:23
@gmarz
Copy link
Contributor

gmarz commented Aug 5, 2016

Merged to 2.x so we can push out a release today. Still needs porting to master though @forloop.

@gmarz
Copy link
Contributor

gmarz commented Aug 5, 2016

@michaelbudnik 2.4.3 is now out on nuget.org

@michaelbudnik
Copy link
Contributor

@gmarz Many thanks for this!

russcam added a commit that referenced this pull request Aug 6, 2016
Add Contains to geoshape relations.
Forward port of PR #2187
@russcam
Copy link
Contributor Author

russcam commented Aug 7, 2016

Ported to master in e28e03e 👍

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