Skip to content

Vector overlay#656

Open
baruchInsert-tech wants to merge 11 commits intoCesiumGS:mainfrom
baruchInsert-tech:vector-overlay
Open

Vector overlay#656
baruchInsert-tech wants to merge 11 commits intoCesiumGS:mainfrom
baruchInsert-tech:vector-overlay

Conversation

@baruchInsert-tech
Copy link

@baruchInsert-tech baruchInsert-tech commented Jan 27, 2026

Description

Issue number or link

Author checklist

  • I have submitted a Contributor License Agreement (only needed once).
  • I have done a full self-review of my code.
  • I have updated CHANGES.md with a short summary of my change (for user-facing changes).
  • I have added or updated unit tests to ensure consistent code coverage as necessary.
  • I have updated the documentation as necessary.

Remaining Tasks

Testing plan

@baruchInsert-tech baruchInsert-tech marked this pull request as draft January 27, 2026 08:54
…ent structure

- TestCesiumGeoJsonObject: tests for child access, feature IDs, properties, and per-feature styling
 - TestCesiumVectorStyle: tests for CesiumColor32, line styles, polygon styles, and style configuration
@j9liu
Copy link
Contributor

j9liu commented Jan 30, 2026

Thank you so much for the contribution @baruchInsert-tech! Just to confirm, would you like a review on this PR? I see that it's marked "Draft", so we weren't sure if it was still in progress.

@baruchInsert-tech baruchInsert-tech marked this pull request as ready for review February 1, 2026 07:22
@baruchInsert-tech
Copy link
Author

Thank you so much for the contribution @baruchInsert-tech! Just to confirm, would you like a review on this PR? I see that it's marked "Draft", so we weren't sure if it was still in progress.

Hello!
this PR is ready for review.
Thank you!

@asafMasa
Copy link

asafMasa commented Feb 4, 2026

@j9liu @kring can you please review this PR?

@asafMasa
Copy link

Thank you so much for the contribution @baruchInsert-tech! Just to confirm, would you like a review on this PR? I see that it's marked "Draft", so we weren't sure if it was still in progress.

Hi @j9liu , is there any update on this PR review?
Do you have an estimation when it can be reviewed?
It solves a long standing issue in unity - https://community.cesium.com/t/display-geojson-data-on-map/27125
Thanks.

@j9liu
Copy link
Contributor

j9liu commented Feb 10, 2026

Hi @asafMasa, sorry for the delay. Our team hasn't had bandwidth in the past few weeks, but I definitely want someone to take a look by next week at the latest.

@baruchInsert-tech
Copy link
Author

Thanks @j9liu and @azrogers for picking this up and slotting it into the March release milestone! Looking forward to the review, happy to make any changes needed to get this merged.

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.

3 participants