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.27 #11131

Conversation

andreslucena
Copy link
Member

馃帺 What? Why?

Backport #11105 to v0.27

鈾ワ笍 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.27.4 milestone Jul 5, 2023
@alecslupu
Copy link
Contributor

@andreslucena could you have a look here ?

@ahukkanen
Copy link
Contributor

ahukkanen commented Jul 12, 2023

@andreslucena Can I take on this and #11132? There are some differences how assets are handled in 0.26 and 0.27 since the switch to Shakapacker and the changes in the map providers, so this needs a bit more work to make it work properly at 0.27 and 0.26.

@andreslucena
Copy link
Member Author

@ahukkanen go ahead! I didn't have time to focus on these two backports. I'll assign them to you

@ahukkanen
Copy link
Contributor

@alecslupu Pipeline is green here as well.

@ahukkanen
Copy link
Contributor

And just to be sure I also tested it out according to the original testing instructions at #11105 in both versions.

@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 1cf18db into release/0.27-stable Jul 13, 2023
46 checks passed
@alecslupu alecslupu deleted the backport/0.27/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