Skip to content

Integrate projections in MapState#288

Merged
boeckMt merged 38 commits intomainfrom
fix-ci
Mar 18, 2026
Merged

Integrate projections in MapState#288
boeckMt merged 38 commits intomainfrom
fix-ci

Conversation

@boeckMt
Copy link
Copy Markdown
Member

@boeckMt boeckMt commented Mar 5, 2026

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • Other... Please describe:

What is the current behavior?

Issue Number: #204 and ci for publish GitHub Packages is not working.

What is the new behavior?

The package @dlr-eoc/services-map-state can now register and set projection, which has already been implemented in @dlr-eoc/map-ol. The packages @dlr-eoc/map-cesium and @dlr-eoc/map-maplibre only use a small part of it to set the current MapState.

Does this PR introduce a breaking change?

Other information

Is it part of one/more packages and which?

  • @dlr-eoc/services-map-state
  • @dlr-eoc/ngx-ukis-ui-clarity
  • @dlr-eoc/map-ol
  • @dlr-eoc/map-maplibre
  • @dlr-eoc/map-cesium
  • demo-maps

Unfortunately, the tests in Chrome's headless do not work now and this fix ci: try to fix WebGL error in actions test was unsuccessful.

boeckMt added 20 commits March 2, 2026 14:08
…ition now needs inputs of mapSvc and mapState
…egister proj4 projections and use inputs of MapState.setProjection #204
@boeckMt boeckMt changed the title Set Projection Integrate projections in MapState Mar 6, 2026
@lucas-angermann
Copy link
Copy Markdown
Contributor

Thank you, that fixed it!

Copy link
Copy Markdown
Contributor

@lucas-angermann lucas-angermann left a comment

Choose a reason for hiding this comment

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

Looks good, thank you for the changes!

@boeckMt boeckMt merged commit 73b3d7f into main Mar 18, 2026
16 checks passed
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