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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Backport 'Fix issues with overriding maps and loading Leaflet' to v0.26 #11132

Conversation

andreslucena
Copy link
Member

馃帺 What? Why?

Backport #11105 to v0.26

鈾ワ笍 Thank you!

* Fix issues with overriding maps and loading Leaflet

* Remove commented configuration

* Prevent external requests during the custom map spec

* Test that the custom map also shows the marker correctly

* Correct the dummy tiles route for OSM maps

* Apply review recommendations
@andreslucena andreslucena added backport Pull Requests that are a backport for a fixed bug module: core type: fix PRs that implement a fix for a bug labels Jul 5, 2023
@alecslupu alecslupu added this to the 0.26.8 milestone Jul 5, 2023
@alecslupu
Copy link
Contributor

@andreslucena could you have a look here ?

@ahukkanen
Copy link
Contributor

@alecslupu Pipeline is green on this one (except for codecov which does not work always).

@alecslupu alecslupu self-assigned this Jul 12, 2023
Copy link
Contributor

@alecslupu alecslupu left a comment

Choose a reason for hiding this comment

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

The code LGTM

@alecslupu alecslupu merged commit 5c8852c into release/0.26-stable Jul 13, 2023
48 checks passed
@alecslupu alecslupu deleted the backport/0.26/fix-issues-with-overriding-map-11105 branch July 13, 2023 06:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport Pull Requests that are a backport for a fixed bug module: core type: fix PRs that implement a fix for a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants