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

set thumbnail function for geonode-client #3151

Merged
merged 7 commits into from
Jul 21, 2017
Merged

set thumbnail function for geonode-client #3151

merged 7 commits into from
Jul 21, 2017

Conversation

hishamkaram
Copy link
Contributor

@hishamkaram hishamkaram commented Jul 1, 2017

  • Issues
    • set thumbnail button not working with geonode-client
  • solution:
    • geonode-client use boundless sdk which contains a component to create thumbnail or create image for current view as blob so I created a function to post this image data (blob) to our endpoint here.
    • but our end point generate thumbnail server-side so we need to skip this step if preview=='react'(a flag in the request to identify sender).

this PR skip generating image (server-side) and save blob data as thumbnail image

@ehl120
Copy link

ehl120 commented Jul 12, 2017

I'd like remove the thumbnails place holder on bother layers/ and maps/ templates because the thumbnails don't work for me either. However the layouts (with panels.html) folder doesn't exist in this path:

/usr/local/lib/python2.7/dist-packages/geonode/layers/templates

Where to customize the page layout? Thanks a bunch.

@afabiani
Copy link
Member

@hishamkaram there are failures with Travis, can you check?

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.007%) to 46.979% when pulling 56794c2 on hishamkaram:master into a70ecbe on GeoNode:master.

@hishamkaram
Copy link
Contributor Author

@afabiani Done

@afabiani
Copy link
Member

thanks @hishamkaram

@afabiani afabiani merged commit 4428ac9 into GeoNode:master Jul 21, 2017
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.

4 participants