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

DataStore Map and Explorer not displaying map tiles since 11 July 2016 #3162

Closed
MrkGrgsn opened this issue Jul 13, 2016 · 6 comments
Closed

DataStore Map and Explorer not displaying map tiles since 11 July 2016 #3162

MrkGrgsn opened this issue Jul 13, 2016 · 6 comments
Assignees

Comments

@MrkGrgsn
Copy link
Contributor

@MrkGrgsn MrkGrgsn commented Jul 13, 2016

Direct tile access to MapQuest maps has been discontinued as of 11 July 2016 and the DataStore Map and Explorer previews no longer display map tiles.

The issue actually lies with recline.js and it has been logged datopian/portal.js#500 and there is a referenced patch to replace MapQuest with Open Street Map frodrigo/recline@3df0c2a

Thought it would be useful to have a record here and request the packaged recline.js should be updated.

MrkGrgsn pushed a commit to MrkGrgsn/ckan that referenced this issue Jul 13, 2016
MrkGrgsn pushed a commit to MrkGrgsn/ckan that referenced this issue Jul 13, 2016
nibecker added a commit to datopian/ckan that referenced this issue Jul 26, 2016
Fixing ckan#3162 Stamen terrain has been chosen due to its open CC license
and clean look
@kfishwick
Copy link

@kfishwick kfishwick commented Aug 9, 2016

We have another solution, implemented for bcgov, that allows you to customize the map tiles for the recline view based on ini parameters (similar to the geoview & spatial extensions). If no customized parameters are set, the default configuration (Stamen) would be used.

Would this be suitable for anyone else, we can do a pull request?

@MrkGrgsn
Copy link
Contributor Author

@MrkGrgsn MrkGrgsn commented Aug 10, 2016

I think that would be great @kfishwick

@nibecker
Copy link

@nibecker nibecker commented Aug 10, 2016

Sounds good @kfishwick

@kfishwick
Copy link

@kfishwick kfishwick commented Aug 10, 2016

Great we will put it together and submit this week.

@jrods jrods mentioned this issue Aug 10, 2016
5 tasks
@amercader
Copy link
Member

@amercader amercader commented Aug 12, 2016

Super stuff, added some comments to the PR

@kfishwick
Copy link

@kfishwick kfishwick commented Aug 15, 2016

Thanks, we have updated the code to address the comments @amercader. Please let us know if it can be incorporated into 2.5.x.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

4 participants