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

fix: tidy up geometry and bbox TDE-522 #227

Merged
merged 2 commits into from
Nov 24, 2022
Merged

fix: tidy up geometry and bbox TDE-522 #227

merged 2 commits into from
Nov 24, 2022

Conversation

amfage
Copy link
Contributor

@amfage amfage commented Nov 23, 2022 •
edited
Loading

The bbox and geometry were coerced into the existing code when TDE-502 was done, this is to tidy that up.

@amfage amfage requested a review from a team as a code owner November 23, 2022 01:28
@amfage amfage marked this pull request as draft November 23, 2022 01:29
@amfage amfage marked this pull request as ready for review November 23, 2022 01:31
@kodiakhq kodiakhq bot merged commit 02b06b5 into master Nov 24, 2022
@kodiakhq kodiakhq bot deleted the refactor/bbox-geometry-tde-522 branch November 24, 2022 20:09
@github-actions github-actions bot mentioned this pull request Nov 24, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@paulfouquet paulfouquet paulfouquet approved these changes

@blacha blacha Awaiting requested review from blacha blacha is a code owner automatically assigned from linz/li-topo-data-engineering

@MDavidson17 MDavidson17 Awaiting requested review from MDavidson17 MDavidson17 is a code owner automatically assigned from linz/li-topo-data-engineering

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@amfage @paulfouquet