Skip to content

feat: parse Obstacle GeoJSON from third-party (MAPCO-10524)#29

Merged
asafmas-rnd merged 2 commits into
masterfrom
MAPCO-10524
May 20, 2026
Merged

feat: parse Obstacle GeoJSON from third-party (MAPCO-10524)#29
asafmas-rnd merged 2 commits into
masterfrom
MAPCO-10524

Conversation

@RonIsraeli123
Copy link
Copy Markdown
Collaborator

  • buildLayerQuery: new selection set matching Obstacle/DisplayInObstacle/Identifiers schema
  • geometryParser: maps geography.coordinates + graphicsObjectKind to Point/LineString/Polygon
  • layersClient: filters deleted=true and maps raw obstacles to {id, geom, properties}
  • adds unit tests covering happy + error cases
Question Answer
Bug fix ✔/✖
New feature ✔/✖
Breaking change ✔/✖
Deprecations ✔/✖
Documentation ✔/✖
Tests added ✔/✖
Chore ✔/✖

Related issues: #XXX , #XXX ...
Closes #XXX ...

Further information:
https://mapcolonies.atlassian.net/browse/MAPCO-10524

- buildLayerQuery: new selection set matching Obstacle/DisplayInObstacle/Identifiers schema
- geometryParser: maps geography.coordinates + graphicsObjectKind to Point/LineString/Polygon
- layersClient: filters deleted=true and maps raw obstacles to {id, geom, properties}
- adds unit tests covering happy + error cases

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 15, 2026

🎫 Related Jira Issue: MAPCO-10524

import { buildLayerQuery } from '../layersClientModel';
import { geographyToGeoJSON, type RawGeography } from './geometryParser';

interface RawLayerObject {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why is it not defined in "types/thirdParty"?

Comment on lines +6 to +7
createdBy
creationTime
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Do we need to query it?

createdBy
creationTime
deleted
entityVersion
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Do we need to query it?
Check all fields with the product

@asafmas-rnd asafmas-rnd merged commit f9e0306 into master May 20, 2026
7 checks passed
@asafmas-rnd asafmas-rnd deleted the MAPCO-10524 branch May 20, 2026 07:48
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