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

build(deps): upgrade Open Layers #474

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

jessicamcinchak
Copy link
Member

@jessicamcinchak jessicamcinchak commented Aug 6, 2024

Ran pnpm upgrade ol ol-ext ol-mapbox-style @types/ol-ext

A number of type errors to sort out! Especially still need to debug / adjust a breaking change to our Snap interaction's pointsSource per changelog here: https://github.com/openlayers/openlayers/releases/tag/v9.2.0

See current type errors:

Type 'VectorSource<never>' is not assignable to type 'VectorSource<Feature<Geometry>>'.
  Types of property 'setLoader' are incompatible.

Then might try to get us all the way up to v10.x

Copy link

netlify bot commented Aug 6, 2024

Deploy Preview for oslmap failed. Why did it fail? →

Name Link
🔨 Latest commit 865adfd
🔍 Latest deploy log https://app.netlify.com/sites/oslmap/deploys/66caf3fa3786d900082d3839

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.

1 participant