Skip to content

Conversation

@xsaco07
Copy link
Contributor

@xsaco07 xsaco07 commented Jun 7, 2019

This PR fix issue #96, #125, #127

mapTileSources

The options CycleMap and MapQuest are not available any more for using them in the app.

The only source available that woks in the same way is MAPNIK so we decided that, by the moment, the app will use by default the MAPNIK tile source. If this is the best for the app and users, we can let this change be permanent.

With this PR the map tile source can not be changed. This is for avoiding exceptions and because MAPNIK will be used anyway if the selected source does not work correctly (throws an exception) and this will happen if MapQuest or CycleMap are chosen.

In conclusion the functionality of choosing the map tile source was useless right now.

@jamescr jamescr added this to the v0.7.2 milestone Jun 7, 2019
@EmmanuelAB
Copy link
Contributor

I checked this PR and indeed now the option to select the tile provider has been removed imagen

@jamescr jamescr merged commit 299d98f into labexp:develop Jun 14, 2019
@xsaco07 xsaco07 deleted the mapTilesSources branch July 10, 2020 21:51
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.

3 participants