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

Make sure default interaction with Maps is working #207

Open
2 of 4 tasks
lucernae opened this issue May 5, 2017 · 7 comments · Fixed by #330, #350, #369 or #376
Open
2 of 4 tasks

Make sure default interaction with Maps is working #207

lucernae opened this issue May 5, 2017 · 7 comments · Fixed by #330, #350, #369 or #376
Assignees

Comments

@lucernae
Copy link
Collaborator

lucernae commented May 5, 2017

When you go to Map detail, you will be presented with buttons:

screen shot 2017-05-05 at 19 19 17

What doesn't works:

  • Download map > Download Data Layers (Not working. Is it still relevant when using QGIS Server Backend?)
  • Edit Map > Set Thumbnail (Not working) (Implement or Hide 'Set thumbnail' #238)
  • Edit Map > Map Edit (Not working. Should be redirected to QGIS Server map edit)
  • Edit Map > Map Remove (Not working)

child of #457

@gubuntu
Copy link

gubuntu commented May 30, 2017

download data layers should give asc/geotiff/shapefile dowload links for each layer but instead returns a mangled page.

set thumbnail doesn't throw an error but it doesn't set the thumbnail either.

edit map should take you to the create map page to let you add, remove, re-order layers? 30/5/2017 on staging it just gives a 500.

@gubuntu gubuntu assigned boney-bun and unassigned Gustry Sep 11, 2017
boney-bun added a commit to boney-bun/geonode that referenced this issue Sep 19, 2017
boney-bun added a commit to boney-bun/geonode that referenced this issue Sep 19, 2017
@boney-bun
Copy link

i saw this line of code for removing a map:
delete_map.delay(object_id=map_obj.id)

is there any particular reason why geonode uses celery task to remove the map @lucernae ?

boney-bun added a commit to boney-bun/geonode that referenced this issue Sep 28, 2017
boney-bun added a commit to boney-bun/geonode that referenced this issue Oct 10, 2017
boney-bun added a commit that referenced this issue Oct 12, 2017
* fix default map action for downloading and edit
* fix the flow in maps/views.py
* tidy up unittest for both backends

partially fix #207
boney-bun added a commit to boney-bun/geonode that referenced this issue Oct 13, 2017
boney-bun added a commit that referenced this issue Oct 13, 2017
* fix set thumbnail and remove map
* add logger

partially fix #207
@boney-bun boney-bun reopened this Oct 13, 2017
@gubuntu
Copy link

gubuntu commented Oct 20, 2017

broken
Screen Shot 2017-10-20 at 18.30.15.png

@boney-bun
Copy link

@gubuntu i don't think naming the qlr, wmc, and zip files using map name would be a good idea.
it was previously using this naming convention.
but then i replaced them as it does not represent the content of the map.
the map name was: map30, map 31, map32, etc.
if i recall it correctly, it was @NyakudyaA who proposes using layer name for the naming convention.

do you still want to proceed with using the map name for the files?

@gubuntu
Copy link

gubuntu commented Jul 20, 2018

not the map id, but the user's map name in geonode

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment