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

Fix QGIS layer download text #390

Closed
gubuntu opened this issue Dec 1, 2017 · 4 comments · Fixed by #400 or #409
Closed

Fix QGIS layer download text #390

gubuntu opened this issue Dec 1, 2017 · 4 comments · Fixed by #400 or #409
Assignees
Milestone

Comments

@gubuntu
Copy link

gubuntu commented Dec 1, 2017

Maps download says 'QLR layer' It should say 'QGIS layer file (.qlr)'

Check and fix in Map download and in Layer download

child of #207

@boney-bun
Copy link

I think this issue no longer valid.

On layer download:

Screen Shot 2018-02-05 at 11.47.48 PM.png

On map download:
Do we want to add (.qlr) on the map download?

Screen Shot 2018-02-05 at 11.47.17 PM.png

@boney-bun boney-bun added testing and removed ready labels Feb 5, 2018
@gubuntu
Copy link
Author

gubuntu commented Feb 5, 2018

@boney-bun the layer functionality looks OK but look carefully, the Map one is not fixed as per the OP, so please fix it.

@gubuntu
Copy link
Author

gubuntu commented Feb 19, 2018

@boney-bun the procedure is a tester (me in this case) closes once they are happy that it is working on the testing server (which it isn't yet)

@boney-bun boney-bun removed the testing label Feb 20, 2018
@boney-bun boney-bun reopened this Feb 20, 2018
@gubuntu gubuntu added this to the Mozambique milestone May 8, 2018
boney-bun added a commit to boney-bun/geonode that referenced this issue Jun 19, 2018
boney-bun added a commit that referenced this issue Jul 2, 2018
* fix map functionalities
* replace the map qlr link name with the new wording in test_view
* use layer name for the zip file when download data layers in the map

fix #207
fix #390
fix #152
@gubuntu
Copy link
Author

gubuntu commented Jul 20, 2018

LGTM

@gubuntu gubuntu closed this as completed Jul 20, 2018
@gubuntu gubuntu removed the testing label Jul 20, 2018
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 a pull request may close this issue.

2 participants