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

ol-stac migration #531 #535

Open
wants to merge 13 commits into
base: main
Choose a base branch
from
Open

ol-stac migration #531 #535

wants to merge 13 commits into from

Conversation

m-mohr
Copy link
Collaborator

@m-mohr m-mohr commented Jan 14, 2025

Implements/Fixes

Open issues:

Potential features to be considered:

Breaking changes:

  • Removed maxPreviewsOnMap
  • buildTileUrlTemplate default to null now. The function has a different signature now, an Asset object (see stac-js) is passed as only argument. No other parameters are passen. To get the absolute url for the URL constuction, use asset.getAbsoluteUrl()
  • basemaps.config.js has a new format. Multiple basemaps per planet can be provided. The default behaviour of configureBasemap has changed, including a removal of the map parameter.

@m-mohr m-mohr added the mapping label Jan 14, 2025
@m-mohr m-mohr added this to the 3.4.0 milestone Jan 14, 2025
@m-mohr m-mohr force-pushed the ol-stac branch 2 times, most recently from 13d5ae5 to bcbbf3a Compare January 14, 2025 23:38
@luipir

This comment was marked as outdated.

@luipir

This comment was marked as outdated.

@m-mohr

This comment was marked as outdated.

@luipir

This comment was marked as outdated.

@m-mohr

This comment was marked as outdated.

@luipir

This comment was marked as outdated.

@m-mohr

This comment was marked as outdated.

@luipir

This comment was marked as outdated.

@luipir

This comment was marked as outdated.

@m-mohr

This comment was marked as outdated.

@luipir

This comment was marked as outdated.

@luipir

This comment was marked as outdated.

@m-mohr

This comment was marked as outdated.

@luipir

This comment was marked as outdated.

@luipir

This comment was marked as outdated.

@luipir

This comment was marked as outdated.

@luipir

This comment was marked as outdated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
5 participants