-
Notifications
You must be signed in to change notification settings - Fork 82
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
Finish maplibre upgrade #5119
Finish maplibre upgrade #5119
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
( | ||
ev: MapMouseEvent & { | ||
features?: Feature<Geometry, GeoJsonProperties>[] | undefined; | ||
} & MaplibreEvent | ||
) => { | ||
(ev: MapLayerMouseEvent) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See MapLayerMouseEvent
is the same type here: https://github.com/maplibre/maplibre-gl-js/blob/91e49458f9c06726550510e94ee549c380c81f38/src/ui/events.ts#L17
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Suspect IssuesThis pull request was deployed and Sentry observed the following issues:
Did you find this useful? React with a 👍 or 👎 |
@nabramow error here after this PR
|
Closes #5106
Upgrading maplibre to the most recent stable version 4.7.1.
v3 had no breaking changes for us.
v4 had these breaking changes that were addressed:
Guide: https://github.com/maplibre/maplibre-gl-js/blob/main/CHANGELOG.md#400
loadImage
maplibre/maplibre-gl-js#3422)Actor
maplibre/maplibre-gl-js#3233)Besides that, I:
MapLayerMouseEvent
as it's the same thing and shorterMaplibreEvent
as it no longer exists in the package and I removed it from related types. It didn't seem to have any effect so assuming this got deprecated and they forgot to document it.Web frontend checklist
yarn format
yarn lint --fix