Skip to content

Support level MultiPolygons#498

Merged
irees merged 2 commits intomainfrom
level-multipolygon
Sep 18, 2025
Merged

Support level MultiPolygons#498
irees merged 2 commits intomainfrom
level-multipolygon

Conversation

@irees
Copy link
Copy Markdown
Contributor

@irees irees commented Sep 17, 2025

No description provided.

@irees irees marked this pull request as ready for review September 18, 2025 08:23
@irees irees requested a review from Copilot September 18, 2025 08:25
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the Level entity's geometry field from Polygon to MultiPolygon across the codebase to support more complex geometric representations of building levels.

Key changes:

  • Updated Go model types to use MultiPolygon instead of Polygon for Level geometry
  • Modified GraphQL schema to reflect the geometry type change
  • Added database migration to convert existing Polygon data to MultiPolygon format

Reviewed Changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 1 comment.

File Description
server/model/models_gen.go Updated LevelSetInput geometry field type to MultiPolygon
server/model/models.go Changed Level struct geometry field from Polygon to MultiPolygon
schema/postgres/migrations/20250916235833_level-multipolygon.up.pgsql Database migration to convert geometry column from Polygon to MultiPolygon
schema/graphql/schema.graphqls Updated GraphQL schema for Level type and LevelSetInput to use MultiPolygon

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@irees irees merged commit 14dd757 into main Sep 18, 2025
6 checks passed
@irees irees deleted the level-multipolygon branch September 18, 2025 08:32
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.

2 participants