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.

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

PB-1204 : split CesiumMap component into multiple sub-parts #1170

Merged
merged 1 commit into from
Dec 13, 2024

Conversation

pakb
Copy link
Contributor

@pakb pakb commented Dec 13, 2024

with the same logic/approach that we had with the OpenLayers counterpart, leaving only the responsibility to create and share the Cesium instance to the main component, and giving all others (interactions, layer management, feature highlighting) to sub-components.

Test link

Copy link

cypress bot commented Dec 13, 2024

web-mapviewer    Run #4098

Run Properties:  status check passed Passed #4098  •  git commit 7ef9136f74: PB-1204 : split CesiumMap component into multiple sub-parts
Project web-mapviewer
Branch Review feat-PB-1204-split-cesium-map-component
Run status status check passed Passed #4098
Run duration 04m 46s
Commit git commit 7ef9136f74: PB-1204 : split CesiumMap component into multiple sub-parts
Committer Pascal Barth
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 21
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 216
View all changes introduced in this branch ↗︎

@pakb pakb force-pushed the feat-PB-1204-split-cesium-map-component branch 2 times, most recently from ae7c571 to f778b52 Compare December 13, 2024 10:26
with the same logic/approach that we had with the OpenLayers counterpart, leaving only the responsibility to create and share the Cesium instance to the main component, and giving all others (interactions, layer management, feature highlighting) to sub-components.
@pakb pakb force-pushed the feat-PB-1204-split-cesium-map-component branch from f778b52 to 7ef9136 Compare December 13, 2024 10:51
@pakb pakb requested a review from sommerfe December 13, 2024 11:02
@pakb pakb merged commit d4914df into develop Dec 13, 2024
6 checks passed
@pakb pakb deleted the feat-PB-1204-split-cesium-map-component branch December 13, 2024 11:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants