Skip to content

Improving underground navigation in Cesium #11362#11363

Merged
dsuren1 merged 3 commits intogeosolutions-it:masterfrom
rowheat02:fix/collisionDetectionSetting
Jul 25, 2025
Merged

Improving underground navigation in Cesium #11362#11363
dsuren1 merged 3 commits intogeosolutions-it:masterfrom
rowheat02:fix/collisionDetectionSetting

Conversation

@rowheat02
Copy link
Copy Markdown
Contributor

@rowheat02 rowheat02 commented Jul 25, 2025

Description

Please check if the PR fulfills these requirements

What kind of change does this PR introduce? (check one with "x", remove the others)

  • Enhancement

Issue

What is the current behavior?

fixes #11362

What is the new behavior?

  • The user can enable and disable Terrain Collision detection settings from the UI.
  • If disabled, the user can navigate underground too.
image

Breaking change

Does this PR introduce a breaking change? (check one with "x", remove the other)

  • Yes, and I documented them in migration notes
  • No

Other useful information

This PR can be closed as all features from the mentioned PR are covered here.

@rowheat02 rowheat02 added this to the 2025.01.02 milestone Jul 25, 2025
@rowheat02 rowheat02 self-assigned this Jul 25, 2025
Copy link
Copy Markdown
Member

@tdipisa tdipisa left a comment

Choose a reason for hiding this comment

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

@rowheat02 you know about #10691 so, I suppose that can be closed in favor of this new one...
You should mention this in the PR description, please.

@tdipisa tdipisa requested a review from dsuren1 July 25, 2025 10:09
@tdipisa tdipisa added the BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch label Jul 25, 2025
@rowheat02 rowheat02 requested a review from tdipisa July 25, 2025 10:15
@dsuren1 dsuren1 enabled auto-merge (squash) July 25, 2025 12:19
@dsuren1 dsuren1 merged commit a5ad349 into geosolutions-it:master Jul 25, 2025
6 checks passed
@dsuren1
Copy link
Copy Markdown
Contributor

dsuren1 commented Jul 25, 2025

@ElenaGallo Kindly test this in DEV, when available and let us know if it's good to be backported. Thanks!

@ElenaGallo
Copy link
Copy Markdown
Contributor

Test passed, @rowheat02 please backport to 2025.01.xx. Thanks

rowheat02 added a commit to rowheat02/MapStore2 that referenced this pull request Aug 11, 2025
…solutions-it#11363)

* Add UI toggle for Cesium collision detection with live map update and tests

* fix: update label

* fix: fix translation
@tdipisa tdipisa linked an issue Aug 11, 2025 that may be closed by this pull request
@tdipisa tdipisa removed the BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch label Aug 11, 2025
tdipisa pushed a commit that referenced this pull request Aug 11, 2025
* Add UI toggle for Cesium collision detection with live map update and tests

* fix: update label

* fix: fix translation
rowheat02 added a commit to rowheat02/MapStore2 that referenced this pull request Aug 13, 2025
…solutions-it#11363)

* Add UI toggle for Cesium collision detection with live map update and tests

* fix: update label

* fix: fix translation
tdipisa pushed a commit that referenced this pull request Aug 13, 2025
#11362 (#11363) (#11383)

* Improving underground navigation in Cesium #11362 (#11363)

* Add UI toggle for Cesium collision detection with live map update and tests

* fix: update label

* fix: fix translation

* skip click on layer should return intersected features test on Cesium tests (#11365)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improving underground navigation in Cesium

4 participants