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

MapQuestOpen is discontinued. OpenWeatherMap? #18

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

MapQuestOpen is discontinued. OpenWeatherMap? #18

wants to merge 1 commit into from

Conversation

GeorgiaData
Copy link

Added Esri.WorldImagery to replace discontinued MapQuestOpen.Aerial.
Please adjust my inclusion of OpenWeatherMap to provide a sample without the error:
Uncaught Error: No value provided for variable {variant}

Added Esri.WorldImagery to replace discontinued MapQuestOpen.Aerial.
Please adjust my inclusion of OpenWeatherMap to provide a sample without the error:
Uncaught Error: No value provided for variable {variant}
@aparshin
Copy link
Owner

It's better late than never :)

Thank you for replacing MapQuestOpen.Aerial!

But I don't think that OpenWeatherMap is a good improvement for this example: the error can be fixed by replacing OpenWeatherMap with OpenWeatherMap.Clouds, but then I have two more problems:

  1. The OWM servers return 401 Unauthorized code. And I don't know what to do with that.
  2. OWM layer is an overlay - it should be shown above other layers. I don't want to complicate an example by adding basemaps and overlays - I would prefer to keep a simple list of basemaps.

So I suggest removing OWM from this example. I would do this change as a separate commit because I don't have access to your repository. Is it OK to you?

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.

2 participants